cc: Add main frame timing info to frame timing tracker.

cc/trees/layer_tree_host_impl.h

 // Copyright 2011 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.
 
 #ifndef CC_TREES_LAYER_TREE_HOST_IMPL_H_
 #define CC_TREES_LAYER_TREE_HOST_IMPL_H_
 
 #include <set>
 #include <string>
 #include <vector>
@@ skipping 93 matching lines @@
   // Returns true if resources were deleted by this call.
   virtual bool ReduceContentsTextureMemoryOnImplThread(
       size_t limit_bytes,
       int priority_cutoff) = 0;
   virtual bool IsInsideDraw() = 0;
   virtual void RenewTreePriority() = 0;
   virtual void PostDelayedAnimationTaskOnImplThread(const base::Closure& task,
                                                     base::TimeDelta delay) = 0;
   virtual void DidActivateSyncTree() = 0;
   virtual void DidPrepareTiles() = 0;
+  virtual base::TimeTicks GetNextPredictedRequestAnimationFrameTime() const = 0;
 
   // Called when page scale animation has completed on the impl thread.
   virtual void DidCompletePageScaleAnimationOnImplThread() = 0;
 
  protected:
   virtual ~LayerTreeHostImplClient() {}
 };
 
 // LayerTreeHostImpl owns the LayerImpl trees as well as associated rendering
 // state.
@@ skipping 387 matching lines @@
       scoped_ptr<TileTaskWorkerPool>* tile_task_worker_pool,
       scoped_ptr<ResourcePool>* resource_pool,
       scoped_ptr<ResourcePool>* staging_resource_pool);
 
   bool prepare_tiles_needed() const { return tile_priorities_dirty_; }
 
   FrameTimingTracker* frame_timing_tracker() {
     return frame_timing_tracker_.get();
   }
 
+  void SetCurrentRequestAnimationFrameTime(base::TimeTicks frame_time);
+  void RecordMainFrameTiming();

[mithro-old, 2015/03/20 02:17:54]

This method should definitely be taking a BeginFrameArgs not a base::TimeTicks.

SetCurrentRequestAnimationFrameTime is a bit confusing because there is the
animation frame time used for the impl thread and the animation frame time used
for the main thread which are not equal. I can quickly see this value getting
used in the wrong places.

[vmpstr, 2015/03/20 18:47:26]

Ok, I will work on passing BeginFrameArgs here instead. My, again clearly
flawed, understanding was that the impl thread time and the raf animation time
was in fact the same time. Please let me know where they diverge.

+
  protected:
   LayerTreeHostImpl(
       const LayerTreeSettings& settings,
       LayerTreeHostImplClient* client,
       Proxy* proxy,
       RenderingStatsInstrumentation* rendering_stats_instrumentation,
       SharedBitmapManager* shared_bitmap_manager,
       gpu::GpuMemoryBufferManager* gpu_memory_buffer_manager,
       int id);
 
@@ skipping 197 matching lines @@
   gpu::GpuMemoryBufferManager* gpu_memory_buffer_manager_;
   int id_;
 
   std::set<SwapPromiseMonitor*> swap_promise_monitor_;
   std::vector<PictureLayerImpl::Pair> picture_layer_pairs_;
 
   bool requires_high_res_to_draw_;
   bool is_likely_to_require_a_draw_;
 
   scoped_ptr<FrameTimingTracker> frame_timing_tracker_;
+  base::TimeTicks request_animation_frame_time_;
 
   DISALLOW_COPY_AND_ASSIGN(LayerTreeHostImpl);
 };
 
 }  // namespace cc
 
 #endif  // CC_TREES_LAYER_TREE_HOST_IMPL_H_