[Web Intents] Basic location bar UI for window disposition picker affordance.

chrome/browser/ui/gtk/location_bar_view_gtk.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CHROME_BROWSER_UI_GTK_LOCATION_BAR_VIEW_GTK_H_
 #define CHROME_BROWSER_UI_GTK_LOCATION_BAR_VIEW_GTK_H_
 
 #include <gtk/gtk.h>
 
 #include <map>
@@ skipping 126 matching lines @@
                                 InstantCompleteBehavior behavior) OVERRIDE;
   virtual string16 GetInputString() const OVERRIDE;
   virtual WindowOpenDisposition GetWindowOpenDisposition() const OVERRIDE;
   virtual content::PageTransition GetPageTransition() const OVERRIDE;
   virtual void AcceptInput() OVERRIDE;
   virtual void FocusLocation(bool select_all) OVERRIDE;
   virtual void FocusSearch() OVERRIDE;
   virtual void UpdateContentSettingsIcons() OVERRIDE;
   virtual void UpdatePageActions() OVERRIDE;
   virtual void InvalidatePageActions() OVERRIDE;
+  virtual void UpdateWebIntentsTool() OVERRIDE;

[Bernhard Bauer, 2012/07/24 20:37:03]

Is this method actually called from outside of this class? If not, you could
remove it from the LocationBar interface and make it private.

   virtual void SaveStateToContents(content::WebContents* contents) OVERRIDE;
   virtual void Revert() OVERRIDE;
   virtual const OmniboxView* GetLocationEntry() const OVERRIDE;
   virtual OmniboxView* GetLocationEntry() OVERRIDE;
   virtual LocationBarTesting* GetLocationBarForTesting() OVERRIDE;
 
   // LocationBarTesting:
   virtual int PageActionCount() OVERRIDE;
   virtual int PageActionVisibleCount() OVERRIDE;
   virtual ExtensionAction* GetPageAction(size_t index) OVERRIDE;
   virtual ExtensionAction* GetVisiblePageAction(size_t index) OVERRIDE;
   virtual void TestPageActionPressed(size_t index) OVERRIDE;
 
   // content::NotificationObserver:
   virtual void Observe(int type,
                        const content::NotificationSource& source,
                        const content::NotificationDetails& details) OVERRIDE;
 
   // CommandObserver:
   virtual void EnabledStateChangedForCommand(int id, bool enabled) OVERRIDE;
 
   // Edit background color.
   static const GdkColor kBackgroundColor;
 
  private:
+  // TODO(gbillock): cause this public interface to be a platform-independent
+  // model interface for all the tools displayed in the location bar.

[Bernhard Bauer, 2012/07/24 20:37:03]

Can you explain what this class does? IIUC, at the moment it's still specific to
Web Intents, but you want to abstract it in the future? For now, we should
probably still have a name with Web Intents though, because a general name would
imply that the class is general-use.

+  class PageToolViewGtk : public ui::AnimationDelegate {
+   public:
+    PageToolViewGtk(const LocationBarViewGtk* parent);
+    virtual ~PageToolViewGtk();
+
+    GtkWidget* widget();
+
+    bool IsVisible();
+    void Update(TabContents* tab_contents);  // TODO(gbillock): make TC const
+
+    // Overridden from ui::AnimationDelegate:
+    virtual void AnimationProgressed(const ui::Animation* animation) OVERRIDE;
+    virtual void AnimationEnded(const ui::Animation* animation) OVERRIDE;
+    virtual void AnimationCanceled(const ui::Animation* animation) OVERRIDE;
+
+   private:
+    // Start the process of showing the label.
+    void StartAnimating();
+
+    CHROMEGTK_CALLBACK_1(PageToolViewGtk, gboolean, OnButtonPressed, GdkEvent*);
+    CHROMEGTK_CALLBACK_1(PageToolViewGtk, gboolean, OnExpose, GdkEventExpose*);
+
+    // The widgets for this view.
+    ui::OwnedWidgetGtk alignment_;
+    ui::OwnedWidgetGtk event_box_;
+    GtkWidget* hbox_;
+    ui::OwnedWidgetGtk image_;
+
+    // Explanatory text (e.g. "popup blocked").
+    ui::OwnedWidgetGtk label_;
+
+    // The owning LocationBarViewGtk.
+    const LocationBarViewGtk* parent_;
+
+    // When we show explanatory text, we slide it in/out.
+    ui::SlideAnimation animation_;
+
+    // The label's default requisition (cached so we can animate accordingly).
+    GtkRequisition label_req_;
+
+    base::WeakPtrFactory<PageToolViewGtk> weak_factory_;

[Bernhard Bauer, 2012/07/24 20:37:03]

Is this used?

+
+    DISALLOW_COPY_AND_ASSIGN(PageToolViewGtk);
+  };
+
   class ContentSettingImageViewGtk : public BubbleDelegateGtk,
                                      public ui::AnimationDelegate {
    public:
     ContentSettingImageViewGtk(ContentSettingsType content_type,
                                const LocationBarViewGtk* parent);
     virtual ~ContentSettingImageViewGtk();
 
     GtkWidget* widget() { return alignment_.get(); }
 
     bool IsVisible();
@@ skipping 282 matching lines @@
   ui::OwnedWidgetGtk content_setting_hbox_;
   ScopedVector<ContentSettingImageViewGtk> content_setting_views_;
 
   // Extension page actions.
   std::vector<ExtensionAction*> page_actions_;
 
   // Extension page action icons.
   ui::OwnedWidgetGtk page_action_hbox_;
   ScopedVector<PageActionViewGtk> page_action_views_;
 
+  // Control for web intents window disposition picker control.
+  ui::OwnedWidgetGtk web_intents_hbox_;
+  PageToolViewGtk web_intents_tool_view_;
+
   // The widget that contains our tab hints and the location bar.
   GtkWidget* entry_box_;
 
   // Area on the left shown when in tab to search mode.
   GtkWidget* tab_to_search_alignment_;
   GtkWidget* tab_to_search_box_;
   GtkWidget* tab_to_search_magnifier_;
   GtkWidget* tab_to_search_full_label_;
   GtkWidget* tab_to_search_partial_label_;
 
@@ skipping 55 matching lines @@
   BooleanPrefMember edit_bookmarks_enabled_;
 
   // Used to remember the URL and title text when drag&drop has begun.
   GURL drag_url_;
   string16 drag_title_;
 
   DISALLOW_COPY_AND_ASSIGN(LocationBarViewGtk);
 };
 
 #endif  // CHROME_BROWSER_UI_GTK_LOCATION_BAR_VIEW_GTK_H_