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

Unified Diff: chrome/browser/ui/intents/web_intent_picker_controller.h

Issue 10796116: [Web Intents] Basic location bar UI for window disposition picker affordance. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Refactor content settings to PageTool base class. Add stubs for mac/win. Created 8 years, 5 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/intents/web_intent_picker_controller.h
diff --git a/chrome/browser/ui/intents/web_intent_picker_controller.h b/chrome/browser/ui/intents/web_intent_picker_controller.h
index 487f76cd7bb143213ce430191027e312bb80ebf2..82b0c13fd96c41c0a0b65776bf4a1cd8edad22bd 100644
--- a/chrome/browser/ui/intents/web_intent_picker_controller.h
+++ b/chrome/browser/ui/intents/web_intent_picker_controller.h
@@ -57,6 +57,11 @@ class WebIntentPickerController
void ShowDialog(const string16& action,
const string16& type);
+ // Called by the location bar to see whether the web intents picker affordance
+ // should be shown. TODO(gbillock): refactor this into a
+ // LocationBarPageToolModel.
+ bool ShowLocationBarPickerTool();
+
protected:
// content::NotificationObserver implementation.
virtual void Observe(int type,
@@ -98,6 +103,12 @@ class WebIntentPickerController
picker_model_->set_observer(observer);
}
+ // Notify the controller that it's TabContents is hosting a web intents
+ // service.
+ void set_window_disposition_target(bool target) {
+ window_disposition_target_ = target;
+ }
+
// Called by the WebIntentsRegistry, returning |services|, which is
// a list of WebIntentServiceData matching the query.
void OnWebIntentServicesAvailable(
@@ -189,6 +200,10 @@ class WebIntentPickerController
// case, a picker may be non-NULL before it is shown.
bool picker_shown_;
+ // Set to true if the TabContents with which this controller is associated is
+ // hosting a web intents window disposition service.
+ bool window_disposition_target_;
+
// Weak pointer to the routing object for the renderer which launched the
// intent. Contains the intent data and a way to signal back to the
// client page.

Powered by Google App Engine
This is Rietveld 408576698