View API/implementation cleanup:...

ui/views/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 UI_VIEWS_VIEW_H_
 #define UI_VIEWS_VIEW_H_
 
 #include <vector>
 
 #include "base/logging.h"
@@ skipping 21 matching lines @@
 class FocusTraversable;
 class KeyEvent;
 class LayoutManager;
 class MouseEvent;
 class MouseWheelEvent;
 class OSExchangeData;
 class ThemeProvider;
 class Widget;
 
 ////////////////////////////////////////////////////////////////////////////////
-// View class
+// View
 //
 //  View encapsulates rendering, layout and event handling for rectangles within
 //  a view hierarchy.
 //
 //  Client code typically subclasses View and overrides virtual methods to
 //  handle painting, child view positioning and handle certain types of events.
 //
 //  Views are owned by their parent View unless specified otherwise. This means
 //  that in most cases Views are automatically destroyed when the window that
 //  contains them is destroyed.
 //
 // TODO(beng): consider the visibility of many of these methods.
 //             consider making RootView a friend and making many private or
 //             protected.
+
 class View {
  public:
-  typedef std::vector<View*> ViewVector;
+  typedef std::vector<View*> Views;
 
   // Creation and lifetime -----------------------------------------------------
   View();
   virtual ~View();
 
   // By default a View is owned by its parent unless specified otherwise here.
   bool parent_owned() const { return parent_owned_; }
   void set_parent_owned(bool parent_owned) { parent_owned_ = parent_owned; }
 
   void set_drag_controller(DragController* drag_controller) {
     drag_controller_ = drag_controller;
   }
 
   // Size and disposition ------------------------------------------------------
 
-  void SetBounds(int x, int y, int width, int height);
-  void SetBoundsRect(const gfx::Rect& bounds);
-  void SetSize(const gfx::Size& size);
-  void SetPosition(const gfx::Point& position);
   gfx::Rect bounds() const { return bounds_; }
+  // TODO(pkasting): How is this different than bounds()?

[Ben Goodger (Google), 2011/02/18 19:02:28]

This returns the visible bounds of the view in the root view. It is only used
for clipping native views. Think of a native button scrolled out of view in a
scrollview. In that case this would return empty rect. I am still figuring out
where this should live.

[Peter Kasting, 2011/02/18 19:09:43]

I updated the comment to try and capture that.

   gfx::Rect GetVisibleBounds() const;
-  int x() const { return bounds_.x(); }
-  int y() const { return bounds_.y(); }
-  int width() const { return bounds_.width(); }
-  int height() const { return bounds_.height(); }
-
-  // Owned by the view.
-  void SetBorder(Border* border);
-  Border* border() { return border_.get(); }
-
   // Returns the bounds of the content area of the view, i.e. the rectangle
   // enclosed by the view's border.
   gfx::Rect GetContentsBounds() const;
+  void SetBounds(const gfx::Rect& bounds);
+
+  gfx::Point origin() const { return bounds().origin(); }
+  int x() const { return bounds().x(); }
+  int y() const { return bounds().y(); }
+  void SetOrigin(const gfx::Point& origin);
+
+  gfx::Size size() const { return bounds().size(); }
+  int width() const { return bounds().width(); }
+  int height() const { return bounds().height(); }
+  void SetSize(const gfx::Size& size);
+
+  // Owned by the view.
+  Border* border() { return border_.get(); }
+  void SetBorder(Border* border);
 
   // Override to be notified when the bounds of a view have changed.
   virtual void OnBoundsChanged();
 
   virtual gfx::Size GetPreferredSize() const;
   virtual gfx::Size GetMinimumSize() const;
 
   void SetLayoutManager(LayoutManager* layout_manager);
   virtual void Layout();
 
   // If a View is not visible, it will not be rendered, focused, etc.
   bool visible() const { return visible_; }
   void SetVisible(bool visible);
 
   // Disabled Views will not receive mouse press/release events, nor can they be
   // focused.
   bool enabled() const { return enabled_; }
   void SetEnabled(bool enabled);
 
   // Attributes ----------------------------------------------------------------
 
   int id() const { return id_; }
   void set_id(int id) { id_ = id; }
   int group() const { return group_; }
   void set_group(int group) { group_ = group; }
 
   // Returns the View within this View's hierarchy whose id matches that
   // specified.
-  View* GetViewById(int id) const;
+  View* GetViewByID(int id);
 
-  // Populates a ViewVector with the Views within this View's hierarchy that
-  // match the specified group id.
-  void GetViewsWithGroup(int group, ViewVector* vec) const;
+  // Populates |vec| with the Views within this View's hierarchy that match the
+  // specified group id.
+  void GetViewsInGroup(int group, Views* vec);
 
   // TODO(beng): implementme
   virtual View* GetSelectedViewForGroup(int group_id);
 
   // Coordinate conversion -----------------------------------------------------
 
   // Converts a point from the coordinate system of |source| to |target|.
-  static void ConvertPointToView(View* source, View* target, gfx::Point* point);
+  static void ConvertPointToView(const View& source,
+                                 const View& target,
+                                 gfx::Point* point);
 
   // Converts a point from the coordinate system of |source| to the screen.
   // If |source| is not attached to a Widget that is in screen space, |point| is
   // not modified.
-  static void ConvertPointToScreen(View* source, gfx::Point* point);
+  static void ConvertPointToScreen(const View& source, gfx::Point* point);
 
   // Converts a point from the coordinate system of |source| to the Widget that
   // most closely contains it.
-  static void ConvertPointToWidget(View* source, gfx::Point* point);
+  static void ConvertPointToWidget(const View& source, gfx::Point* point);
 
   // Tree operations -----------------------------------------------------------
 
   // Returns the Widget that contains this View, or NULL if it is not contained
   // within a Widget.
-  virtual Widget* GetWidget() const;
+  Widget* GetWidget() {
+    return const_cast<Widget*>(static_cast<const View*>(this)->GetWidget());
+  }
+  virtual const Widget* GetWidget() const;
 
   // Adds a View as a child of this one, optionally at |index|.
   void AddChildView(View* view);
   void AddChildViewAt(View* view, size_t index);
 
-  // Removes a View as a child of this View. Does not delete the child.
-  View* RemoveChildView(View* view);
+  // If |view| is a child of this View, removes it and optionally deletes it.
+  void RemoveChildView(View* view, bool delete_child);
 
   // Removes all View children of this View. Deletes the children if
   // |delete_children| is true.
   void RemoveAllChildViews(bool delete_children);
 
-  // Returns the View at the specified |index|.
-  View* GetChildViewAt(size_t index);
-
-  // Returns the number of child views.
-  size_t child_count() const { return children_.size(); }
+  // STL-style accessors.
+  Views::const_iterator children_begin() { return children_.begin(); }
+  Views::const_iterator children_end() { return children_.end(); }
+  Views::const_reverse_iterator children_rbegin() { return children_.rbegin(); }
+  Views::const_reverse_iterator children_rend() { return children_.rend(); }
+  size_t children_size() const { return children_.size(); }
+  bool children_empty() const { return children_.empty(); }
+  View* child_at(size_t index) {
+    DCHECK_LT(index, children_size());
+    return children_[index];
+  }
 
   // Returns the parent View, or NULL if this View has no parent.
-  View* parent() const { return parent_; }
+  View* parent() { return parent_; }
+  const View* parent() const { return parent_; }
 
   // Returns true if |child| is contained within this View's hierarchy, even as
   // an indirect descendant. Will return true if child is also this View.
-  bool Contains(View* child);
+  bool Contains(const View& child) const;
 
   // Painting ------------------------------------------------------------------
 
   // Add all or part of a View's bounds to the enclosing Widget's invalid
   // rectangle. This will result in those areas being re-painted on the next
   // update.
   void Invalidate();
   virtual void InvalidateRect(const gfx::Rect& invalid_rect);
 
   // Input ---------------------------------------------------------------------
 
   // Returns true if the specified point is contained within this View or its
   // hit test mask. |point| is in this View's coordinates.
   bool HitTest(const gfx::Point& point) const;
 
   // Accelerators --------------------------------------------------------------
 
   // Accelerator Registration.
   void AddAccelerator(const Accelerator& accelerator);
   void RemoveAccelerator(const Accelerator& accelerator);
   void RemoveAllAccelerators();
 
   // Focus ---------------------------------------------------------------------
 
   // Manager.
-  FocusManager* GetFocusManager() const;
+  FocusManager* GetFocusManager();
+  const FocusManager* GetFocusManager() const;
 
   // Traversal.
-  virtual FocusTraversable* GetFocusTraversable() const;
-  View* GetNextFocusableView() const;
-  View* GetPreviousFocusableView() const;
+  virtual FocusTraversable* GetFocusTraversable();
+  View* GetNextFocusableView();
+  View* GetPreviousFocusableView();
 
   // Attributes.
+  bool is_focusable() const { return focusable_ && enabled_ && visible_; }

[Ben Goodger (Google), 2011/02/18 19:02:28]

This actually needs to be more complex in reality. I am going to combine it with
IsFocusableInRootView() in the real code.

[Peter Kasting, 2011/02/18 19:09:43]

Changed back to a non-inline function.

   void set_focusable(bool focusable) { focusable_ = focusable; }
-  bool IsFocusable() const;
 
   bool HasFocus() const;
   void RequestFocus();
 
   // Context menus -------------------------------------------------------------
 
   void set_context_menu_controller(
       ContextMenuController* context_menu_controller) {
     context_menu_controller_ = context_menu_controller;
   }
 
   // Resources -----------------------------------------------------------------
 
-  ThemeProvider* GetThemeProvider() const;
+  ThemeProvider* GetThemeProvider();
 
  protected:
   // Tree operations -----------------------------------------------------------
 
-  // Called on every view in the hierarchy when a view is added or removed.
-  virtual void OnViewAdded(View* parent, View* child);
-  virtual void OnViewRemoved(View* parent, View* child);
+  // Called on every view in |parent|'s and |child|'s hierarchies, as well as
+  // ancestors, when |child| is added to or removed from |parent|.
+  virtual void OnViewAdded(const View& parent, const View& child);
+  virtual void OnViewRemoved(const View& parent, const View& child);
 
-  // Called on a View when it is added or removed from a Widget.
+  // Called on a View when it is part of a hierarchy that has been added to or
+  // removed from a Widget.
   virtual void OnViewAddedToWidget();
   virtual void OnViewRemovedFromWidget();
 
   // Painting ------------------------------------------------------------------
 
   // Responsible for calling Paint() on child Views. Override to control the
   // order child Views are painted.
   virtual void PaintChildren(gfx::Canvas* canvas);
 
   // Override to provide rendering in any part of the View's bounds. Typically
@@ skipping 14 matching lines @@
   virtual void OnPaintFocusBorder(gfx::Canvas* canvas);
 
   // Input ---------------------------------------------------------------------
 
   // Returns the visible View that would like to handle events occurring at the
   // specified |point|, in this View's coordinates.
   // This function is used by the event processing system in the Widget to
   // locate views for event targeting. Override this function if you wish to
   // specify a view other than the one most closely enclosing |point| to receive
   // notifications for events within it.
-  virtual View* GetEventHandlerForPoint(const gfx::Point& point) const;
+  virtual View* GetEventHandlerForPoint(const gfx::Point& point);
 
-  virtual gfx::NativeCursor GetCursorForPoint(const gfx::Point& point);
+  virtual gfx::NativeCursor GetCursorForPoint(const gfx::Point& point) const;
 
   virtual bool OnKeyPressed(const KeyEvent& event);
   virtual bool OnKeyReleased(const KeyEvent& event);
   virtual bool OnMouseWheel(const MouseWheelEvent& event);
   // To receive OnMouseDragged() or OnMouseReleased() events, overriding classes
   // must return true from this function.
   virtual bool OnMousePressed(const MouseEvent& event);
   virtual bool OnMouseDragged(const MouseEvent& event);
   virtual void OnMouseReleased(const MouseEvent& event);
   virtual void OnMouseCaptureLost();
   virtual void OnMouseMoved(const MouseEvent& event);
   virtual void OnMouseEntered(const MouseEvent& event);
   virtual void OnMouseExited(const MouseEvent& event);
 
   // Accelerators --------------------------------------------------------------
 
   virtual bool OnAcceleratorPressed(const Accelerator& accelerator);
 
   // Focus ---------------------------------------------------------------------
 
   virtual bool SkipDefaultKeyEventProcessing(const KeyEvent& event) const;
   virtual bool IsGroupFocusTraversable() const;
   // TODO(beng): kill these, move to focus manager.
   virtual bool IsFocusableInRootView() const;
   virtual bool IsAccessibilityFocusableInRootView() const;
-  virtual FocusTraversable* GetPaneFocusTraversable() const;
+  virtual FocusTraversable* GetPaneFocusTraversable();
 
   virtual void OnFocus(const FocusEvent& event);
   virtual void OnBlur(const FocusEvent& event);
 
  private:
   friend internal::RootView;
   friend class FocusManager;
   friend class FocusSearch;
 
   // State collected during a MousePressed event to detect possible drag
   // operations.
   struct DragInfo {
     // Sets possible_drag to false and start_x/y to 0. This is invoked by
     // RootView prior to invoke MousePressed().
     void Reset();
 
     // Sets possible_drag to true and start_pt to the specified point.
     // This is invoked by the target view if it detects the press may generate
     // a drag.
     void PossibleDrag(const gfx::Point& point);
 
     // Whether the press may generate a drag.
     bool possible_drag;
 
     // Position of the mouse press in screen coordinates.
     gfx::Point press_point;
   };
 
   // Tree operations -----------------------------------------------------------
-  void NotifyHierarchyChanged(View* parent, View* child, bool is_add);
-  void NotifyHierarchyChangedUp(View* parent, View* child, bool is_add);
-  void NotifyHierarchyChangedDown(View* parent, View* child, bool is_add,
+
+  // Notifies all views in our and |child|'s hierarchies, as well as ancestors,
+  // that |child| was added to or removed from |this|.
+  void NotifyHierarchyChanged(View* child, bool is_add);
+
+  // Notifies all views in our hierarchy that |child| was added to or removed
+  // from |this|.  |has_widget| is true iff |this| is in a widget.

[Ben Goodger (Google), 2011/02/18 19:02:28]

if

[Peter Kasting, 2011/02/18 19:09:43]

No, I really meant iff ("if and only if").

+  void NotifyHierarchyChangedDown(const View& child,
+                                  bool is_add,
                                   bool has_widget);
+
+  // Notifies |target| that |child| was added to or removed from |this|.
+  // |has_widget| is true iff |this| is in a widget.
   void CallViewNotification(View* target,
-                            View* parent,
-                            View* child,
+                            const View& child,
                             bool is_add,
                             bool has_widget);
 
   // Painting ------------------------------------------------------------------
 
   // Called by the framework to paint a View. Performs translation and clipping
   // for View coordinates and language direction as required, allows the View
   // to paint itself via the various OnPaint*() event handlers and then paints
   // the hierarchy beneath it.
   void Paint(gfx::Canvas* canvas);
 
-  // Input --------------------------------------------------------------
+  // Input ---------------------------------------------------------------------
+
   //  These methods are designed to be called by the RootView. The RootView
   //  should limit its interaction with the View to these methods and the public
   //  API.
   bool MousePressed(const MouseEvent& event, DragInfo* drag_info);
   bool MouseDragged(const MouseEvent& event, DragInfo* drag_info);
   void MouseReleased(const MouseEvent& event);
 
   // Focus ---------------------------------------------------------------------
 
   // Called when |child| is inserted into this View's children_ at |index|.
   // Sets up next/previous focus views
   // TODO(beng): Move this to FocusManager.
   void InitFocusSiblings(View* child, size_t index);
 
   // Drag & Drop ---------------------------------------------------------------
+
   int GetDragOperations(const gfx::Point& point);
   void WriteDragData(const gfx::Point& point, OSExchangeData* data);
   void StartShellDrag(const MouseEvent& event, const gfx::Point& press_point);
 
   //////////////////////////////////////////////////////////////////////////////
 
   // Creation and lifetime -----------------------------------------------------
 
   // True if the hierarchy (i.e. the parent View) is responsible for deleting
   // this View. Default is true.
@@ skipping 25 matching lines @@
   // An identifier for a group of potentially related Views.
   int group_;
 
   // Tree operations -----------------------------------------------------------
 
   // The View's parent view. This is set and reset when the View is added and
   // removed from a hierarchy.
   View* parent_;
 
   // The View's children.
-  ViewVector children_;
+  Views children_;
 
   // Focus ---------------------------------------------------------------------
 
   // True if this View is focusable by the FocusManager.
   bool focusable_;
 
   // Focus siblings for this View.
   View* next_focusable_view_;
   View* prev_focusable_view_;
 
@@ skipping 24 matching lines @@
 - rtl
 - l10n
 - mousewheel
 - more on painting
 - layer stuff
 - investigate why assorted notifications are necessary
 - native_widget_views
 - native_widget_gtk
 
 */