Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(54)

Side by Side Diff: chrome/browser/extensions/api/webstore_private/webstore_private_api.cc

Issue 110373007: Delay loading the NTP after sign in until MergeSession has been performed in (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix typo in chromeos Created 6 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/extensions/api/webstore_private/webstore_private_api.h" 5 #include "chrome/browser/extensions/api/webstore_private/webstore_private_api.h"
6 6
7 #include "base/bind_helpers.h" 7 #include "base/bind_helpers.h"
8 #include "base/command_line.h" 8 #include "base/command_line.h"
9 #include "base/lazy_instance.h" 9 #include "base/lazy_instance.h"
10 #include "base/memory/scoped_vector.h" 10 #include "base/memory/scoped_vector.h"
(...skipping 433 matching lines...) Expand 10 before | Expand all | Expand 10 after
444 // Matches the AddRef in RunImpl(). 444 // Matches the AddRef in RunImpl().
445 Release(); 445 Release();
446 } 446 }
447 447
448 void WebstorePrivateBeginInstallWithManifest3Function::SigninSuccess() { 448 void WebstorePrivateBeginInstallWithManifest3Function::SigninSuccess() {
449 signin_tracker_.reset(); 449 signin_tracker_.reset();
450 450
451 SigninCompletedOrNotNeeded(); 451 SigninCompletedOrNotNeeded();
452 } 452 }
453 453
454 void WebstorePrivateBeginInstallWithManifest3Function::MergeSessionComplete(
455 const GoogleServiceAuthError& error) {
456 // TODO(rogerta): once the embeded inline flow is enabled, the code in
457 // WebstorePrivateBeginInstallWithManifest3Function::SigninSuccess()
458 // should move to here.
459 }
460
454 void WebstorePrivateBeginInstallWithManifest3Function:: 461 void WebstorePrivateBeginInstallWithManifest3Function::
455 SigninCompletedOrNotNeeded() { 462 SigninCompletedOrNotNeeded() {
456 content::WebContents* web_contents = GetAssociatedWebContents(); 463 content::WebContents* web_contents = GetAssociatedWebContents();
457 if (!web_contents) // The browser window has gone away. 464 if (!web_contents) // The browser window has gone away.
458 return; 465 return;
459 install_prompt_.reset(new ExtensionInstallPrompt(web_contents)); 466 install_prompt_.reset(new ExtensionInstallPrompt(web_contents));
460 install_prompt_->ConfirmWebstoreInstall( 467 install_prompt_->ConfirmWebstoreInstall(
461 this, 468 this,
462 dummy_extension_.get(), 469 dummy_extension_.get(),
463 &icon_, 470 &icon_,
(...skipping 210 matching lines...) Expand 10 before | Expand all | Expand 10 after
674 return true; 681 return true;
675 } 682 }
676 683
677 bool WebstorePrivateIsInIncognitoModeFunction::RunImpl() { 684 bool WebstorePrivateIsInIncognitoModeFunction::RunImpl() {
678 results_ = IsInIncognitoMode::Results::Create( 685 results_ = IsInIncognitoMode::Results::Create(
679 GetProfile() != GetProfile()->GetOriginalProfile()); 686 GetProfile() != GetProfile()->GetOriginalProfile());
680 return true; 687 return true;
681 } 688 }
682 689
683 } // namespace extensions 690 } // namespace extensions
OLDNEW
« no previous file with comments | « chrome/browser/extensions/api/webstore_private/webstore_private_api.h ('k') | chrome/browser/signin/account_reconcilor.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698