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

Unified Diff: ui/views/view.h

Issue 10384068: views: Have a more accurate name for View parent_owned accessors. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rebase Created 8 years, 7 months 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
« no previous file with comments | « ui/views/touchui/touch_selection_controller_impl.cc ('k') | ui/views/view.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ui/views/view.h
diff --git a/ui/views/view.h b/ui/views/view.h
index 53ae271f693347617809ece44a30385868a28819..5450f2ea416fc53229ef953ac12d4645eb8d061f 100644
--- a/ui/views/view.h
+++ b/ui/views/view.h
@@ -106,8 +106,7 @@ 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; }
+ void set_owned_by_client() { owned_by_client_ = true; }
// Tree operations -----------------------------------------------------------
@@ -1320,9 +1319,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_;
+ // False if this View is owned by its parent - i.e. it will be deleted by its
+ // parent during its parents destruction. False is the default.
+ bool owned_by_client_;
// Attributes ----------------------------------------------------------------
« no previous file with comments | « ui/views/touchui/touch_selection_controller_impl.cc ('k') | ui/views/view.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698