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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: ui/aura/window.cc

Issue 388893003: Fixes for re-enabling more MSVC level 4 warnings: ui/ edition (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 5 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/accessibility/ax_node_data.cc ('k') | ui/events/event_targeter.cc » ('j') | ui/native_theme/native_theme_win.cc » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ui/aura/window.cc
diff --git a/ui/aura/window.cc b/ui/aura/window.cc
index 7eea861e61df66e8aced2740b7f187cef199b1f0..46fae7abcb7eecfea9a2ed3f2c9403f7dc88ae17 100644
--- a/ui/aura/window.cc
+++ b/ui/aura/window.cc
@@ -257,8 +257,8 @@ Window::~Window() {
if (delegate_)
delegate_->OnWindowDestroyed(this);
ObserverListBase<WindowObserver>::Iterator iter(observers_);
- WindowObserver* observer;
- while ((observer = iter.GetNext())) {
+ for (WindowObserver* observer = iter.GetNext(); observer;
+ observer = iter.GetNext()) {
RemoveObserver(observer);
observer->OnWindowDestroyed(this);
}
« no previous file with comments | « ui/accessibility/ax_node_data.cc ('k') | ui/events/event_targeter.cc » ('j') | ui/native_theme/native_theme_win.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698