| commit | d8dd133b137d8fb63fe261546f04b2a3affe9b7a | |
| author | ccameron <ccameron@chromium.org> | |
| Thu, 8 Jan 2015 02:13:14 +0000 (7 18:13 -0800) | ||
| committer | Commit bot <commit-bot@chromium.org> | |
| Thu, 8 Jan 2015 02:14:05 +0000 (8 02:14 +0000) | ||
| tree | d550f48c29ef38179f626cf06af42883298f6f36 | treesnapshot (tar.gz zip) |
| parent | d2ba9ec2f1bd12bf49fae11063ed2f79d547dc6d | commitdiff |
Clean up RenderWidgetHostImpl::WaitForSurface
Remove comments talking about backing stores, because they are no
longer a thing.
Don't break if WaitForSingleTaskToRun returns false, because that function
only promises to wait "at most" the timeout specified -- it could return
immediately, with no task having ran.
Change the return on success to a break, since the return will miss the
UMA statistics.
Use Now() for the UMA statistics, because the path where a surface is
found will not update the time to reflect the time it took to wait for the
message, and to put the frame on screen.
BUG=
Review URL: https://codereview.chromium.org/844493002
Cr-Commit-Position: refs/heads/master@{#310442}
Remove comments talking about backing stores, because they are no
longer a thing.
Don't break if WaitForSingleTaskToRun returns false, because that function
only promises to wait "at most" the timeout specified -- it could return
immediately, with no task having ran.
Change the return on success to a break, since the return will miss the
UMA statistics.
Use Now() for the UMA statistics, because the path where a surface is
found will not update the time to reflect the time it took to wait for the
message, and to put the frame on screen.
BUG=
Review URL: https://codereview.chromium.org/844493002
Cr-Commit-Position: refs/heads/master@{#310442}
| content/browser/renderer_host/render_widget_host_impl.cc | diffblobblamehistory |