Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(409)

Unified Diff: content/browser/renderer_host/compositing_iosurface_mac.mm

Issue 10824357: Revert 151982 - Revert 151932 (caused gpu_throughput_tests to start failing on Mac GPU bots) - Hook… (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Created 8 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: content/browser/renderer_host/compositing_iosurface_mac.mm
===================================================================
--- content/browser/renderer_host/compositing_iosurface_mac.mm (revision 152097)
+++ content/browser/renderer_host/compositing_iosurface_mac.mm (working copy)
@@ -10,7 +10,6 @@
#include "base/command_line.h"
#include "base/debug/trace_event.h"
#include "base/threading/platform_thread.h"
-#include "content/browser/renderer_host/render_widget_host_view_mac.h"
#include "content/public/browser/browser_thread.h"
#include "gpu/command_buffer/service/gpu_switches.h"
#include "ui/gfx/rect.h"
@@ -261,6 +260,7 @@
void CompositingIOSurfaceMac::GetVSyncParameters(base::TimeTicks* timebase,
uint32* interval_numerator,
uint32* interval_denominator) {
+ base::AutoLock lock(lock_);
*timebase = vsync_timebase_;
*interval_numerator = vsync_interval_numerator_;
*interval_denominator = vsync_interval_denominator_;
@@ -552,16 +552,18 @@
}
void CompositingIOSurfaceMac::DisplayLinkTick(CVDisplayLinkRef display_link,
- const CVTimeStamp* output_time) {
+ const CVTimeStamp* time) {
+ TRACE_EVENT0("gpu", "CompositingIOSurfaceMac::DisplayLinkTick");
base::AutoLock lock(lock_);
// Increment vsync_count but don't let it get ahead of swap_count.
vsync_count_ = std::min(vsync_count_ + 1, swap_count_);
- CalculateVsyncParametersLockHeld(output_time);
+ CalculateVsyncParametersLockHeld(time);
}
void CompositingIOSurfaceMac::CalculateVsyncParametersLockHeld(
const CVTimeStamp* time) {
+ lock_.AssertAcquired();
vsync_interval_numerator_ = static_cast<uint32>(time->videoRefreshPeriod);
vsync_interval_denominator_ = time->videoTimeScale;
// Verify that videoRefreshPeriod is 32 bits.

Powered by Google App Engine
This is Rietveld 408576698