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); |
}; |