Issue metadata
Sign in to add a comment
|
setInnerHTML very slow in m50 for Google Keep |
||||||||||||||||||||||
Issue descriptionVersion: m50 OS: Linux (at least) What steps will reproduce the problem? (1) Open Google Keep (2) Enter a very long note with html content. I just used "Text with https://crbug.com" repeated about 50 times. (3) Click on done and wait for something to happen What is the expected output? Something happens fast What do you see instead? In m50, but not m49 or m51, it takes a very long time to setInnerHtml the formatted text. The Keep team may be able to get us a smaller reproduction case. But it seems whatever is was has been fixed for m51, and maybe needs to be merged back in to m50.
,
Apr 27 2016
I'm with Keep team. I tried to create a repro case in JSFiddle, but the problem was only reproducing for me in the JS console in the Keep tab. Perhaps related to how many resources are available in the tab? Still looking into it. Thanks. I'll reopen as necessary
,
Apr 27 2016
Oh -- looks like I can't close it on my own.
,
Apr 28 2016
I could reproduce this on Linux 50.0.2661.86 (Official Build) (64-bit). I don't see innerHTML lighting up in traces though; it appears the main thread is spending that time in Keep JavaScript. However there's something weird going on, the compositor and browser are very busy animating and painting something, despite nothing really happening on screen. Could you provide a trace or show me what points to innerHTML? Do you have any non-obfuscated debug frontends up somewhere I could poke at? From our side we could try bisecting this to find the regression and progression.
,
Apr 28 2016
if on corp network or vpn, url param jsmode=du will server unobfuscated JS. keep.google.com?jsmode=du
,
May 2 2016
If possible, could anyone attach any reproduction case or jsfiddle to try a repro and reverse bisect to find the CL that fixed this issue in M-51.
,
May 9 2016
I took a look at this with the unobfuscated code in 50.0.2661.94 (Official Build) (64-bit) Linux. Pasting ~20 copies of this bug text hangs the renderer for ~1.5s doing layout that seem to be triggered by the spell checker. Then theres ~14s thunking through something that's proxying event dispatching. Is there something going on in Keep with event handlers? I don't see innerHTML setting showing up on this trace.
,
May 10 2016
I was unable to reproduce this on Linux Ubuntu 14.04 on the latest stable(50.0.2661.94). Didn't observe any slowness or lag on saving the Note as per C#0 and on the URL: keep.google.com?jsmode=du Could anyone review the screen-cast and confirm if anything being missed here & if the issue is seen on Ubuntu at all.
,
May 18 2016
,
May 26 2016
Stable is rolling out to 51.0.2704.63 and as per C#0 Issue is M-50 specific. schenney@: Could you please confirm if we can close this.
,
Jun 3 2016
Dominic does this bug need to get sent to the layout team?
,
Jun 3 2016
I think now that m51 has shipped to stable we can ignore this - the bug didn't repro there. Furthermore, the Keep team has made changes to mitigate the problem which has had beneficial performance results regardless. So now it will be hard to debug and fix. I also suspect that this is the same issue that is hitting gmail very hard related to Range objects and GC, since the reproduction actions are quite similar (putting in lots of text that requires formatting in some form). I vote for WontFix at this time.
,
Jun 3 2016
WontFix is okay with us (Keep Team)
,
Jun 6 2016
,
Jun 4 2018
|
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by schenney@chromium.org
, Apr 27 2016