| Index: chrome/browser/ui/browser_commands.cc
|
| diff --git a/chrome/browser/ui/browser_commands.cc b/chrome/browser/ui/browser_commands.cc
|
| index fbd211b945da6567a12f91d43ac75dca6589610f..0cda9b9361e30371394e561f5306c44a8aecd452 100644
|
| --- a/chrome/browser/ui/browser_commands.cc
|
| +++ b/chrome/browser/ui/browser_commands.cc
|
| @@ -18,6 +18,7 @@
|
| #include "chrome/browser/devtools/devtools_window.h"
|
| #include "chrome/browser/dom_distiller/tab_utils.h"
|
| #include "chrome/browser/extensions/api/commands/command_service.h"
|
| +#include "chrome/browser/extensions/api/extension_action/extension_action_api.h"
|
| #include "chrome/browser/extensions/tab_helper.h"
|
| #include "chrome/browser/favicon/favicon_tab_helper.h"
|
| #include "chrome/browser/lifetime/application_lifetime.h"
|
| @@ -745,18 +746,16 @@ void BookmarkCurrentPage(Browser* browser) {
|
| switch (command_type) {
|
| case extensions::CommandService::NAMED:
|
| browser->window()->ExecuteExtensionCommand(extension, command);
|
| - return;
|
| -
|
| + break;
|
| case extensions::CommandService::BROWSER_ACTION:
|
| - // BookmarkCurrentPage is called through a user gesture, so it is safe
|
| - // to call ShowBrowserActionPopup.
|
| - browser->window()->ShowBrowserActionPopup(extension);
|
| - return;
|
| -
|
| case extensions::CommandService::PAGE_ACTION:
|
| - browser->window()->ShowPageActionPopup(extension);
|
| - return;
|
| + // BookmarkCurrentPage is called through a user gesture, so it is safe
|
| + // to grant the active tab permission.
|
| + extensions::ExtensionActionAPI::Get(browser->profile())->
|
| + ShowExtensionActionPopup(extension, browser, true);
|
| + break;
|
| }
|
| + return;
|
| }
|
|
|
| BookmarkCurrentPageInternal(browser);
|
|
|