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

Unified Diff: blimp/client/core/compositor/blimp_compositor_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: moar gn fix 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/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
new file mode 100644
index 0000000000000000000000000000000000000000..335739ccd70b0fe184984582348d6c51d687624b
--- /dev/null
+++ b/blimp/client/core/compositor/blimp_compositor_unittest.cc
@@ -0,0 +1,97 @@
+// 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.
+
+#ifndef BLIMP_CLIENT_CORE_COMPOSITOR_BLIMP_COMPOSITOR_UNITTEST_CC_
+#define BLIMP_CLIENT_CORE_COMPOSITOR_BLIMP_COMPOSITOR_UNITTEST_CC_
+
+#include "blimp/client/core/compositor/blimp_compositor.h"
+
+#include "blimp/client/core/compositor/compositor_deps_provider.h"
+#include "cc/proto/compositor_message.pb.h"
+#include "testing/gtest/include/gtest/gtest.h"
+
+namespace blimp {
+namespace client {
+
+class MockBlimpCompositorClient : public BlimpCompositorClient {
+ void DidCompleteSwapBuffers() override {}
+ void DidCommitAndDrawFrame() override {}
+ void SendWebGestureEvent(
+ const blink::WebGestureEvent& gesture_event) override {}
+ void SendCompositorMessage(
+ const cc::proto::CompositorMessage& message) override {}
+ void RequestOutputSurface() override {}
+};
+
+class BlimpCompositorTest : public testing::Test {
+ public:
+ BlimpCompositorTest() {}
+
+ void SetUp() override {
+ compositor_deps_provider_ = base::MakeUnique<CompositorDepsProvider>(true);
+ image_serialization_processor_ =
+ base::MakeUnique<BlobImageSerializationProcessor>();
+ compositor_ = base::MakeUnique<BlimpCompositor>(
+ compositor_deps_provider_->GetTaskGraphRunner(),
+ compositor_deps_provider_->GetGpuMemoryBufferManager(),
+ image_serialization_processor_.get(),
+ compositor_deps_provider_->GetLayerTreeSettings(),
+ compositor_deps_provider_->GetCompositorTaskRunner(), &client_);
+ }
+
+ void TearDown() override {
+ compositor_.reset();
+ compositor_deps_provider_.reset();
+ image_serialization_processor_.reset();
+ }
+
+ void SendInitializeMessage() {
+ std::unique_ptr<cc::proto::CompositorMessage> message;
+ message.reset(new cc::proto::CompositorMessage);
+ cc::proto::CompositorMessageToImpl* to_impl = message->mutable_to_impl();
+ to_impl->set_message_type(
+ cc::proto::CompositorMessageToImpl::INITIALIZE_IMPL);
+ cc::proto::InitializeImpl* initialize_message =
+ to_impl->mutable_initialize_impl_message();
+ cc::LayerTreeSettings settings;
+ settings.ToProtobuf(initialize_message->mutable_layer_tree_settings());
+ compositor_->OnCompositorMessageReceived(std::move(message));
+ }
+
+ void SendShutdownMessage() {
+ std::unique_ptr<cc::proto::CompositorMessage> message;
+ message.reset(new cc::proto::CompositorMessage);
+ cc::proto::CompositorMessageToImpl* to_impl = message->mutable_to_impl();
+ to_impl->set_message_type(cc::proto::CompositorMessageToImpl::CLOSE_IMPL);
+ compositor_->OnCompositorMessageReceived(std::move(message));
+ }
+
+ base::MessageLoop loop_;
+ std::unique_ptr<CompositorDepsProvider> compositor_deps_provider_;
+ std::unique_ptr<BlobImageSerializationProcessor>
+ image_serialization_processor_;
+ MockBlimpCompositorClient client_;
+ std::unique_ptr<BlimpCompositor> compositor_;
+};
+
+TEST_F(BlimpCompositorTest, CreatesHostCorrectly) {
+ // Create the host and make it visible.
+ SendInitializeMessage();
+ EXPECT_NE(compositor_->GetHostForTesting(), nullptr);
+
+ compositor_->SetVisible(true);
+ EXPECT_TRUE(compositor_->GetHostForTesting()->visible());
+
+ // Destroy this host and recreate a new one. Make sure that the visibility is
+ // set correctly on this host.
+ SendShutdownMessage();
+ SendInitializeMessage();
+ EXPECT_NE(compositor_->GetHostForTesting(), nullptr);
+ EXPECT_TRUE(compositor_->GetHostForTesting()->visible());
+}
+
+} // namespace client
+} // namespace blimp
+
+#endif // BLIMP_CLIENT_CORE_COMPOSITOR_BLIMP_COMPOSITOR_UNITTEST_CC_

Powered by Google App Engine
This is Rietveld 408576698