Support distributing vsync info with surfaces.

content/browser/compositor/onscreen_display_client.cc

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/browser/compositor/onscreen_display_client.h"
 
 #include "base/debug/trace_event.h"
 #include "cc/output/output_surface.h"
 #include "cc/surfaces/surface_factory.h"
 #include "cc/surfaces/surface_manager.h"
+#include "content/browser/compositor/surface_display_output_surface.h"
 #include "content/common/host_shared_bitmap_manager.h"
 
 namespace content {
 
 OnscreenDisplayClient::OnscreenDisplayClient(
     scoped_ptr<cc::OutputSurface> output_surface,
     cc::SurfaceManager* manager,
     scoped_refptr<base::SingleThreadTaskRunner> task_runner)
     : output_surface_(output_surface.Pass()),
       display_(
           new cc::Display(this, manager, HostSharedBitmapManager::current())),
       task_runner_(task_runner),
       scheduled_draw_(false),
       weak_ptr_factory_(this) {
 }
 
 OnscreenDisplayClient::~OnscreenDisplayClient() {
 }
 
 scoped_ptr<cc::OutputSurface> OnscreenDisplayClient::CreateOutputSurface() {
   DCHECK(output_surface_.get());
   return output_surface_.Pass();
 }
 
+void OnscreenDisplayClient::CommitVSyncParameters(base::TimeTicks timebase,
+                                                  base::TimeDelta interval) {
+  surface_display_output_surface_->ReceivedVSyncParameters(timebase, interval);

[jamesr, 2014/09/17 05:47:11]

we have an output_surface_ member already, and we know it's always of actual
type SurfaceDisplayOutputSurface, so why not downcast?

or are we just trying to call output_surface_->CommitVSyncParameter()s? if so,
why not do that? because it's protected? making it public seems cleaner than
routing in this roundabout way

+}
+
 void OnscreenDisplayClient::DisplayDamaged() {
   if (scheduled_draw_)
     return;
   TRACE_EVENT0("content", "OnscreenDisplayClient::DisplayDamaged");
   scheduled_draw_ = true;
   task_runner_->PostTask(
       FROM_HERE,
       base::Bind(&OnscreenDisplayClient::Draw, weak_ptr_factory_.GetWeakPtr()));
 }
 
 void OnscreenDisplayClient::Draw() {
   TRACE_EVENT0("content", "OnscreenDisplayClient::Draw");
   scheduled_draw_ = false;
   display_->Draw();
 }
 
 }  // namespace content