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

Unified Diff: chrome/browser/ui/views/location_bar/location_bar_view.h

Issue 768413003: TEST ONLY - DO NOT SUBMIT - FOR TRYBOTS (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix bad merge Created 5 years, 11 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/location_bar/location_bar_view.h
diff --git a/chrome/browser/ui/views/location_bar/location_bar_view.h b/chrome/browser/ui/views/location_bar/location_bar_view.h
index 17385d96891f3194e22a9278f2fe731d524d31cd..c99cf11ae31282c8a22e8303a8e6024b0ae60aca 100644
--- a/chrome/browser/ui/views/location_bar/location_bar_view.h
+++ b/chrome/browser/ui/views/location_bar/location_bar_view.h
@@ -39,11 +39,9 @@ class KeywordHintView;
class LocationIconView;
class OpenPDFInReaderView;
class ManagePasswordsIconView;
-class OriginChipView;
class PageActionWithBadgeView;
class PageActionImageView;
class Profile;
-class SearchButton;
class SelectedKeywordView;
class StarView;
class TemplateURLService;
@@ -54,10 +52,6 @@ namespace content {
struct SSLStatus;
}
-namespace gfx {
-class SlideAnimation;
-}
-
namespace views {
class BubbleDelegateView;
class ImageButton;
@@ -81,7 +75,6 @@ class LocationBarView : public LocationBar,
public views::DragController,
public OmniboxEditController,
public DropdownBarHostDelegate,
- public gfx::AnimationDelegate,
public TemplateURLServiceObserver,
public SearchModelObserver {
public:
@@ -255,7 +248,6 @@ class LocationBarView : public LocationBar,
// OmniboxEditController:
void Update(const content::WebContents* contents) override;
void ShowURL() override;
- void EndOriginChipAnimations(bool cancel_fade) override;
ToolbarModel* GetToolbarModel() override;
content::WebContents* GetWebContents() override;
@@ -268,11 +260,6 @@ class LocationBarView : public LocationBar,
static const int kItemPadding;
// Amount of padding built into the standard omnibox icons.
static const int kIconInternalPadding;
- // Amount of padding to place between the origin chip and the leading edge of
- // the location bar.
- static const int kOriginChipEdgeItemPadding;
- // Amount of padding built into the origin chip.
- static const int kOriginChipBuiltinPadding;
// Space between the edge and a bubble.
static const int kBubblePadding;
@@ -343,9 +330,6 @@ class LocationBarView : public LocationBar,
// directions from its current location.
double GetValueForAnimation(bool hide) const;
- // Resets |show_url_animation_| and the color changes it causes.
- void ResetShowAnimationAndColors();
-
// LocationBar:
void ShowFirstRunBubble() override;
GURL GetDestinationURL() const override;
@@ -398,16 +382,11 @@ class LocationBarView : public LocationBar,
void OnSetFocus() override;
InstantController* GetInstant() override;
const ToolbarModel* GetToolbarModel() const override;
- void HideURL() override;
// DropdownBarHostDelegate:
void SetFocusAndSelection(bool select_all) override;
void SetAnimationOffset(int offset) override;
- // gfx::AnimationDelegate:
- void AnimationProgressed(const gfx::Animation* animation) override;
- void AnimationEnded(const gfx::Animation* animation) override;
-
// TemplateURLServiceObserver:
void OnTemplateURLServiceChanged() override;
@@ -428,9 +407,6 @@ class LocationBarView : public LocationBar,
// Object used to paint the border.
scoped_ptr<views::Painter> border_painter_;
- // The origin chip that may appear in the location bar.
- OriginChipView* origin_chip_view_;
-
// An icon to the left of the edit field.
LocationIconView* location_icon_view_;
@@ -486,9 +462,6 @@ class LocationBarView : public LocationBar,
// The star.
StarView* star_view_;
- // The search/go button.
- SearchButton* search_button_;
-
// Whether we're in popup mode. This value also controls whether the location
// bar is read-only.
const bool is_popup_mode_;
@@ -510,46 +483,6 @@ class LocationBarView : public LocationBar,
// toolbar in the right location.
int dropdown_animation_offset_;
- // Origin chip animations.
- //
- // For the "show URL" animation, we instantly hide the origin chip and show
- // the |omnibox_view_| in its place, containing the complete URL. However, we
- // clip that view (using the XXX_leading_inset_ and XXX_width_ members) so
- // that only the hostname is visible. We also offset the omnibox (using the
- // XXX_offset_ members) so the hostname is in the same place as it was in the
- // origin chip. Finally, we set the selection text and background color of
- // the text to match the pressed origin chip. Then, as the animation runs,
- // all of these values are animated to their steady-state values (no omnibox
- // offset, no inset, width equal to the full omnibox text [which is reset to
- // "no width clamp" after the animation ends], and standard selection colors).
- //
- // For the hide animation, we run the positioning and clipping parts of the
- // animation in reverse, but instead of changing the selection color, because
- // there usually isn't a selection when hiding, we leave the omnibox colors
- // alone, and when the hide animation has ended, tell the origin chip to
- // fade-in its background.
- scoped_ptr<gfx::SlideAnimation> show_url_animation_;
- scoped_ptr<gfx::SlideAnimation> hide_url_animation_;
- // The omnibox offset may be positive or negative. The starting offset is the
- // amount necessary to shift the |omnibox_view_| by such that the hostname
- // portion of the URL aligns with the hostname in the origin chip. As the
- // show animation runs, the current offset gradually moves to 0.
- int starting_omnibox_offset_;
- int current_omnibox_offset_;
- // The leading inset is always positive. The starting inset is the width of
- // the text between the leading edge of the omnibox and the edge of the
- // hostname, which is clipped off at the start of the show animation. Note
- // that in RTL mode, this will be the part of the URL that is logically after
- // the hostname. As the show animation runs, the current inset gradually
- // moves to 0.
- int starting_omnibox_leading_inset_;
- int current_omnibox_leading_inset_;
- // The width is always positive. The ending width is the width of the entire
- // omnibox URL. As the show animation runs, the current width gradually moves
- // from the width of the hostname to the ending value.
- int current_omnibox_width_;
- int ending_omnibox_width_;
-
// This is a debug state variable that stores if the WebContents was null
// during the last RefreshPageAction.
bool web_contents_null_at_last_refresh_;
« no previous file with comments | « chrome/browser/ui/views/frame/browser_view.cc ('k') | chrome/browser/ui/views/location_bar/location_bar_view.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698