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

Side by Side Diff: chrome/browser/ui/cocoa/extensions/extension_action_context_menu_controller.mm

Issue 421193002: Fix ExtensionServiceTest.ClearExtensionData flakiness (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: use base::DoNothing for empty callbacks Created 6 years, 4 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 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 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 #import "chrome/browser/ui/cocoa/extensions/extension_action_context_menu_contro ller.h" 5 #import "chrome/browser/ui/cocoa/extensions/extension_action_context_menu_contro ller.h"
6 6
7 #include "base/strings/sys_string_conversions.h" 7 #include "base/strings/sys_string_conversions.h"
8 #include "chrome/browser/extensions/api/extension_action/extension_action_api.h" 8 #include "chrome/browser/extensions/api/extension_action/extension_action_api.h"
9 #include "chrome/browser/extensions/extension_action_manager.h" 9 #include "chrome/browser/extensions/extension_action_manager.h"
10 #include "chrome/browser/extensions/extension_service.h" 10 #include "chrome/browser/extensions/extension_service.h"
(...skipping 41 matching lines...) Expand 10 before | Expand all | Expand 10 after
52 } 52 }
53 53
54 virtual ~AsyncUninstaller() {} 54 virtual ~AsyncUninstaller() {}
55 55
56 // ExtensionUninstallDialog::Delegate: 56 // ExtensionUninstallDialog::Delegate:
57 virtual void ExtensionUninstallAccepted() OVERRIDE { 57 virtual void ExtensionUninstallAccepted() OVERRIDE {
58 extensions::ExtensionSystem::Get(profile_) 58 extensions::ExtensionSystem::Get(profile_)
59 ->extension_service() 59 ->extension_service()
60 ->UninstallExtension(extension_->id(), 60 ->UninstallExtension(extension_->id(),
61 extensions::UNINSTALL_REASON_USER_INITIATED, 61 extensions::UNINSTALL_REASON_USER_INITIATED,
62 base::Bind(&base::DoNothing),
62 NULL); 63 NULL);
63 } 64 }
64 virtual void ExtensionUninstallCanceled() OVERRIDE {} 65 virtual void ExtensionUninstallCanceled() OVERRIDE {}
65 66
66 private: 67 private:
67 // The extension that's being uninstalled. 68 // The extension that's being uninstalled.
68 const Extension* extension_; 69 const Extension* extension_;
69 70
70 // The current profile. Weak. 71 // The current profile. Weak.
71 Profile* profile_; 72 Profile* profile_;
(...skipping 152 matching lines...) Expand 10 before | Expand all | Expand 10 after
224 return; 225 return;
225 226
226 [ExtensionPopupController showURL:url 227 [ExtensionPopupController showURL:url
227 inBrowser:browser_ 228 inBrowser:browser_
228 anchoredAt:popupPoint 229 anchoredAt:popupPoint
229 arrowLocation:info_bubble::kTopRight 230 arrowLocation:info_bubble::kTopRight
230 devMode:YES]; 231 devMode:YES];
231 } 232 }
232 233
233 @end 234 @end
OLDNEW
« no previous file with comments | « chrome/browser/ui/browser_browsertest.cc ('k') | chrome/browser/ui/views/apps/app_info_dialog/app_info_footer_panel.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698