Merge 134293 - [cros] Open getting started guide in app window. Add support for new GSG URL.

chrome/browser/chromeos/login/existing_user_controller.cc

 // Copyright (c) 2012 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/chromeos/login/existing_user_controller.h"
 
 #include <vector>
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
@@ skipping 14 matching lines @@
 #include "chrome/browser/chromeos/customization_document.h"
 #include "chrome/browser/chromeos/dbus/dbus_thread_manager.h"
 #include "chrome/browser/chromeos/dbus/session_manager_client.h"
 #include "chrome/browser/chromeos/login/helper.h"
 #include "chrome/browser/chromeos/login/login_display_host.h"
 #include "chrome/browser/chromeos/login/login_utils.h"
 #include "chrome/browser/chromeos/login/user_manager.h"
 #include "chrome/browser/chromeos/login/wizard_controller.h"
 #include "chrome/browser/chromeos/system/statistics_provider.h"
 #include "chrome/browser/google/google_util.h"
+#include "chrome/browser/net/browser_url_util.h"
 #include "chrome/browser/policy/browser_policy_connector.h"
 #include "chrome/browser/prefs/pref_service.h"
 #include "chrome/browser/prefs/session_startup_pref.h"
 #include "chrome/browser/profiles/profile_manager.h"
 #include "chrome/browser/ui/dialog_style.h"
 #include "chrome/browser/ui/views/window.h"
 #include "chrome/common/chrome_notification_types.h"
 #include "chrome/common/chrome_switches.h"
 #include "chrome/common/net/gaia/google_service_auth_error.h"
 #include "chrome/common/pref_names.h"
 #include "content/public/browser/browser_thread.h"
 #include "content/public/browser/notification_service.h"
 #include "content/public/browser/notification_types.h"
 #include "grit/generated_resources.h"
 #include "net/http/http_auth_cache.h"
 #include "net/http/http_network_session.h"
 #include "net/http/http_transaction_factory.h"
 #include "net/url_request/url_request_context.h"
 #include "net/url_request/url_request_context_getter.h"
 #include "ui/base/l10n/l10n_util.h"
 #include "ui/views/widget/widget.h"
 
 namespace chromeos {
 
 namespace {
 
 // Url for setting up sync authentication.
 const char kSettingsSyncLoginURL[] = "chrome://settings/personal";
 
-// URL that will be opened when user logs in first time on the device.
+// Getting started guide URL, will be opened as in app window for each new
+// user who logs on the device.
+// TODO(nkostylev): Remove this when new URL is live.
+#if defined(NEW_GSG_URL)
+const char kGetStartedURLPattern[] =
+    "http://gweb-gettingstartedguide.appspot.com/";
+#else
 const char kGetStartedURLPattern[] =
     "http://www.gstatic.com/chromebook/gettingstarted/index-%s.html";
-
-// Divider that starts parameters in URL.
-const char kGetStartedParamsStartMark[] = "#";
+#endif
 
 // Parameter to be added to GetStarted URL that contains board.
-const char kGetStartedBoardParam[] = "board=%s";
+const char kGetStartedBoardParam[] = "board";
 
 // Parameter to be added to GetStarted URL
-// when first user signs in for the first time.
-// TODO(nkostylev): Uncomment once server side supports new param format.
-// const char kGetStartedOwnerParam[] = "/first";
-const char kGetStartedOwnerParam[] = "first";
+// when first user signs in for the first time (OOBE case).
+#if defined(NEW_GSG_URL)
+const char kGetStartedOwnerParam[] = "owner";
+const char kGetStartedOwnerParamValue[] = "true";
+#else
+const char kGetStartedOwnerParam[] = "#first";
+#endif
 
 // URL for account creation.
 const char kCreateAccountURL[] =
     "https://www.google.com/accounts/NewAccount?service=mail";
 
 // ChromeVox tutorial URL (used in place of "getting started" url when
 // accessibility is enabled).
 const char kChromeVoxTutorialURLPattern[] =
     "http://www.chromevox.com/tutorial/index.html?lang=%s";
 
@@ skipping 590 matching lines @@
   }
   host_->StartWizard(screen_name, params);
 }
 
 gfx::NativeWindow ExistingUserController::GetNativeWindow() const {
   return host_->GetNativeWindow();
 }
 
 void ExistingUserController::InitializeStartUrls() const {
   std::vector<std::string> start_urls;
+  // Guide URL is not added to start URLs as it should be passed as an app.
+  std::string guide_url;
+
   PrefService* prefs = g_browser_process->local_state();
-  const std::string current_locale =
-      StringToLowerASCII(prefs->GetString(prefs::kApplicationLocale));
-  std::string start_url;
   const base::ListValue *urls;
   if (UserManager::Get()->IsLoggedInAsDemoUser() &&
       CrosSettings::Get()->GetList(kStartUpUrls, &urls)) {
     // the demo user will get its start urls from the special policy if it is
     // set.
     for (base::ListValue::const_iterator it = urls->begin();
          it != urls->end(); ++it) {
       std::string url;
       if ((*it)->GetAsString(&url))
         start_urls.push_back(url);
     }
   } else {
     if (prefs->GetBoolean(prefs::kSpokenFeedbackEnabled)) {
       const char* url = kChromeVoxTutorialURLPattern;
-      start_url = base::StringPrintf(url, current_locale.c_str());
+      const std::string current_locale =
+          StringToLowerASCII(prefs->GetString(prefs::kApplicationLocale));
+      std::string vox_url = base::StringPrintf(url, current_locale.c_str());
+      start_urls.push_back(vox_url);
     } else {
-      const char* url = kGetStartedURLPattern;
-      start_url = base::StringPrintf(url, current_locale.c_str());
-      std::string params_str;
-#if 0
-      const char kMachineInfoBoard[] = "CHROMEOS_RELEASE_BOARD";
-      std::string board;
-      system::StatisticsProvider* provider =
-          system::StatisticsProvider::GetInstance();
-      if (!provider->GetMachineStatistic(kMachineInfoBoard, &board))
-        LOG(ERROR) << "Failed to get board information";
-      if (!board.empty()) {
-        params_str.append(base::StringPrintf(kGetStartedBoardParam,
-                                             board.c_str()));
-      }
-#endif
-      if (is_owner_login_)
-        params_str.append(kGetStartedOwnerParam);
-      if (!params_str.empty()) {
-        params_str.insert(0, kGetStartedParamsStartMark);
-        start_url.append(params_str);
-      }
+      guide_url = GetGettingStartedGuideURL();
     }
-    start_urls.push_back(start_url);
   }
 
   ServicesCustomizationDocument* customization =
       ServicesCustomizationDocument::GetInstance();
   if (!ServicesCustomizationDocument::WasApplied() &&
       customization->IsReady()) {
-    std::string locale = g_browser_process->GetApplicationLocale();
-    std::string initial_start_page =
-        customization->GetInitialStartPage(locale);
-    if (!initial_start_page.empty())
-      start_urls.push_back(initial_start_page);
+    // Since we don't use OEM start URL anymore, just mark as applied.
     customization->ApplyCustomization();
   }
 
-  for (size_t i = 0; i < start_urls.size(); ++i)
-    CommandLine::ForCurrentProcess()->AppendArg(start_urls[i]);
+  if (!guide_url.empty()) {
+    CommandLine::ForCurrentProcess()->AppendSwitchASCII(switches::kApp,
+                                                        guide_url);
+  } else {
+    // We should not be adding any start URLs if guide
+    // is defined as it launches as a standalone app window.
+    for (size_t i = 0; i < start_urls.size(); ++i)
+      CommandLine::ForCurrentProcess()->AppendArg(start_urls[i]);
+  }
 }
 
 void ExistingUserController::SetStatusAreaEnabled(bool enable) {
   if (!host_)
     return;
   host_->SetStatusAreaEnabled(enable);
 }
 
