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

Unified Diff: blimp/client/core/render_widget/blimp_render_widget_unittest.cc

Issue 2241623002: blimp: Move compositing, input and render widget feature to client/core. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: addressed comments from #7 Created 4 years, 4 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: blimp/client/core/render_widget/blimp_render_widget_unittest.cc
diff --git a/blimp/client/core/render_widget/blimp_render_widget_unittest.cc b/blimp/client/core/render_widget/blimp_render_widget_unittest.cc
new file mode 100644
index 0000000000000000000000000000000000000000..84483b76217746252a8e6b443e3bc18684b12281
--- /dev/null
+++ b/blimp/client/core/render_widget/blimp_render_widget_unittest.cc
@@ -0,0 +1,82 @@
+// Copyright 2016 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.
+
+#include "blimp/client/core/render_widget/blimp_render_widget.h"
+
+#include "blimp/client/core/compositor/compositor_deps_provider.h"
+#include "blimp/client/core/render_widget/blimp_render_widget_delegate.h"
+#include "cc/proto/compositor_message.pb.h"
+#include "testing/gmock/include/gmock/gmock.h"
+#include "testing/gtest/include/gtest/gtest.h"
+
+using testing::_;
+
+namespace blimp {
+namespace client {
+
+class MockRenderWidgetDelegate : public BlimpRenderWidgetDelegate {
+ void SendWebGestureEvent(
+ BlimpRenderWidget* render_widget,
+ const blink::WebGestureEvent& gesture_event) override {}
+ void SendCompositorMessage(
+ BlimpRenderWidget* render_widget,
+ const cc::proto::CompositorMessage& message) override {}
+ void CompositorDidCompleteSwapBuffers() override {}
+};
+
+class BlimpRenderWidgetTest : public testing::Test {
+ public:
+ BlimpRenderWidgetTest() : render_widget_id_(1), window_(42u) {}
+
+ void SetUpRenderWidget(bool use_internal_display) {
+ CompositorDepsProvider::Initialize(use_internal_display);
+ image_serialization_processor_ =
+ base::MakeUnique<BlobImageSerializationProcessor>();
+ render_widget_ =
+ base::MakeUnique<BlimpRenderWidget>(render_widget_id_, &delegate_);
+ }
+
+ void TearDown() override {
+ render_widget_.reset();
+ CompositorDepsProvider::Terminate();
+ image_serialization_processor_.reset();
+ }
+
+ ~BlimpRenderWidgetTest() override {}
+
+ base::MessageLoop loop_;
+ int render_widget_id_;
+ MockRenderWidgetDelegate delegate_;
+ std::unique_ptr<BlobImageSerializationProcessor>
+ image_serialization_processor_;
+ std::unique_ptr<BlimpRenderWidget> render_widget_;
+ gfx::AcceleratedWidget window_;
+};
+
+TEST_F(BlimpRenderWidgetTest, ToggleVisibilityAndWidgetWithHost) {
+ // This should only be possible if we are running in direct rendering mode.
+ SetUpRenderWidget(true);
+
+ // Make the render widget visible and give it a widget.
+ render_widget_->SetVisible(true);
+ render_widget_->SetAcceleratedWidget(window_);
+
+ // Check that the visibility is set correctly on the compositor.
+ EXPECT_TRUE(render_widget_->GetCompositorForTesting()->IsVisible());
+
+ // Make the render widget invisible. This should drop the output surface and
+ // make the |compositor_| invisible.
+ render_widget_->SetVisible(false);
+ EXPECT_FALSE(render_widget_->GetCompositorForTesting()->IsVisible());
+
+ // Make the render widget visible and release the widget. This should make the
+ // |compositor_| invisible.
+ render_widget_->SetVisible(true);
+ EXPECT_TRUE(render_widget_->GetCompositorForTesting()->IsVisible());
+ render_widget_->SetAcceleratedWidget(gfx::kNullAcceleratedWidget);
+ EXPECT_FALSE(render_widget_->GetCompositorForTesting()->IsVisible());
+}
+
+} // namespace client
+} // namespace blimp

Powered by Google App Engine
This is Rietveld 408576698