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

Unified Diff: third_party/WebKit/Source/platform/graphics/CanvasSurfaceLayerBridgeTest.cpp

Issue 2479563005: Create manager to track OffscreenCanvasSurfaceImpl instances (Closed)
Patch Set: Created 4 years, 1 month 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/CanvasSurfaceLayerBridgeTest.cpp
diff --git a/third_party/WebKit/Source/platform/graphics/CanvasSurfaceLayerBridgeTest.cpp b/third_party/WebKit/Source/platform/graphics/CanvasSurfaceLayerBridgeTest.cpp
index 582e1e856515b9d66213a1b7f504980e2e8af291..d37db56fa82fb900b0f5ff0fadc8aacf7b6d618d 100644
--- a/third_party/WebKit/Source/platform/graphics/CanvasSurfaceLayerBridgeTest.cpp
+++ b/third_party/WebKit/Source/platform/graphics/CanvasSurfaceLayerBridgeTest.cpp
@@ -26,7 +26,7 @@ class MockOffscreenCanvasSurface final
mojom::blink::OffscreenCanvasSurfacePtr GetProxy();
- void GetSurfaceId(const GetSurfaceIdCallback&) override;
+ void GetSurfaceId(uint32_t canvasId, const GetSurfaceIdCallback&) override;
void Require(const cc::SurfaceId&, const cc::SurfaceSequence&) override {}
void Satisfy(const cc::SurfaceSequence&) override {}
@@ -62,6 +62,7 @@ mojom::blink::OffscreenCanvasSurfacePtr MockOffscreenCanvasSurface::GetProxy() {
}
void MockOffscreenCanvasSurface::GetSurfaceId(
+ uint32_t canvasId,
const GetSurfaceIdCallback& callback) {
callback.Run(cc::SurfaceId(cc::FrameSinkId(10, 11), cc::LocalFrameId(15, 0)));
}
@@ -72,7 +73,7 @@ void CanvasSurfaceLayerBridgeTest::SetUp() {
}
TEST_F(CanvasSurfaceLayerBridgeTest, SurfaceLayerCreation) {
- bool success = this->surfaceLayerBridge()->createSurfaceLayer(50, 50);
+ bool success = this->surfaceLayerBridge()->createSurfaceLayer(0, 50, 50);
EXPECT_TRUE(success);
}

Powered by Google App Engine
This is Rietveld 408576698