Export a private webstore API to call into the new inline sign-in flow.

chrome/browser/extensions/api/webstore_private/webstore_private_api.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/extensions/api/webstore_private/webstore_private_api.h"
 
 #include "base/bind_helpers.h"
 #include "base/command_line.h"
 #include "base/lazy_instance.h"
 #include "base/memory/scoped_vector.h"
@@ skipping 12 matching lines @@
 #include "chrome/browser/extensions/webstore_installer.h"
 #include "chrome/browser/gpu/gpu_feature_checker.h"
 #include "chrome/browser/profiles/profile_manager.h"
 #include "chrome/browser/signin/signin_manager.h"
 #include "chrome/browser/signin/signin_manager_factory.h"
 #include "chrome/browser/sync/profile_sync_service.h"
 #include "chrome/browser/sync/profile_sync_service_factory.h"
 #include "chrome/browser/ui/app_list/app_list_service.h"
 #include "chrome/browser/ui/app_list/app_list_util.h"
 #include "chrome/browser/ui/browser.h"
+#include "chrome/browser/ui/webui/signin/login_ui_service.h"
+#include "chrome/browser/ui/webui/signin/login_ui_service_factory.h"
 #include "chrome/common/extensions/extension_constants.h"
 #include "chrome/common/extensions/extension_l10n_util.h"
 #include "chrome/common/pref_names.h"
 #include "content/public/browser/gpu_data_manager.h"
 #include "content/public/browser/notification_details.h"
 #include "content/public/browser/notification_source.h"
 #include "content/public/browser/web_contents.h"
 #include "extensions/browser/extension_prefs.h"
 #include "extensions/browser/extension_system.h"
 #include "extensions/common/error_utils.h"
 #include "extensions/common/extension.h"
 #include "grit/chromium_strings.h"
 #include "grit/generated_resources.h"
 #include "ui/base/l10n/l10n_util.h"
 
 using content::GpuDataManager;
 
 namespace extensions {
 
 namespace BeginInstallWithManifest3 =
     api::webstore_private::BeginInstallWithManifest3;
 namespace CompleteInstall = api::webstore_private::CompleteInstall;
 namespace GetBrowserLogin = api::webstore_private::GetBrowserLogin;
 namespace GetIsLauncherEnabled = api::webstore_private::GetIsLauncherEnabled;
 namespace GetStoreLogin = api::webstore_private::GetStoreLogin;
 namespace GetWebGLStatus = api::webstore_private::GetWebGLStatus;
 namespace InstallBundle = api::webstore_private::InstallBundle;
 namespace IsInIncognitoMode = api::webstore_private::IsInIncognitoMode;
+namespace SignIn = api::webstore_private::SignIn;
 namespace SetStoreLogin = api::webstore_private::SetStoreLogin;
 
 namespace {
 
 // Holds the Approvals between the time we prompt and start the installs.
 class PendingApprovals {
  public:
   PendingApprovals();
   ~PendingApprovals();
 
@@ skipping 608 matching lines @@
   results_ = GetIsLauncherEnabled::Results::Create(IsAppLauncherEnabled());
   return true;
 }
 
 bool WebstorePrivateIsInIncognitoModeFunction::RunImpl() {
   results_ = IsInIncognitoMode::Results::Create(
       GetProfile() != GetProfile()->GetOriginalProfile());
   return true;
 }
 
+bool WebstorePrivateSignInFunction::RunImpl() {

[Ilya Sherman, 2014/03/12 06:34:29]

Hui, this CL is not nearly ready for a full review, but could you please take a
look at this method and let me know if it looks like I'm on the right track?

[guohui, 2014/03/12 18:17:39]

it looks good in general, plz see my comments below.

+  scoped_ptr<SignIn::Params> params = SignIn::Params::Create(*args_);
+  EXTENSION_FUNCTION_VALIDATE(params);
+
+  // The |continue_url| is required.
+  if (!params->continue_url)
+    return false;
+
+  // If sign-in is disallowed, give up.
+  SigninManagerBase* signin_manager =
+      SigninManagerFactory::GetForProfile(GetProfile());
+  if (!signin_manager || !signin_manager->IsSigninAllowed())
+    return false;

[Ilya Sherman, 2014/03/12 06:34:29]

Am I right to be using a SigninManager here?

[guohui, 2014/03/12 18:17:39]

yes, i think so, though i think you need to notify the webstore of the failure
through a callback, so that webstore could handle different cases properly.

+
+  // If the user is already signed in, there's nothing else to do.
+  if (!signin_manager->GetAuthenticatedUsername().empty())
+    return true;

[Ilya Sherman, 2014/03/12 06:34:29]

Is it appropriate to bail if the user is already signed in?

[guohui, 2014/03/12 18:17:39]

yup, again we should notify the webstore of the result via a callback.

+
+  // TODO(isherman): What to do if an authentication is already in progress?
+  // if (signin_manager->AuthInProgress()) ...

[Ilya Sherman, 2014/03/12 06:34:29]

What's the expected behavior if there is already an authentication in progress? 
Incidentally, does "AuthInProgress()" returning |true| mean that there is a
sign-in page open, or that the user has submitted data through a sign-in page,
but the server hasn't yet replied?

[guohui, 2014/03/12 18:17:39]

this means signin is already triggered through a sign-in page, but not finished
yet, since signin is a multi-step process involving multiple calls to Gaia and
sync server thus it could take a few seconds or more. The simplest solution is
to just return a status code via callback.

+
+  // TODO(isherman): Replace the current tab with the UI, rather than showing it
+  // in a new tab.

[Ilya Sherman, 2014/03/12 06:34:29]

Are you aware of existing code that I can re-use to do this, or do I likely need
to build some plumbing here to enable this functionality?

[guohui, 2014/03/12 18:17:39]

hmm i think you could simply navigate the current tab to the signin URL, see
example at 
https://code.google.com/p/chromium/codesearch#chromium/src/chrome/browser/ui/...

+  // TODO(isherman): Make use of the |continue_url|.
+  LoginUIService* login_ui_service =
+      LoginUIServiceFactory::GetForProfile(GetProfile());
+  login_ui_service->ShowLoginPopup();

[Ilya Sherman, 2014/03/12 06:34:29]

Is a LoginUIService the best way to show an inline weblogin page hosted on the
chrome:// protocol?

[guohui, 2014/03/12 18:17:39]

LoginUIService::ShowLoginPopup shows the login UI in a singleton tab, thus i
don't think we should use it here.

+
+  // TODO(isherman): Just for debugging:
+  DLOG(WARNING) << *params->continue_url;
+  return true;
+}
+
 }  // namespace extensions