Hide compositor_frame_sink_id from RenderWidgetHostView*

content/browser/renderer_host/render_widget_host_impl.h

 // Copyright (c) 2012 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 CONTENT_BROWSER_RENDERER_HOST_RENDER_WIDGET_HOST_IMPL_H_
 #define CONTENT_BROWSER_RENDERER_HOST_RENDER_WIDGET_HOST_IMPL_H_
 
 #include <stddef.h>
 #include <stdint.h>
 
@@ skipping 437 matching lines @@
 
   // Requests the renderer to move the caret selection towards the point.
   void MoveCaret(const gfx::Point& point);
 
   // Called when the reponse to a pending mouse lock request has arrived.
   // Returns true if |allowed| is true and the mouse has been successfully
   // locked.
   bool GotResponseToLockMouseRequest(bool allowed);
 
   // Called by the view in response to OnSwapCompositorFrame.
-  static void SendReclaimCompositorResources(
-      int32_t route_id,
-      uint32_t compositor_frame_sink_id,
-      int renderer_host_id,
+  void SendReclaimCompositorResources(
       bool is_swap_ack,
       const cc::ReturnedResourceArray& resources);
 
   void set_allow_privileged_mouse_lock(bool allow) {
     allow_privileged_mouse_lock_ = allow;
   }
 
   // Resets state variables related to tracking pending size and painting.
   //
   // We need to reset these flags when we want to repaint the contents of
@@ skipping 98 matching lines @@
   // true, RenderWidget will respond immediately. If |monitor_updates| is true,
   // then RenderWidget sends updates for each compositor frame when there are
   // changes, or when the text selection changes inside a frame. If both fields
   // are false, RenderWidget will not send any updates. To avoid sending
   // unnecessary IPCs to RenderWidget (e.g., asking for monitor updates while
   // we are already receiving updates), when
   // |monitoring_composition_info_| == |monitor_updates| no IPC is sent to the
   // renderer unless it is for an immediate request.
   void RequestCompositionUpdates(bool immediate_request, bool monitor_updates);
 
+  // Submits the frame received from RenderWidget.
+  void SubmitCompositorFrame(const cc::LocalSurfaceId& local_surface_id,
+                             cc::CompositorFrame frame);
+
  protected:
   // ---------------------------------------------------------------------------
   // The following method is overridden by RenderViewHost to send upwards to
   // its delegate.
 
   // Callback for notification that we failed to receive any rendered graphics
   // from a newly loaded page. Used for testing.
   virtual void NotifyNewContentRenderingTimeoutForTesting() {}
 
   // ---------------------------------------------------------------------------
@@ skipping 332 matching lines @@
   // compared against current_content_source_id_ to determine whether the
   // received frame belongs to the current page. If a frame for the current page
   // does not arrive in time after nagivation, we clear the graphics of the old
   // page. See RenderWidget::current_content_source_id_ for more information.
   uint32_t last_received_content_source_id_ = 0;
 
 #if defined(OS_MACOSX)
   std::unique_ptr<device::PowerSaveBlocker> power_save_blocker_;
 #endif
 
+  // These information are used to verify that the renderer does not misbehave
+  // when it comes to allocating LocalSurfaceIds. If frame size or device scale
+  // factor change, a new LocalSurfaceId must be created.
   cc::LocalSurfaceId last_local_surface_id_;
   gfx::Size last_frame_size_;
   float last_device_scale_factor_;
 
+  // Each instance of RendererCompositorFrameSink has an ID that we keep track
+  // of so we can tell when a new instance has been created for the purpose of
+  // not returning stale resources.
+  uint32_t last_compositor_frame_sink_id_ = 0;
+
   base::WeakPtrFactory<RenderWidgetHostImpl> weak_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(RenderWidgetHostImpl);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_RENDERER_HOST_RENDER_WIDGET_HOST_IMPL_H_