Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(46)

Unified Diff: third_party/WebKit/Source/platform/graphics/CanvasSurfaceLayerBridge.h

Issue 2956713003: Replaces VideoLayer with a SurfaceLayer in WebMediaPlayerImpl (Closed)
Patch Set: Moves some implementation into SurfaceLayerBridge. Created 3 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/platform/graphics/CanvasSurfaceLayerBridge.h
diff --git a/third_party/WebKit/Source/platform/graphics/CanvasSurfaceLayerBridge.h b/third_party/WebKit/Source/platform/graphics/CanvasSurfaceLayerBridge.h
index c82436cc6d369947339ce6f063595d1ff85a9b98..822eaf1d9e000dab7f82471f1d92a3a90279d35b 100644
--- a/third_party/WebKit/Source/platform/graphics/CanvasSurfaceLayerBridge.h
+++ b/third_party/WebKit/Source/platform/graphics/CanvasSurfaceLayerBridge.h
@@ -12,6 +12,7 @@
#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/modules/offscreencanvas/offscreen_canvas_surface.mojom-blink.h"
namespace cc {
@@ -33,21 +34,19 @@ class PLATFORM_EXPORT CanvasSurfaceLayerBridgeObserver {
};
class PLATFORM_EXPORT CanvasSurfaceLayerBridge
- : NON_EXPORTED_BASE(
- public blink::mojom::blink::OffscreenCanvasSurfaceClient) {
+ : public NON_EXPORTED_BASE(SurfaceLayerBridge) {
public:
explicit CanvasSurfaceLayerBridge(CanvasSurfaceLayerBridgeObserver*,
WebLayerTreeView*);
~CanvasSurfaceLayerBridge();
+
apacible 2017/07/07 18:25:38 nit: remove newline.
CJ 2017/07/07 18:50:47 I think you are reviewing an older patch set. Thre
void CreateSolidColorLayer();
- WebLayer* GetWebLayer() const { return web_layer_.get(); }
- const cc::FrameSinkId& GetFrameSinkId() const { return frame_sink_id_; }
// 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&);
+ WebLayer* GetWebLayer() const { return web_layer_.get(); }
+ const cc::FrameSinkId& GetFrameSinkId() const { return frame_sink_id_; }
private:
scoped_refptr<cc::Layer> cc_layer_;
@@ -58,7 +57,6 @@ class PLATFORM_EXPORT CanvasSurfaceLayerBridge
CanvasSurfaceLayerBridgeObserver* observer_;
- mojom::blink::OffscreenCanvasSurfacePtr service_;
mojo::Binding<blink::mojom::blink::OffscreenCanvasSurfaceClient> binding_;
const cc::FrameSinkId frame_sink_id_;

Powered by Google App Engine
This is Rietveld 408576698