Comment #3 on issue 8835 by [email protected]: CPU usage spikes up
when two youtube videos are in sight
http://code.google.com/p/chromium/issues/detail?id=8835
The following revision refers to this bug:
http://src.chromium.org/viewvc/chrome?view=rev&revision=12651
------------------------------------------------------------------------
r12651 | [email protected] | 2009-03-27 09:21:25 -0700 (Fri, 27 Mar 2009)
| 12 lines
Changed paths:
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/plugin/webplugin_proxy.cc?r1=12651&r2=12650
M
http://src.chromium.org/viewvc/chrome/trunk/src/webkit/glue/plugins/test/npapi_test_plugin.vcproj?r1=12651&r2=12650
M
http://src.chromium.org/viewvc/chrome/trunk/src/webkit/glue/plugins/test/plugin_client.cc?r1=12651&r2=12650
D
/trunk/src/webkit/glue/plugins/test/plugin_execute_script_delete_test.cc
D /trunk/src/webkit/glue/plugins/test/plugin_execute_script_delete_test.h
A
http://src.chromium.org/viewvc/chrome/trunk/src/webkit/glue/plugins/test/plugin_windowless_test.cc
A
http://src.chromium.org/viewvc/chrome/trunk/src/webkit/glue/plugins/test/plugin_windowless_test.h
We need to set the clipping region of the HDC passed in to a windowless
flash plugin instance in NPP_HandleEvent for WM_PAINT. The windowless flash
plugin instance queries the clipping region of the DC and on not finding it
proceeds
to paint the plugin window rect, which causes unnecessary CPU spikes.
This fixes bug http://code.google.com/p/chromium/issues/detail?id=8835,
where the plugin process would consume CPU even when we scrolled down to
the static video images.
With this fix the CPU usage for windowless flash plugins is on par with
Firefox.
Added a uitest which validates whether the hdc passed in to HandleEvent for
WM_PAINT has a valid clipping region.
Bug=8835
Review URL: http://codereview.chromium.org/53106
------------------------------------------------------------------------
--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings
--~--~---------~--~----~------------~-------~--~----~
Automated mail from issue updates at http://crbug.com/
Subscription options: http://groups.google.com/group/chromium-bugs
-~----------~----~----~----~------~----~------~--~---
