win32ss / supermium

Chromium fork for Windows XP/2003 and up
https://win32subsystem.live/supermium/
BSD 3-Clause "New" or "Revised" License
2.46k stars 85 forks source link

[Functional, Annoying, 126 R3] DevTools hangs on long inlined JS in certain cases #869

Open XakerTwo opened 2 months ago

XakerTwo commented 2 months ago

Description DevTools hangs on long inlined JS

not tested with files, mostly i have this issue with file-less code (VM-ed) and i'm well sick of it when DevTools hangs, it consume entire core and slowly grow in RAM consumtion. right now i have one, hanged for an hour and almost 1GB.

To Reproduce

  1. Some how get and open to view well long minified code in sources category (e.g. refernce to VM in console)
  2. disable pretty print for it (if there no error)
  3. switch to console and set focus to input(so you will see blinking cursor)
  4. now try switch back to the sources category

Desktop:

Additional context while rederer of DevTools continue to work but interaction not works, the page has opposite issue - DevTools UI part not updates but still interactive if tab was switched by browser(e.g. breakpoint) it's ok and even tab with syntax error can be closed, just not focusing it pretty print save from this situation, but on syntax error it's not work. syntax typo like that(missing semicolon after method call)

x=Date.now()y=6

search (#807) not involved, but i aware about SM 126 and possibly related fix. if my scenario is reproducible for someone else on SM 124 but not SM 126 let me know. Looking at amount of issues with 126 i'm not ready for update right now.

XakerTwo commented 2 months ago

Confirmed - it's actual and reproducible on SM 126, but slightly different steps chain

  1. Open long minified js without pretty print(e.g. from console by clicking on VM reference)
  2. Close it - now it closed correctly(i tested it once, probably it closed by occassion)
  3. Open it again - bingo and we hang and leak again
screenshot ![dev-tool_inlined-js-hang](https://github.com/user-attachments/assets/d4ff9c9f-2125-4247-a7e4-72082e2be443)