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

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

Issue 12207105: [win] Remove 'installing...' icons from the app list if the install fails. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: fix mac Created 7 years, 10 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
« no previous file with comments | « no previous file | chrome/browser/ui/app_list/app_list_util.h » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 485 matching lines...) Expand 10 before | Expand all | Expand 10 after
496 SendResponse(true); 496 SendResponse(true);
497 497
498 // Matches the AddRef in RunImpl(). 498 // Matches the AddRef in RunImpl().
499 Release(); 499 Release();
500 } 500 }
501 501
502 void CompleteInstallFunction::OnExtensionInstallFailure( 502 void CompleteInstallFunction::OnExtensionInstallFailure(
503 const std::string& id, 503 const std::string& id,
504 const std::string& error, 504 const std::string& error,
505 WebstoreInstaller::FailureReason reason) { 505 WebstoreInstaller::FailureReason reason) {
506 #if defined(ENABLE_APP_LIST)
507 chrome::NotifyAppListOfExtensionInstallFailure(profile(), id);
508 #endif
506 if (test_webstore_installer_delegate) { 509 if (test_webstore_installer_delegate) {
507 test_webstore_installer_delegate->OnExtensionInstallFailure( 510 test_webstore_installer_delegate->OnExtensionInstallFailure(
508 id, error, reason); 511 id, error, reason);
509 } 512 }
510 513
511 error_ = error; 514 error_ = error;
512 SendResponse(false); 515 SendResponse(false);
513 516
514 // Matches the AddRef in RunImpl(). 517 // Matches the AddRef in RunImpl().
515 Release(); 518 Release();
(...skipping 65 matching lines...) Expand 10 before | Expand all | Expand 10 after
581 &GetIsLauncherEnabledFunction::OnIsLauncherCheckCompleted, this)); 584 &GetIsLauncherEnabledFunction::OnIsLauncherCheckCompleted, this));
582 return true; 585 return true;
583 } 586 }
584 587
585 void GetIsLauncherEnabledFunction::OnIsLauncherCheckCompleted(bool is_enabled) { 588 void GetIsLauncherEnabledFunction::OnIsLauncherCheckCompleted(bool is_enabled) {
586 SetResult(Value::CreateBooleanValue(is_enabled)); 589 SetResult(Value::CreateBooleanValue(is_enabled));
587 SendResponse(true); 590 SendResponse(true);
588 } 591 }
589 592
590 } // namespace extensions 593 } // namespace extensions
OLDNEW
« no previous file with comments | « no previous file | chrome/browser/ui/app_list/app_list_util.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698