Set the user agent for 0.6

blimp/client/core/settings/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/core/settings/settings_feature.h"
 
 #include "blimp/client/core/settings/settings.h"
+#include "blimp/client/core/settings/user_agent.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(Settings* settings) : settings_(settings) {}

[Khushal, 2016/11/02 01:32:49]

DCHECK(settings_) here now? :)

[Menglin, 2016/11/02 16:34:57]

Done.

 
 SettingsFeature::~SettingsFeature() = default;
 
 void SettingsFeature::set_outgoing_message_processor(
     std::unique_ptr<BlimpMessageProcessor> processor) {
   outgoing_message_processor_ = std::move(processor);
 }
 
-// TODO(mlliu): remove this method once we set the user agent in PushSettings()
-// http://crbug.com/652032.
-void SettingsFeature::SendUserAgentOSVersionInfo(const std::string& osVersion) {
-  EngineSettingsMessage* engine_settings;
-  std::unique_ptr<BlimpMessage> message = CreateBlimpMessage(&engine_settings);
-  engine_settings->set_client_os_info(osVersion);
-  outgoing_message_processor_->ProcessMessage(std::move(message),
-                                              net::CompletionCallback());
-}
-
 void SettingsFeature::ProcessMessage(std::unique_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);
 }
 
 void SettingsFeature::OnRecordWholeDocumentChanged(bool enable) {
   EngineSettingsMessage* engine_settings;
   std::unique_ptr<BlimpMessage> message = CreateBlimpMessage(&engine_settings);
   engine_settings->set_record_whole_document(enable);
   outgoing_message_processor_->ProcessMessage(std::move(message),
                                               net::CompletionCallback());
 }
 
 void SettingsFeature::PushSettings() {
-  // TODO(mlliu): set the user agent on the proto as well after moving
-  // blimp/client/app/user_agent.* to this directory (http://crbug.com/652032).
   EngineSettingsMessage* engine_settings;
   std::unique_ptr<BlimpMessage> message = CreateBlimpMessage(&engine_settings);
   engine_settings->set_record_whole_document(
       settings_->IsRecordWholeDocument());
+  engine_settings->set_client_os_info(GetOSVersionInfoForUserAgent());
   outgoing_message_processor_->ProcessMessage(std::move(message),
                                               net::CompletionCallback());
 }
 
 }  // namespace client
 }  // namespace blimp