views: Drop the prefix "is_" from View::is_visible_ member variable.
No other boolean variable in the View class has the "is_" prefix and this is
making me nervous for a while now :)
BUG=72040
TEST=None
[email protected]
Review URL: http://codereview.chromium.org/7104098
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@88748 0039d316-1c4b-4281-b951-d872f2087c98
diff --git a/views/view.cc b/views/view.cc
index 807316d6..d33e80b35 100644
--- a/views/view.cc
+++ b/views/view.cc
@@ -99,7 +99,7 @@
id_(0),
group_(-1),
parent_(NULL),
- is_visible_(true),
+ visible_(true),
enabled_(true),
registered_for_visible_bounds_notification_(false),
clip_x_(0.0),
@@ -343,26 +343,26 @@
}
void View::SetVisible(bool visible) {
- if (visible != is_visible_) {
+ if (visible != visible_) {
// If the tab is currently visible, schedule paint to refresh parent.
- if (is_visible_)
+ if (visible_)
SchedulePaint();
else
ResetTexture();
- is_visible_ = visible;
+ visible_ = visible;
// This notifies all sub-views recursively.
- PropagateVisibilityNotifications(this, is_visible_);
+ PropagateVisibilityNotifications(this, visible_);
// If we are newly visible, schedule paint.
- if (is_visible_)
+ if (visible_)
SchedulePaint();
}
}
bool View::IsVisible() const {
- return is_visible_;
+ return visible_;
}
bool View::IsVisibleInRootView() const {
