Let DelegateFrameHost in Mac and Android use correct FrameSinkId during creation

content/browser/compositor/gpu_process_transport_factory.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/gpu_process_transport_factory.h"
 
 #include <string>
 #include <utility>
 
 #include "base/bind.h"
@@ skipping 672 matching lines @@
 ui::ContextFactory* GpuProcessTransportFactory::GetContextFactory() {
   return this;
 }
 
 ui::ContextFactoryPrivate*
 GpuProcessTransportFactory::GetContextFactoryPrivate() {
   return this;
 }
 
 cc::FrameSinkId GpuProcessTransportFactory::AllocateFrameSinkId() {
+  // TODO(crbug.com/685777): Centralize allocation in one place.

[boliu, 2017/01/26 22:52:40]

I was expecting something like

Must be unique with RenderWidgetHostViewFoo FrameSinkId allocations.

And then in all the RWHV places, add that it must be unique with this, etc etc.

   return cc::FrameSinkId(0, next_sink_id_++);
 }
 
 void GpuProcessTransportFactory::SetDisplayVisible(ui::Compositor* compositor,
                                                    bool visible) {
   PerCompositorDataMap::iterator it = per_compositor_data_.find(compositor);
   if (it == per_compositor_data_.end())
     return;
   PerCompositorData* data = it->second.get();
   DCHECK(data);
@@ skipping 207 matching lines @@
       shared_vulkan_context_provider_ =
           cc::VulkanInProcessContextProvider::Create();
     }
 
     shared_vulkan_context_provider_initialized_ = true;
   }
   return shared_vulkan_context_provider_;
 }
 
 }  // namespace content