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

Unified Diff: blimp/client/core/compositor/blimp_compositor_unittest.cc

Issue 2400923002: Move BlimpInputManager to BlimpDocument from BlimpCompositor. (Closed)
Patch Set: Removed unnecessary DCHECK. Created 4 years, 2 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
« no previous file with comments | « blimp/client/core/compositor/blimp_compositor.cc ('k') | blimp/client/core/render_widget/BUILD.gn » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: blimp/client/core/compositor/blimp_compositor_unittest.cc
diff --git a/blimp/client/core/compositor/blimp_compositor_unittest.cc b/blimp/client/core/compositor/blimp_compositor_unittest.cc
index ef4e2219789f81750eab39782796f1ada89bbb99..cbcc3aaca29ceb3a77c110aab5401f50d624eb72 100644
--- a/blimp/client/core/compositor/blimp_compositor_unittest.cc
+++ b/blimp/client/core/compositor/blimp_compositor_unittest.cc
@@ -25,16 +25,11 @@ class MockBlimpCompositorClient : public BlimpCompositorClient {
MockBlimpCompositorClient() = default;
~MockBlimpCompositorClient() override = default;
- void SendWebGestureEvent(
- const blink::WebGestureEvent& gesture_event) override {
- MockableSendWebGestureEvent();
- }
void SendCompositorMessage(
const cc::proto::CompositorMessage& message) override {
MockableSendCompositorMessage();
}
- MOCK_METHOD0(MockableSendWebGestureEvent, void());
MOCK_METHOD0(MockableSendCompositorMessage, void());
private:
@@ -52,10 +47,6 @@ class BlimpCompositorForTesting : public BlimpCompositor {
SendCompositorProto(proto);
}
- void SendGestureEvent(const blink::WebGestureEvent& gesture_event) {
- SendWebGestureEvent(gesture_event);
- }
-
cc::LayerTreeHost* host() const { return host_.get(); }
};
@@ -98,10 +89,8 @@ TEST_F(BlimpCompositorTest, ToggleVisibilityWithHost) {
TEST_F(BlimpCompositorTest, MessagesHaveCorrectId) {
EXPECT_CALL(compositor_client_, MockableSendCompositorMessage()).Times(1);
- EXPECT_CALL(compositor_client_, MockableSendWebGestureEvent()).Times(1);
compositor_->SendProto(cc::proto::CompositorMessage());
- compositor_->SendGestureEvent(blink::WebGestureEvent());
}
} // namespace client
« no previous file with comments | « blimp/client/core/compositor/blimp_compositor.cc ('k') | blimp/client/core/render_widget/BUILD.gn » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698