views: Have a more accurate name for View parent_owned accessors.

ui/views/view.h

Index: ui/views/view.h
diff --git a/ui/views/view.h b/ui/views/view.h
index e847abcaec3c9940337774d2bf59d3597130dad2..e66d6dacd09edcdb1a98e8cd0bbeb4cd0e58d390 100644
--- a/ui/views/view.h
+++ b/ui/views/view.h
@@ -118,8 +118,8 @@ class VIEWS_EXPORT View : public ui::LayerDelegate,
   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; }
+  bool owned_by_parent() const { return owned_by_parent_; }

[Ben Goodger (Google), 2012/05/10 16:51:20]

is this accessor used by anyone outside of View? seems like it could be
unnecessary.

+  void set_not_owned_by_parent() { owned_by_parent_ = false; }

[Ben Goodger (Google), 2012/05/10 16:51:20]

this function name feels awkward to me (though so did the old one).

there are really two modes of ownership:

- hierarchy (specifically, the parent views::View)
- client code (code that uses views)

Perhaps this could be set_owned_by_client().

 
   // Tree operations -----------------------------------------------------------
 
@@ -1332,9 +1332,9 @@ class VIEWS_EXPORT View : public ui::LayerDelegate,
 
   // Creation and lifetime -----------------------------------------------------
 
-  // True if the hierarchy (i.e. the parent View) is responsible for deleting
-  // this View. Default is true.
-  bool parent_owned_;
+  // True if this views is owned by its parent - i.e. it will be deleted by its
+  // parent during its parents destruction. True is the default.
+  bool owned_by_parent_;
 
   // Attributes ----------------------------------------------------------------