blimp: Add settings to allow the client to download the whole page.

blimp/client/feature/settings_feature.cc

+// 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/feature/settings_feature.h"
+
+#include "blimp/common/create_blimp_message.h"
+#include "blimp/common/proto/blimp_message.pb.h"
+#include "blimp/common/proto/settings.pb.h"
+#include "net/base/net_errors.h"
+
+namespace blimp {
+namespace client {
+
+SettingsFeature::SettingsFeature() : record_whole_document_(false) {}
+
+SettingsFeature::~SettingsFeature() {}
+
+void SettingsFeature::set_outgoing_message_processor(
+    scoped_ptr<BlimpMessageProcessor> processor) {
+  outgoing_message_processor_ = std::move(processor);
+}
+
+void SettingsFeature::SetRecordWholeDocument(bool record_whole_document) {
+  if (record_whole_document_ == record_whole_document)
+    return;
+
+  record_whole_document_ = record_whole_document;
+
+  EngineSettingsMessage* engine_settings;
+  scoped_ptr<BlimpMessage> message = CreateBlimpMessage(&engine_settings);
+  engine_settings->set_record_whole_document(record_whole_document_);
+  outgoing_message_processor_->ProcessMessage(std::move(message),
+                                              net::CompletionCallback());
+}
+
+void SettingsFeature::ProcessMessage(scoped_ptr<BlimpMessage> message,
+                                     const net::CompletionCallback& callback) {
+  // We don't receive any messages from the engine yet.
+  NOTREACHED() << "Invalid settings message received from the engine.";
+  callback.Run(net::OK);

[haibinlu, 2016/03/22 17:28:39]

remove callback.Run(net::OK);

[Khushal, 2016/03/25 23:00:22]

Are we sure we want to do this? In case of an error, the NOTREACHED() would just
log in release builds. If we don't run the callback, it would stall the complete
message pipeline.

+}
+
+}  // namespace client
+}  // namespace blimp