My favorites | Sign in
Logo
             
New issue | Search
for
| Advanced search | Search tips
Issue 8653: CPU pegged, if scrolled on flash
6 people starred this issue and may be notified of changes. Back to list
 
Reported by venkataramana@chromium.org, Mar 10, 2009
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. 

Comment 1 by laforge@chromium.org, 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
Comment 2 by jon@chromium.org, Mar 11, 2009
(No comment was entered for this change.)
Status: Assigned
Owner: ana...@chromium.org
Comment 3 by gwilson@chromium.org, Mar 11, 2009
Adding John, since it involves scrolling with plugins.
Cc: j...@chromium.org
Comment 4 by ananta@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
------------------------------------------------------------------------


Comment 5 by jon@chromium.org, Mar 11, 2009
This should be MStone 2.0
Labels: -Mstone-2.1 Mstone-2.0
Comment 6 by ananta@chromium.org, 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
Comment 7 by ananta@chromium.org, Mar 17, 2009
Fixed in revision 11801
Status: Fixed
Comment 8 by gwilson@chromium.org, Apr 16, 2009
 Issue 8810  has been merged into this issue.
Comment 9 by venkataramana@chromium.org, May 29, 2009
I know this is fixed long time back :).
Status: Verified
Sign in to add a comment

Powered by Google Project Hosting