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

Unified Diff: chrome/browser/ui/toolbar/action_box_button_controller.h

Issue 11073009: Include individual share intents in the action box and allow direct triggering without an intermedi… (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: fix nit, rebase on master Created 8 years, 2 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/toolbar/action_box_button_controller.h
diff --git a/chrome/browser/ui/toolbar/action_box_button_controller.h b/chrome/browser/ui/toolbar/action_box_button_controller.h
index 45c56928f63afcbe1c648200a9d16b08762bb0bb..75ac869057e12137664e4735a1e9c24f62c43c14 100644
--- a/chrome/browser/ui/toolbar/action_box_button_controller.h
+++ b/chrome/browser/ui/toolbar/action_box_button_controller.h
@@ -68,11 +68,25 @@ class ActionBoxButtonController : public ui::SimpleMenuModel::Delegate,
const content::NotificationSource& source,
const content::NotificationDetails& details) OVERRIDE;
+ // Handles "share with X" commands.
+ void TriggerExplicitShareIntent(const GURL& share_service_url);
+
+ // Handles the "Find places to share" command. Navigates the browser to the
+ // web store to find extensions with share intents.
+ void NavigateToWebStoreShareIntentsList();
+
Browser* browser_;
Delegate* delegate_;
// The command ID to assign to the next extension that needs one.
Peter Kasting 2012/10/15 19:33:09 Nit: Comment is wrong
skare_ 2012/10/15 21:07:27 removed member On 2012/10/15 19:33:09, Peter Kasti
+ int next_share_intent_command_id_;
+
+ // The share service strings that have commands associated with them.
+ typedef std::map<int, GURL> ShareIntentServiceCommandMap;
+ ShareIntentServiceCommandMap share_intent_service_ids_;
+
+ // The command ID to assign to the next extension that needs one.
int next_extension_command_id_;
// The extension IDs that have commands associated with them.

Powered by Google App Engine
This is Rietveld 408576698