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

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: Use ContentBrowserClient to override prefs. 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 3d9c91a6cd202f07fb40c53b43be435bb18f22db..93057753e5ec237e543a713f63c41cb6f6c538f4 100644
--- a/blimp/engine/feature/engine_render_widget_feature_unittest.cc
+++ b/blimp/engine/feature/engine_render_widget_feature_unittest.cc
@@ -26,6 +26,7 @@ using testing::InSequence;
using testing::Sequence;
namespace blimp {
+namespace engine {
namespace {
@@ -227,7 +228,7 @@ void SendCompositorMessage(BlimpMessageProcessor* processor,
class EngineRenderWidgetFeatureTest : public testing::Test {
public:
- EngineRenderWidgetFeatureTest() {}
+ EngineRenderWidgetFeatureTest() : feature_(&settings_) {}
void SetUp() override {
render_widget_message_sender_ = new MockBlimpMessageProcessor;
@@ -251,6 +252,7 @@ class EngineRenderWidgetFeatureTest : public testing::Test {
MockHostRenderWidgetMessageDelegate delegate1_;
MockHostRenderWidgetMessageDelegate delegate2_;
MockTextInputClient text_input_client_;
+ SettingsManager settings_;
EngineRenderWidgetFeature feature_;
};
@@ -372,5 +374,5 @@ TEST_F(EngineRenderWidgetFeatureTest, RepliesHaveCorrectRenderWidgetId) {
feature_.SendCompositorMessage(1, &render_widget_host1_, payload);
}
-
+} // namespace engine
} // namespace blimp
« no previous file with comments | « blimp/engine/feature/engine_render_widget_feature.cc ('k') | blimp/engine/feature/engine_settings_feature.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698