Use local ids for Surfaces APIs that can only apply to local surfaces

examples/surfaces_app/child_impl.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 "examples/surfaces_app/child_impl.h"
 
 #include "base/bind.h"
 #include "cc/output/compositor_frame.h"
 #include "cc/output/delegated_frame_data.h"
 #include "cc/quads/render_pass.h"
@@ skipping 10 matching lines @@
 namespace mojo {
 namespace examples {
 
 using cc::RenderPass;
 using cc::RenderPassId;
 using cc::DrawQuad;
 using cc::SolidColorDrawQuad;
 using cc::DelegatedFrameData;
 using cc::CompositorFrame;
 
-ChildImpl::ChildImpl(ApplicationConnection* surfaces_service_connection) {
+static const uint32_t kLocalId = 1u;
+
+ChildImpl::ChildImpl(ApplicationConnection* surfaces_service_connection)
+    : id_namespace_(0u) {
   surfaces_service_connection->ConnectToService(&surface_);
   surface_.set_client(this);
   surface_.WaitForIncomingMethodCall();  // Wait for ID namespace to arrive.
+  DCHECK_NE(0u, id_namespace_);
 }
 
 ChildImpl::~ChildImpl() {
-  if (surface_)
-    surface_->DestroySurface(mojo::SurfaceId::From(id_));
+  surface_->DestroySurface(kLocalId);
 }
 
 void ChildImpl::SetIdNamespace(uint32_t id_namespace) {
-  allocator_.reset(new cc::SurfaceIdAllocator(id_namespace));
+  id_namespace_ = id_namespace;
 }
 
 void ChildImpl::ReturnResources(
     Array<ReturnedResourcePtr> resources) {
   DCHECK(!resources.size());
 }
 
-void ChildImpl::ProduceFrame(
-    ColorPtr color,
-    SizePtr size_ptr,
-    const mojo::Callback<void(SurfaceIdPtr id)>& callback) {
-  id_ = allocator_->GenerateId();
-  surface_->CreateSurface(mojo::SurfaceId::From(id_));
+void ChildImpl::ProduceFrame(ColorPtr color,
+                             SizePtr size_ptr,
+                             const ProduceCallback& callback) {
   gfx::Size size = size_ptr.To<gfx::Size>();
   gfx::Rect rect(size);
   RenderPassId id(1, 1);
   scoped_ptr<RenderPass> pass = RenderPass::Create();
   pass->SetNew(id, rect, rect, gfx::Transform());
 
   CreateAndAppendSimpleSharedQuadState(pass.get(), gfx::Transform(), size);
 
   SolidColorDrawQuad* color_quad =
       pass->CreateAndAppendDrawQuad<SolidColorDrawQuad>();
   bool force_anti_aliasing_off = false;
-  color_quad->SetNew(pass->shared_quad_state_list.back(),
-                     rect,
-                     rect,
-                     color.To<SkColor>(),
-                     force_anti_aliasing_off);
+  color_quad->SetNew(pass->shared_quad_state_list.back(), rect, rect,
+                     color.To<SkColor>(), force_anti_aliasing_off);
 
   scoped_ptr<DelegatedFrameData> delegated_frame_data(new DelegatedFrameData);
   delegated_frame_data->render_pass_list.push_back(pass.Pass());
 
   scoped_ptr<CompositorFrame> frame(new CompositorFrame);
   frame->delegated_frame_data = delegated_frame_data.Pass();
 
-  surface_->SubmitFrame(mojo::SurfaceId::From(id_), mojo::Frame::From(*frame),
-                        mojo::Closure());
-  callback.Run(SurfaceId::From(id_));
+  surface_->CreateSurface(kLocalId);

[sky, 2015/01/13 18:14:27]

I'm not familiar with this class. Is ProduceFrame only run once?

[jamesr, 2015/01/14 01:50:29]

That's right - this app produces exactly one frame ever and is done.

+  surface_->SubmitFrame(kLocalId, mojo::Frame::From(*frame), mojo::Closure());
+  auto qualified_id = mojo::SurfaceId::New();
+  qualified_id->id_namespace = id_namespace_;
+  qualified_id->local = kLocalId;
+  callback.Run(qualified_id.Pass());
 }
 
 }  // namespace examples
 }  // namespace mojo