Comment #6 on issue 4104 by [email protected]: Tab-switching tests
always output zero
http://code.google.com/p/chromium/issues/detail?id=4104
The following revision refers to this bug:
http://src.chromium.org/viewvc/chrome?view=rev&revision=31632
------------------------------------------------------------------------
r31632 | [email protected] | 2009-11-10 17:09:39 -0800 (Tue, 10 Nov 2009)
| 24 lines
Changed paths:
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/renderer_host/render_widget_host_view_gtk.cc?r1=31632&r2=31631
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/renderer_host/render_widget_host_view_gtk.h?r1=31632&r2=31631
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/renderer_host/render_widget_host_view_mac.h?r1=31632&r2=31631
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/renderer_host/render_widget_host_view_mac.mm?r1=31632&r2=31631
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/renderer_host/render_widget_host_view_win.cc?r1=31632&r2=31631
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/renderer_host/render_widget_host_view_win.h?r1=31632&r2=31631
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/test/tab_switching/tab_switching_test.cc?r1=31632&r2=31631
Add a RWH_TabSwitchPaintDuration histogram.
The RWH_TabSwitchPaintDuration histogram calculates
the amount of time it took after a particular view was
selected for it to be fully painted.
I modified the tab switching test to use the new tab
switch paint duration histogram in place of the whiteout
duration histogram. As a measure, whiteout duration
would not carry a value if we kept the backing store
around for a tab, and we would end up with a result of
zero instead of a useful result. Local results with the
new measure are consistent with the number of operations
we perform on the tabs.
The tab switching test now fails if it cannot locate
the correct histogram.
BUG=4104
TEST=count of tab switch paint duration histogram
results in tab switching test equals number of tabs
used in test
Review URL: http://codereview.chromium.org/378038
------------------------------------------------------------------------
--
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
