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

Unified Diff: ui/views/view.cc

Issue 239093007: Update Windows UI on system color changes. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Sync and rebase. Created 6 years, 8 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
Index: ui/views/view.cc
diff --git a/ui/views/view.cc b/ui/views/view.cc
index 43b7ba964d338d5c9cf24776f1630734f78ed3ee..dcc4869e13f28988dc03f13e8989985cf4a9ce3b 100644
--- a/ui/views/view.cc
+++ b/ui/views/view.cc
@@ -237,7 +237,7 @@ void View::AddChildViewAt(View* view, int index) {
RegisterChildrenForVisibleBoundsNotification(view);
const ui::NativeTheme* new_theme = widget->GetNativeTheme();
if (new_theme != old_theme)
- PropagateNativeThemeChanged(new_theme);
+ view->PropagateNativeThemeChanged(new_theme);
if (view->visible())
view->SchedulePaint();
}
@@ -1616,6 +1616,14 @@ bool View::InDrag() {
return widget ? widget->dragged_view() == this : false;
}
+// NativeTheme -----------------------------------------------------------------
+
+void View::PropagateNativeThemeChanged(const ui::NativeTheme* theme) {
+ for (int i = 0, count = child_count(); i < count; ++i)
+ child_at(i)->PropagateNativeThemeChanged(theme);
+ OnNativeThemeChanged(theme);
+}
+
// Debugging -------------------------------------------------------------------
#if !defined(NDEBUG)
@@ -1880,12 +1888,6 @@ void View::ViewHierarchyChangedImpl(
details.parent->needs_layout_ = true;
}
-void View::PropagateNativeThemeChanged(const ui::NativeTheme* theme) {
- for (int i = 0, count = child_count(); i < count; ++i)
- child_at(i)->PropagateNativeThemeChanged(theme);
- OnNativeThemeChanged(theme);
-}
-
// Size and disposition --------------------------------------------------------
void View::PropagateVisibilityNotifications(View* start, bool is_visible) {

Powered by Google App Engine
This is Rietveld 408576698