+#if defined(NEW_GSG_URL)
+std::string ExistingUserController::GetGettingStartedGuideURL() const {
+  GURL guide_url(kGetStartedURLPattern);
+  std::string board;
+  const char kMachineInfoBoard[] = "CHROMEOS_RELEASE_BOARD";
+  system::StatisticsProvider* provider =
+      system::StatisticsProvider::GetInstance();
+  if (!provider->GetMachineStatistic(kMachineInfoBoard, &board))
+    LOG(ERROR) << "Failed to get board information";
+  if (!board.empty()) {
+    guide_url = chrome_browser_net::AppendQueryParameter(guide_url,
+                                                         kGetStartedBoardParam,
+                                                         board);
+  }
+  if (is_owner_login_) {
+    guide_url = chrome_browser_net::AppendQueryParameter(
+        guide_url,
+        kGetStartedOwnerParam,
+        kGetStartedOwnerParamValue);
+  }
+  guide_url = google_util::AppendGoogleLocaleParam(guide_url);
+  return guide_url.spec();
+}
+#else
+std::string ExistingUserController::GetGettingStartedGuideURL() const {
+  const char* url = kGetStartedURLPattern;
+  PrefService* prefs = g_browser_process->local_state();
+  const std::string current_locale =
+      StringToLowerASCII(prefs->GetString(prefs::kApplicationLocale));
+  std::string guide_url = base::StringPrintf(url, current_locale.c_str());
+  if (is_owner_login_)
+    guide_url.append(kGetStartedOwnerParam);
+  return guide_url;
+}
+#endif
+
 void ExistingUserController::ShowError(int error_id,
                                        const std::string& details) {
   // TODO(dpolukhin): show detailed error info. |details| string contains
   // low level error info that is not localized and even is not user friendly.
   // For now just ignore it because error_text contains all required information
   // for end users, developers can see details string in Chrome logs.
   VLOG(1) << details;
   HelpAppLauncher::HelpTopic help_topic_id;
   NetworkLibrary* network_library = CrosLibrary::Get()->GetNetworkLibrary();
   bool is_offline = !network_library || !network_library->Connected();
@@ skipping 23 matching lines @@
   // changed.
   UserManager::Get()->SaveUserOAuthStatus(username,
                                           User::OAUTH_TOKEN_STATUS_INVALID);
 
   login_display_->SetUIEnabled(true);
   SetStatusAreaEnabled(true);
   login_display_->ShowGaiaPasswordChanged(username);
 }
 
 }  // namespace chromeos