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

Unified Diff: blimp/engine/feature/engine_render_widget_feature_unittest.cc

Issue 1810213009: blimp: Add settings to allow the client to download the whole page. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@record_full_document
Patch Set: Minor comment fixes. Created 4 years, 9 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/engine/feature/engine_render_widget_feature_unittest.cc
diff --git a/blimp/engine/feature/engine_render_widget_feature_unittest.cc b/blimp/engine/feature/engine_render_widget_feature_unittest.cc
index ad23d639374dd86f575fff0e26b1c2ae6dd5227f..5668ca2318264090deff7f3a244286733a837da4 100644
--- a/blimp/engine/feature/engine_render_widget_feature_unittest.cc
+++ b/blimp/engine/feature/engine_render_widget_feature_unittest.cc
@@ -24,6 +24,7 @@ using testing::InSequence;
using testing::Sequence;
namespace blimp {
+namespace engine {
namespace {
@@ -158,7 +159,7 @@ void SendCompositorMessage(BlimpMessageProcessor* processor,
class EngineRenderWidgetFeatureTest : public testing::Test {
public:
- EngineRenderWidgetFeatureTest() {}
+ EngineRenderWidgetFeatureTest() : feature_(&settings_) {}
void SetUp() override {
render_widget_message_sender_ = new MockBlimpMessageProcessor;
@@ -178,6 +179,7 @@ class EngineRenderWidgetFeatureTest : public testing::Test {
MockRenderWidgetHost render_widget_host2_;
MockHostRenderWidgetMessageDelegate delegate1_;
MockHostRenderWidgetMessageDelegate delegate2_;
+ SettingsManager settings_;
EngineRenderWidgetFeature feature_;
};
@@ -283,5 +285,5 @@ TEST_F(EngineRenderWidgetFeatureTest, RepliesHaveCorrectRenderWidgetId) {
feature_.SendCompositorMessage(1, &render_widget_host1_, payload);
}
-
+} // namespace engine
} // namespace blimp

Powered by Google App Engine
This is Rietveld 408576698