Adding test to verify that Canvas2DLayerBridge flushes only when required

third_party/WebKit/Source/platform/graphics/Canvas2DLayerBridgeTest.cpp

 /*
  * Copyright (C) 2011 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1.  Redistributions of source code must retain the above copyright
  *     notice, this list of conditions and the following disclaimer.
  * 2.  Redistributions in binary form must reproduce the above copyright
  *     notice, this list of conditions and the following disclaimer in the
@@ skipping 1293 matching lines @@
     bridge->PrepareTextureMailbox(&textureMailbox, &releaseCallback);
   }
 
   bool lostResource = false;
   releaseCallback->Run(gpu::SyncToken(), lostResource);
 
   EXPECT_EQ(1u, gl.createImageCount());
   EXPECT_EQ(1u, gl.destroyImageCount());
 }
 
+class FlushMockGLES2Interface : public gpu::gles2::GLES2InterfaceStub {
+ public:
+  MOCK_METHOD0(Flush, void());
+};
+
+TEST_F(Canvas2DLayerBridgeTest, NoUnnecessaryFlushes) {
+  FlushMockGLES2Interface gl;
+  std::unique_ptr<FakeWebGraphicsContext3DProvider> contextProvider =
+      WTF::wrapUnique(new FakeWebGraphicsContext3DProvider(&gl));
+
+  EXPECT_CALL(gl, Flush()).Times(0);
+  Canvas2DLayerBridgePtr bridge(adoptRef(new Canvas2DLayerBridge(
+      std::move(contextProvider), IntSize(300, 150), 0, NonOpaque,
+      Canvas2DLayerBridge::ForceAccelerationForTesting, nullptr,
+      kN32_SkColorType)));
+  EXPECT_FALSE(bridge->hasRecordedDrawCommands());
+  ::testing::Mock::VerifyAndClearExpectations(&gl);
+
+  EXPECT_CALL(gl, Flush()).Times(0);
+  bridge->didDraw(FloatRect(0, 0, 1, 1));
+  EXPECT_TRUE(bridge->hasRecordedDrawCommands());
+  ::testing::Mock::VerifyAndClearExpectations(&gl);
+
+  EXPECT_CALL(gl, Flush()).Times(1);
+  bridge->flushGpu();
+  EXPECT_FALSE(bridge->hasRecordedDrawCommands());
+  ::testing::Mock::VerifyAndClearExpectations(&gl);
+
+  EXPECT_CALL(gl, Flush()).Times(0);
+  bridge->didDraw(FloatRect(0, 0, 1, 1));
+  EXPECT_TRUE(bridge->hasRecordedDrawCommands());
+  ::testing::Mock::VerifyAndClearExpectations(&gl);
+
+  EXPECT_CALL(gl, Flush()).Times(1);
+  bridge->flushGpu();
+  EXPECT_FALSE(bridge->hasRecordedDrawCommands());
+  ::testing::Mock::VerifyAndClearExpectations(&gl);
+
+  // No flush because already flushed since last draw
+  EXPECT_CALL(gl, Flush()).Times(0);
+  bridge->flushGpu();
+  EXPECT_FALSE(bridge->hasRecordedDrawCommands());
+  ::testing::Mock::VerifyAndClearExpectations(&gl);
+
+  EXPECT_CALL(gl, Flush()).Times(0);
+  bridge->didDraw(FloatRect(0, 0, 1, 1));
+  EXPECT_TRUE(bridge->hasRecordedDrawCommands());
+  ::testing::Mock::VerifyAndClearExpectations(&gl);
+
+  // Flushes recording, but not the gpu
+  EXPECT_CALL(gl, Flush()).Times(0);
+  bridge->flush();
+  EXPECT_FALSE(bridge->hasRecordedDrawCommands());
+  ::testing::Mock::VerifyAndClearExpectations(&gl);
+
+  EXPECT_CALL(gl, Flush()).Times(1);
+  bridge->flushGpu();
+  EXPECT_FALSE(bridge->hasRecordedDrawCommands());
+  ::testing::Mock::VerifyAndClearExpectations(&gl);
+}
+
 }  // namespace blink