Display Compositor: Allocate LocalFrameId in client.

content/browser/renderer_host/offscreen_canvas_compositor_frame_sink.cc

 // Copyright 2016 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/renderer_host/offscreen_canvas_compositor_frame_sink.h"
 
 #include "cc/surfaces/surface.h"
 #include "cc/surfaces/surface_manager.h"
 #include "content/browser/compositor/surface_utils.h"
 #include "mojo/public/cpp/bindings/strong_binding.h"
@@ skipping 27 matching lines @@
 void OffscreenCanvasCompositorFrameSink::Create(
     const cc::SurfaceId& surface_id,
     cc::mojom::MojoCompositorFrameSinkClientPtr client,
     cc::mojom::MojoCompositorFrameSinkRequest request) {
   mojo::MakeStrongBinding(base::MakeUnique<OffscreenCanvasCompositorFrameSink>(
                               surface_id, std::move(client)),
                           std::move(request));
 }
 
 void OffscreenCanvasCompositorFrameSink::SubmitCompositorFrame(
+    const cc::LocalFrameId& local_frame_Id,

[sadrul, 2016/11/22 19:53:29]

*id

[Fady Samuel, 2016/11/22 20:17:26]

Done.

     cc::CompositorFrame frame) {
   ++ack_pending_count_;
   surface_factory_->SubmitCompositorFrame(
       surface_id_.local_frame_id(), std::move(frame),

[sadrul, 2016/11/22 19:53:29]

Do you want to use |local_frame_id| here instead?

[Fady Samuel, 2016/11/22 20:17:26]

Not yet. I don't want to change offscreen canvas yet. The offscreen canvas team
is currently changing that code.

       base::Bind(
           &OffscreenCanvasCompositorFrameSink::DidReceiveCompositorFrameAck,
           base::Unretained(this)));
 }
 
 void OffscreenCanvasCompositorFrameSink::SetNeedsBeginFrame(
     bool needs_begin_frame) {
   NOTIMPLEMENTED();
 }
 
@@ skipping 24 matching lines @@
   client_->DidReceiveCompositorFrameAck();
   DCHECK_GT(ack_pending_count_, 0);
   if (!surface_returned_resources_.empty()) {
     client_->ReclaimResources(surface_returned_resources_);
     surface_returned_resources_.clear();
   }
   ack_pending_count_--;
 }
 
 }  // namespace content