Resolve issue from feedback for 1417363004

chrome/browser/extensions/api/gcd_private/gcd_private_api.cc

 // Copyright 2014 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 "chrome/browser/extensions/api/gcd_private/gcd_private_api.h"
 
+#include "base/command_line.h"
 #include "base/lazy_instance.h"
 #include "base/location.h"
 #include "base/memory/linked_ptr.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/single_thread_task_runner.h"
 #include "base/strings/stringprintf.h"
 #include "base/thread_task_runner_handle.h"
 #include "chrome/browser/local_discovery/cloud_device_list.h"
 #include "chrome/browser/local_discovery/cloud_print_printer_list.h"
 #include "chrome/browser/local_discovery/gcd_api_flow.h"
 #include "chrome/browser/local_discovery/gcd_constants.h"
 #include "chrome/browser/local_discovery/privet_device_lister_impl.h"
 #include "chrome/browser/local_discovery/privet_http_asynchronous_factory.h"
 #include "chrome/browser/local_discovery/privet_http_impl.h"
 #include "chrome/browser/local_discovery/privetv3_session.h"
 #include "chrome/browser/local_discovery/service_discovery_shared_client.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/signin/profile_oauth2_token_service_factory.h"
 #include "chrome/browser/signin/signin_manager_factory.h"
+#include "chrome/common/chrome_switches.h"
 #include "components/signin/core/browser/profile_oauth2_token_service.h"
 #include "components/signin/core/browser/signin_manager.h"
 #include "components/signin/core/browser/signin_manager_base.h"
 #include "extensions/browser/event_router.h"
 #include "net/base/net_util.h"
 
 #if defined(ENABLE_WIFI_BOOTSTRAPPING)
 #include "chrome/browser/local_discovery/wifi/wifi_manager.h"
 #endif
 
@@ skipping 33 matching lines @@
                                     params.Pass()));
   return event.Pass();
 }
 
 GcdPrivateAPI::GCDApiFlowFactoryForTests* g_gcd_api_flow_factory = NULL;
 
 base::LazyInstance<BrowserContextKeyedAPIFactory<GcdPrivateAPI> > g_factory =
     LAZY_INSTANCE_INITIALIZER;
 
 scoped_ptr<local_discovery::GCDApiFlow> MakeGCDApiFlow(Profile* profile) {
-  if (g_gcd_api_flow_factory) {
+  if (g_gcd_api_flow_factory)
     return g_gcd_api_flow_factory->CreateGCDApiFlow();
-  }
 
   ProfileOAuth2TokenService* token_service =
       ProfileOAuth2TokenServiceFactory::GetForProfile(profile);
   if (!token_service)
     return scoped_ptr<local_discovery::GCDApiFlow>();
   SigninManagerBase* signin_manager =
       SigninManagerFactory::GetInstance()->GetForProfile(profile);
   if (!signin_manager)
     return scoped_ptr<local_discovery::GCDApiFlow>();
   return local_discovery::GCDApiFlow::Create(
@@ skipping 121 matching lines @@
   DCHECK(browser_context_);
   if (EventRouter::Get(context)) {
     EventRouter::Get(context)
         ->RegisterObserver(this, gcd_private::OnDeviceStateChanged::kEventName);
     EventRouter::Get(context)
         ->RegisterObserver(this, gcd_private::OnDeviceRemoved::kEventName);
   }
 }
 
 GcdPrivateAPIImpl::~GcdPrivateAPIImpl() {
-  if (EventRouter::Get(browser_context_)) {
+  if (EventRouter::Get(browser_context_))
     EventRouter::Get(browser_context_)->UnregisterObserver(this);
-  }
 }
 
 void GcdPrivateAPIImpl::OnListenerAdded(const EventListenerInfo& details) {
   if (details.event_name == gcd_private::OnDeviceStateChanged::kEventName ||
       details.event_name == gcd_private::OnDeviceRemoved::kEventName) {
     num_device_listeners_++;
 
     if (num_device_listeners_ == 1) {
       service_discovery_client_ =
           local_discovery::ServiceDiscoverySharedClient::GetInstance();
@@ skipping 74 matching lines @@
     return false;
 
   privet_device_lister_->DiscoverNewDevices(true);
 
   return true;
 }
 
 void GcdPrivateAPIImpl::CreateSession(const std::string& service_name,
                                       const CreateSessionCallback& callback) {
   int session_id = last_session_id_++;
+  if (!base::CommandLine::ForCurrentProcess()->HasSwitch(
+          switches::kEnablePrivetV3)) {
+    return callback.Run(session_id, gcd_private::STATUS_SESSIONERROR,
+                        base::DictionaryValue());
+  }
   scoped_ptr<local_discovery::PrivetHTTPAsynchronousFactory> factory(
       local_discovery::PrivetHTTPAsynchronousFactory::CreateInstance(
           browser_context_->GetRequestContext()));
   auto& session_data = sessions_[session_id];
   session_data.http_resolution.reset(
       factory->CreatePrivetHTTP(service_name).release());
   session_data.http_resolution->Start(
       base::Bind(&GcdPrivateAPIImpl::OnServiceResolved, base::Unretained(this),
                  session_id, callback));
 }
@@ skipping 93 matching lines @@
 #else
   callback.Run(false);
 #endif
 }
 
 #if defined(ENABLE_WIFI_BOOTSTRAPPING)
 void GcdPrivateAPIImpl::OnWifiPassword(const SuccessCallback& callback,
                                        bool success,
                                        const std::string& ssid,
                                        const std::string& password) {
-  if (success) {
+  if (success)
     wifi_passwords_[ssid] = password;
-  }
 
   callback.Run(success);
 }
 
 void GcdPrivateAPIImpl::StartWifiIfNotStarted() {
   if (!wifi_manager_) {
     wifi_manager_ = local_discovery::wifi::WifiManager::Create();
     wifi_manager_->Start();
   }
 }
@@ skipping 436 matching lines @@
 }
 
 GcdPrivateGetCommandsListFunction::~GcdPrivateGetCommandsListFunction() {
 }
 
 bool GcdPrivateGetCommandsListFunction::RunAsync() {
   return false;
 }
 
 }  // namespace extensions