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

Unified Diff: chrome/browser/ui/views/toolbar/toolbar_view.h

Issue 2394143004: Harmony - Update anchors for the rest of the location bar icons that (Closed)
Patch Set: actually fix Created 4 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/views/toolbar/toolbar_view.h
diff --git a/chrome/browser/ui/views/toolbar/toolbar_view.h b/chrome/browser/ui/views/toolbar/toolbar_view.h
index 16b0d1e3a6b2b69067d3809fb72a8fdc21200ee7..0a7e481ab636cab15d98ff57789ad247f3b69a9b 100644
--- a/chrome/browser/ui/views/toolbar/toolbar_view.h
+++ b/chrome/browser/ui/views/toolbar/toolbar_view.h
@@ -14,6 +14,8 @@
#include "chrome/browser/ui/toolbar/back_forward_menu_model.h"
#include "chrome/browser/ui/views/location_bar/location_bar_view.h"
#include "components/prefs/pref_member.h"
+#include "components/translate/core/browser/translate_step.h"
+#include "components/translate/core/common/translate_errors.h"
#include "ui/base/accelerators/accelerator.h"
#include "ui/views/accessible_pane_view.h"
#include "ui/views/controls/button/menu_button.h"
@@ -28,6 +30,11 @@ class HomeButton;
class ReloadButton;
class ToolbarButton;
+namespace autofill {
+class SaveCardBubbleController;
+class SaveCardBubbleView;
+}
+
namespace bookmarks {
class BookmarkBubbleObserver;
}
@@ -80,16 +87,17 @@ class ToolbarView : public views::AccessiblePaneView,
bool already_bookmarked,
bookmarks::BookmarkBubbleObserver* observer);
- // Returns the view to which the "Save credit card" bubble should be anchored.
- views::View* GetSaveCreditCardBubbleAnchor();
-
- // Returns the view to which the Translate bubble should be anchored.
- views::View* GetTranslateBubbleAnchor();
-
- // Adds |anchor_view| as an observer of |bubble_widget| to track its
- // visibility.
- void OnBubbleCreatedForAnchor(views::View* anchor_view,
- views::Widget* bubble_widget);
+ // Shows a bubble offering to save a credit card and anchors it appropriately.
+ autofill::SaveCardBubbleView* ShowSaveCreditCardBubble(
+ content::WebContents* contents,
+ autofill::SaveCardBubbleController* controller,
+ bool is_user_gesture);
+
+ // Shows the translate bubble and anchors it appropriately.
+ void ShowTranslateBubble(content::WebContents* web_contents,
+ translate::TranslateStep step,
+ translate::TranslateErrors::Type error_type,
+ bool is_user_gesture);
// Returns the maximum width the browser actions container can have.
int GetMaxBrowserActionsWidth() const;
« no previous file with comments | « chrome/browser/ui/views/passwords/manage_passwords_bubble_view.cc ('k') | chrome/browser/ui/views/toolbar/toolbar_view.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698