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

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

Issue 731423002: Remove OriginChip code. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 1 month 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 c418febb561203fa006b066c3ee1df974b415870..571c59420dc6a5f7e20978d07029d3dc8c28ea77 100644
--- a/chrome/browser/ui/views/location_bar/location_bar_view.h
+++ b/chrome/browser/ui/views/location_bar/location_bar_view.h
@@ -39,7 +39,6 @@ class KeywordHintView;
class LocationIconView;
class OpenPDFInReaderView;
class ManagePasswordsIconView;
-class OriginChipView;
class PageActionWithBadgeView;
class PageActionImageView;
class Profile;
@@ -54,10 +53,6 @@ namespace content {
struct SSLStatus;
}
-namespace gfx {
-class SlideAnimation;
-}
-
namespace views {
class BubbleDelegateView;
class ImageButton;
@@ -81,7 +76,6 @@ class LocationBarView : public LocationBar,
public views::DragController,
public OmniboxEditController,
public DropdownBarHostDelegate,
- public gfx::AnimationDelegate,
public TemplateURLServiceObserver,
public SearchModelObserver {
public:
@@ -252,7 +246,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;
@@ -265,11 +258,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;
@@ -335,9 +323,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;
@@ -391,16 +376,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;
@@ -421,9 +401,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_;
@@ -506,32 +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
@@ -540,11 +491,6 @@ class LocationBarView : public LocationBar,
// moves to 0.
int starting_omnibox_leading_inset_;
Justin Donnelly 2014/11/21 19:15:24 These properties are no longer used.
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_;
DISALLOW_COPY_AND_ASSIGN(LocationBarView);
};

Powered by Google App Engine
This is Rietveld 408576698