[linux_views][Win] spoof proof redesign infobar extension with tab.

chrome/browser/ui/views/infobars/infobar_view.h

 // Copyright (c) 2011 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_VIEWS_INFOBARS_INFOBAR_VIEW_H_
 #define CHROME_BROWSER_UI_VIEWS_INFOBARS_INFOBAR_VIEW_H_
 #pragma once
 
 #include "base/task.h"
 #include "ui/base/animation/animation_delegate.h"
 #include "views/controls/button/button.h"
 #include "views/focus/focus_manager.h"
 
 class InfoBarContainer;
 class InfoBarDelegate;
+class SkPath;
 
 namespace ui {
 class SlideAnimation;
 }
 
 namespace views {
 class ExternalFocusTracker;
 class ImageButton;
 class ImageView;
 class Label;
@@ skipping 39 matching lines @@
   // Makes the infobar hidden.  If |animate| is true, the infobar is first
   // animated to zero size.  Once the infobar is hidden, it is removed from its
   // container (triggering its deletion), and its delegate is closed.
   void Hide(bool animate);
 
   // Closes the InfoBar immediately and removes it from its container. Notifies
   // the delegate that it has closed. The InfoBar is deleted after this function
   // is called.
   void Close();
 
-  // Paint the arrow on |canvas|. |arrow_center_x| indicates the
-  // desired location of the center of the arrow in the |outer_view|
-  // coordinate system.
-  void PaintArrow(gfx::Canvas* canvas, View* outer_view, int arrow_center_x);
+  // Overlap the previous view by this amount, vertically, so that
+  // this InfoBarView may draw its tab on top.
+  int vertical_overlap() const { return preferred_tab_height(); }
 
  protected:
   // The target height of the InfoBar, regardless of what its current height
   // is (due to animation).
   static const int kDefaultTargetHeight;
   static const int kButtonButtonSpacing;
   static const int kEndOfLabelSpacing;
 
   virtual ~InfoBarView();
 
@@ skipping 38 matching lines @@
   void set_target_height(int height) { target_height_ = height; }
 
   ui::SlideAnimation* animation() { return animation_.get(); }
 
   // These return x coordinates delimiting the usable area for subclasses to lay
   // out their controls.
   int StartX() const;
   int EndX() const;
 
   // Returns a centered y-position of a control of height specified in
-  // |prefsize| within the standard InfoBar height. Stable during an animation.
-  int CenterY(const gfx::Size prefsize) const;
-
-  // Returns a centered y-position of a control of height specified in
   // |prefsize| within the standard InfoBar height, adjusted according to the
   // current amount of animation offset the |parent| InfoBar currently has.
   // Changes during an animation.
   int OffsetY(const gfx::Size prefsize) const;
 
  private:
+  static const int kTabPadding;
+  static const int kCurveDistance;
   static const int kHorizontalPadding;
+  static const int kIconWidth;
+  static const int kTabHeight;
+  static const int kTabStrokeWidth;
+  static const int kTabWidth;
+
+  friend class InfoBarBackground;

[Peter Kasting, 2011/03/07 20:14:30]

Nit: If these need to be friends, we're doing something wrong.  We should be
making members public instead.

[Sheridan Rawlins, 2011/03/08 01:38:19]

Fixed by using views::View and casting to InfoBarView where needed to get its
public methods

+  friend class InfoBarContainer;
 
   // views::View:
   virtual AccessibilityTypes::Role GetAccessibleRole();
   virtual gfx::Size GetPreferredSize();
+  virtual void OnBoundsChanged();
+  virtual void PaintChildren(gfx::Canvas* canvas);
 
   // views::FocusChangeListener:
   virtual void FocusWillChange(View* focused_before, View* focused_now);
 
   // ui::AnimationDelegate:
   virtual void AnimationEnded(const ui::Animation* animation);
 
+  // Use the same clip path as the |fill_path_|.
+  SkPath* clip_path() const { return fill_path_.get(); }

[Peter Kasting, 2011/03/07 20:14:30]

Nit: Remove this function and call fill_path() directly where needed.

[Sheridan Rawlins, 2011/03/08 01:38:19]

Done.

+  SkPath* fill_path() const { return fill_path_.get(); }
+  SkPath* stroke_path() const { return stroke_path_.get(); }
+
+  // Returns a centered y-position of a control of height specified in
+  // |prefsize| within the standard InfoBar height. Stable during an animation.
+  int CenterY(const gfx::Size prefsize) const;
+
+  int preferred_tab_height() const;

[Peter Kasting, 2011/03/07 20:14:30]

Nit: Use CamelCase because neither of these is a cheap inline accessor
(Animation::GetCurrentValue() is virtual).

Rename "Preferred" to "Current" for both of these.

Eliminate vertical_overlap() and just make CurrentTabHeight() public.

[Sheridan Rawlins, 2011/03/08 01:38:19]

Choosing Animated instead of Preferred or Current, since that's more accurately
what it is - it's called both by InfoBarContainer to get the right size for
laying out, which then indirectly calls OnBoundsChanged which gets it again.

Done.

+  int preferred_bar_height() const;
+
   // Destroys the external focus tracker, if present. If |restore_focus| is
   // true, restores focus to the view tracked by the focus tracker before doing
   // so.
   void DestroyFocusTracker(bool restore_focus);
 
   // Deletes this object (called after a return to the message loop to allow
   // the stack in ViewHierarchyChanged to unwind).
   void DeleteSelf();
 
   // The InfoBar's container
@@ skipping 11 matching lines @@
   // The animation that runs when the InfoBar is opened or closed.
   scoped_ptr<ui::SlideAnimation> animation_;
 
   // Tracks and stores the last focused view which is not the InfoBar or any of
   // its children. Used to restore focus once the InfoBar is closed.
   scoped_ptr<views::ExternalFocusTracker> focus_tracker_;
 
   // Used to delete this object after a return to the message loop.
   ScopedRunnableMethodFactory<InfoBarView> delete_factory_;
 
-  // The target height for the InfoBarView.
+  // The target height for the bar portion of the InfoBarView.
   int target_height_;
 
+  scoped_ptr<SkPath> fill_path_;
+  scoped_ptr<SkPath> stroke_path_;
+
   DISALLOW_COPY_AND_ASSIGN(InfoBarView);
 };
 
 #endif  // CHROME_BROWSER_UI_VIEWS_INFOBARS_INFOBAR_VIEW_H_