| Issue 8653: | CPU pegged, if scrolled on flash | |
| 6 people starred this issue and may be notified of changes. | Back to list |
Sign in to add a comment
|
Reproduces on 2.0.169.0 (Official Build 11238) and also on Trunk (r11412) -Navigate to finance.google.com and search for quote (eg: GOOG) -Keep mouse pointer on the chart and scroll up/down with the wheel. That's it. The browser almost enters into a un-usable state for a while and you will receive the Flash plugin unresponsive dialog. |
||||||||||||||||||||||
,
Mar 10, 2009
Jon, If you could please triage this. Whatever is going on causes all of Chrome to freeze (perhaps a backend IO Thread issue and w/ plugins).
Cc: j...@chromium.org
Labels: -Area-Compat Area-WebKit Mstone-2.1 Plugins |
|||||||||||||||||||||||
,
Mar 11, 2009
(No comment was entered for this change.)
Status: Assigned
Owner: ana...@chromium.org |
|||||||||||||||||||||||
,
Mar 11, 2009
Adding John, since it involves scrolling with plugins.
Cc: j...@chromium.org
|
|||||||||||||||||||||||
,
Mar 11, 2009
This regressed between revisions 11145 and 11156. It is not clear from the changelogs as to which change caused this. ------------------------------------------------------------------------ r11145 | rvargas@google.com | 2009-03-06 12:05:34 -0800 (Fri, 06 Mar 2009) | 21 lines New disk cache eviction algorithm (partial implementation). Disabled by a #def. When enabled, file format 2.1 is used, with a possible update from ver 2.0. If a version with this code disabled is run after the upgrade, it will just discard the file and go back to 2.0. We now put some of those extra list to use! Entries are separated into various lists depending on how often are used, and we keep track of previously evicted entries. If the new algorithm is not enabled, most of the code just goes through a different path (with the old code instead of the new one). One notable exception is OpenFollowingEntry, used to enumerate the entries on the cache; the code changed significantly to support the new version of the "cache iterator", but functionally should do the same as the current code. Review URL: http://codereview.chromium.org/27345 ------------------------------------------------------------------------ r11146 | mark@chromium.org | 2009-03-06 12:13:23 -0800 (Fri, 06 Mar 2009) | 8 lines Adds a command line flag, --user-scripts-dir=..., to specify a directory to use in place of the default user scripts location. This eases development of userscripts since you can simply point the browser at your SCM checkout location instead of needing to try to sync things into your local userscripts directory. Code review URL: http://codereview.chromium.org/37011 Checking in for slightlyoff ------------------------------------------------------------------------ r11147 | ojan@google.com | 2009-03-06 12:15:58 -0800 (Fri, 06 Mar 2009) | 8 lines Fix _ValidateLists to work when running in linting mode. In linting mode we don't walk the tree to find the list of actual test files, so we don't know the full paths to files. We just use prefix matching to look for duplicate files. So when validating the two test lists in linting mode, we should also use prefix matching. Review URL: http://codereview.chromium.org/40151 ------------------------------------------------------------------------ r11148 | ralphl@chromium.org | 2009-03-06 12:27:51 -0800 (Fri, 06 Mar 2009) | 6 lines Pipeline_Impl was modified to properly render a stream that has video but no audio. A unit test accompanies the change. Note that one minor other change was snuck in with this change. The data_source_impl.cc had a TODO to set a more specific error when a read failed. Because I was already updating the pipeline error enum, I added the error code, changed the call to host_->Error(), and removed the TODO. Review URL: http://codereview.chromium.org/39170 ------------------------------------------------------------------------ r11149 | willchan@chromium.org | 2009-03-06 12:35:45 -0800 (Fri, 06 Mar 2009) | 2 lines In preparation for making changes to worker_pool_linux.cc. Review URL: http://codereview.chromium.org/39081 ------------------------------------------------------------------------ r11150 | sgk@google.com | 2009-03-06 12:38:16 -0800 (Fri, 06 Mar 2009) | 7 lines Add a product_name of 'chrome' on the non-Mac systems. This adds a placeholder condition for building branded 'chrome.exe' vs. 'chromium.exe', which can be updated to build 'chromium.exe' by default whenever we update the infrastructure (buildbots, developer instructions, etc.) to use -Dbranding=Chrome. Review URL: http://codereview.chromium.org/40233 ------------------------------------------------------------------------ r11151 | pkasting@chromium.org | 2009-03-06 12:43:24 -0800 (Fri, 06 Mar 2009) | 6 lines Convert from line scrolling to pixel scrolling. Increase pixel scroll amount on Windows. See also https://bugs.webkit.org/show_bug.cgi?id=24407 . BUG=6172 Review URL: http://codereview.chromium.org/40241 ------------------------------------------------------------------------ r11152 | erg@google.com | 2009-03-06 12:50:52 -0800 (Fri, 06 Mar 2009) | 8 lines RenderWidgetHost now only accepts a new NativeWebKeyboardEvent which owns a copy of the native os event. Only the WebKeyboardEvent is sent over the IPC barrier, but the full structure passed to the unhandled key event handler. BUG=4772 Review URL: http://codereview.chromium.org/40065 ------------------------------------------------------------------------ r11153 | pkasting@chromium.org | 2009-03-06 12:51:09 -0800 (Fri, 06 Mar 2009) | 4 lines Rebaseline form-element-geometry-expected. There are basically two sets of changes from the last baseline, which was done by Finnur on Jan. 26. One is a fix to SELECT scrollbars that looks like it's strictly more correct. The other is the removal of a [border bgcolor="#000000"] on the BRs inside some SELECTs, which I'm less certain of, but which isn't present upstream and wasn't present in our baseline before the last set of changes to it. Review URL: http://codereview.chromium.org/39268 ------------------------------------------------------------------------ r11155 | darin@chromium.org | 2009-03-06 12:55:25 -0800 (Fri, 06 Mar 2009) | 1 line roll DEPS to pickup WebKit@11154 ------------------------------------------------------------------------ r11156 | erikkay@google.com | 2009-03-06 12:58:47 -0800 (Fri, 06 Mar 2009) | 3 lines Change JSONFileValueSerializer to use FilePath by default. Review URL: http://codereview.chromium.org/39271 ------------------------------------------------------------------------ |
|||||||||||||||||||||||
,
Mar 11, 2009
This should be MStone 2.0
Labels: -Mstone-2.1 Mstone-2.0
|
|||||||||||||||||||||||
,
Mar 11, 2009
Adding Peter as he is working on a related fix to the webkit mousewheel handling code. This bug is a regression caused by the following change r11151 | pkasting@chromium.org | 2009-03-06 12:43:24 -0800 (Fri, 06 Mar 2009) | 6 lines Convert from line scrolling to pixel scrolling. Increase pixel scroll amount on Windows. See also https://bugs.webkit.org/show_bug.cgi?id=24407 . BUG=6172 Review URL: http://codereview.chromium.org/40241 Peter mentioned that this bug should be fixed by the fixes for the upstream bug https://bugs.webkit.org/show_bug.cgi?id=24368
Cc: pkast...@chromium.org
|
|||||||||||||||||||||||
,
Mar 17, 2009
Fixed in revision 11801
Status: Fixed
|
|||||||||||||||||||||||
,
Apr 16, 2009
Issue 8810 has been merged into this issue. |
|||||||||||||||||||||||
,
May 29, 2009
I know this is fixed long time back :).
Status: Verified
|
|||||||||||||||||||||||
| ► Sign in to add a comment | |||||||||||||||||||||||