Establish mojo service between Canvas (blink) and SurfaceManager (browser)

third_party/WebKit/Source/platform/graphics/CanvasSurfaceLayerBridge.h

 // 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.
 
 #ifndef CanvasSurfaceLayerBridge_h
 #define CanvasSurfaceLayerBridge_h
 
 #include "base/memory/ref_counted.h"
+#include "cc/surfaces/surface_id.h"
 #include "platform/PlatformExport.h"
+#include "public/platform/modules/offscreencanvas/offscreen_canvas_surface_service.mojom-blink.h"
 #include "wtf/OwnPtr.h"
 
 namespace cc {
-// TODO(611796): replace SolidColorLayer with SurfaceLayer
-class SolidColorLayer;
-}
+class SurfaceLayer;
+struct SurfaceSequence;
+} // namespace cc
 
 namespace blink {
 
 class WebLayer;
 
 class PLATFORM_EXPORT CanvasSurfaceLayerBridge {
 public:
     explicit CanvasSurfaceLayerBridge();
-    ~CanvasSurfaceLayerBridge();
+    virtual ~CanvasSurfaceLayerBridge();
+    bool createSurfaceLayer(int canvasWidth, int canvasHeight);
     WebLayer* getWebLayer() const { return m_webLayer.get(); }
 
+    void satisfyCallback(cc::SurfaceSequence);
+    void requireCallback(cc::SurfaceId, cc::SurfaceSequence);
+
+protected:
+    // Virtual functions for unit tests

[danakj, 2016/06/13 20:53:39]

nit: end comments with punctuation

A common strategy is to pass in a virtual interface to this class that provides
functionality you want to mock/override in tests, rather than putting them on
the class itself. This prevents coupling the implementation of these functions
to the class. Up to you if that makes sense here.

[xlai (Olivia), 2016/06/14 21:10:18]

Done. Removed all the virtual testing things from Bridge and introduce
CanvasSurfaceLayerBridgeClient (a virtual interface) into it.

+    virtual bool syncGetServiceId();
+    virtual void asyncRequestSurfaceCreation();
+    cc::SurfaceId m_surfaceId;

[danakj, 2016/06/13 20:53:39]

This could be private if you made syncGetServiceId() return the SurfaceId (and
null for failure)?

[danakj, 2016/06/13 20:53:39]

use whitespace to make this separate from the comment above.

[xlai (Olivia), 2016/06/14 21:10:17]

Done.

+
 private:
-    scoped_refptr<cc::SolidColorLayer> m_solidColorLayer;
+    scoped_refptr<cc::SurfaceLayer> m_surfaceLayer;
     OwnPtr<WebLayer> m_webLayer;
+
+    mojom::blink::OffscreenCanvasSurfaceServicePtr m_service;
 };
 
-}
+} // namespace blink
 
 #endif // CanvasSurfaceLayerBridge

[danakj, 2016/06/13 20:53:39]

_h ?

[xlai (Olivia), 2016/06/14 21:10:17]

Done.