Replaces VideoLayer with a SurfaceLayer in WebMediaPlayerImpl

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

-// Copyright 2016 The Chromium Authors. All rights reserved.
+// Copyright 2017 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
+#ifndef VideoSurfaceLayerBridge_h
+#define VideoSurfaceLayerBridge_h
 
 #include <memory>
 #include "base/memory/ref_counted.h"
 #include "base/memory/weak_ptr.h"
 #include "cc/surfaces/surface_id.h"
 #include "cc/surfaces/surface_reference_factory.h"
 #include "mojo/public/cpp/bindings/binding.h"
 #include "platform/PlatformExport.h"
+#include "platform/graphics/SurfaceLayerBridge.h"
+#include "public/platform/WebVideoSurfaceLayerBridge.h"
 #include "public/platform/modules/offscreencanvas/offscreen_canvas_surface.mojom-blink.h"
 
 namespace cc {
 class Layer;
 class SurfaceInfo;
 }  // namespace cc
 
 namespace blink {
 
-class WebLayer;
-class WebLayerTreeView;
+class VideoSurfaceLayerBridge : public NON_EXPORTED_BASE(SurfaceLayerBridge),

[liberato (no reviews please), 2017/06/26 22:39:30]

many of these classes could use a few lines of class-level documentation.

[CJ, 2017/06/29 19:51:33]

Done. Please review my wording.

+                                public WebVideoSurfaceLayerBridge {
+ public:
+  explicit VideoSurfaceLayerBridge(WebSurfaceLayerBridgeObserver*);
+  ~VideoSurfaceLayerBridge();
 
-class PLATFORM_EXPORT CanvasSurfaceLayerBridgeObserver {
- public:
-  CanvasSurfaceLayerBridgeObserver() {}
-  virtual ~CanvasSurfaceLayerBridgeObserver() {}
+  // Implementation of WebVideoLayerSurfaceBridge
+  cc::Layer* GetLayer() { return cc_layer_.get(); }
 
-  virtual void OnWebLayerReplaced() = 0;
-};
-
-class PLATFORM_EXPORT CanvasSurfaceLayerBridge
-    : NON_EXPORTED_BASE(
-          public blink::mojom::blink::OffscreenCanvasSurfaceClient) {
- public:
-  explicit CanvasSurfaceLayerBridge(CanvasSurfaceLayerBridgeObserver*,
-                                    WebLayerTreeView*);
-  ~CanvasSurfaceLayerBridge();
+  // Implemenation of SurfaceLayerBridge
   void CreateSolidColorLayer();
-  WebLayer* GetWebLayer() const { return web_layer_.get(); }
-  const cc::FrameSinkId& GetFrameSinkId() const { return frame_sink_id_; }
+  void SatisfyCallback(const cc::SurfaceSequence&);
+  void RequireCallback(const cc::SurfaceId&, const cc::SurfaceSequence&);
 
   // Implementation of blink::mojom::blink::OffscreenCanvasSurfaceClient
   void OnSurfaceCreated(const cc::SurfaceInfo&) override;
 
-  void SatisfyCallback(const cc::SurfaceSequence&);
-  void RequireCallback(const cc::SurfaceId&, const cc::SurfaceSequence&);
+  const cc::FrameSinkId& GetFrameSinkId() const { return frame_sink_id_; }
 
  private:
   scoped_refptr<cc::Layer> cc_layer_;
-  std::unique_ptr<WebLayer> web_layer_;
 
   scoped_refptr<cc::SurfaceReferenceFactory> ref_factory_;
-  base::WeakPtrFactory<CanvasSurfaceLayerBridge> weak_factory_;
+  base::WeakPtrFactory<SurfaceLayerBridge> weak_factory_;
 
-  CanvasSurfaceLayerBridgeObserver* observer_;
+  WebSurfaceLayerBridgeObserver* observer_;
 
   mojom::blink::OffscreenCanvasSurfacePtr service_;
   mojo::Binding<blink::mojom::blink::OffscreenCanvasSurfaceClient> binding_;
 
   const cc::FrameSinkId frame_sink_id_;
   cc::SurfaceId current_surface_id_;
   const cc::FrameSinkId parent_frame_sink_id_;
 };
 
 }  // namespace blink
 
-#endif  // CanvasSurfaceLayerBridge_h
+#endif  // VideoSurfaceLayerBridge_h