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

Side by Side Diff: ui/views/widget/widget.cc

Issue 2522093002: Notify WidgetRemovalsObserver when RootView is removed (Closed)
Patch Set: Add test Created 4 years, 1 month 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 unified diff | Download patch
« no previous file with comments | « no previous file | ui/views/widget/widget_unittest.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "ui/views/widget/widget.h" 5 #include "ui/views/widget/widget.h"
6 6
7 #include "base/logging.h" 7 #include "base/logging.h"
8 #include "base/macros.h" 8 #include "base/macros.h"
9 #include "base/message_loop/message_loop.h" 9 #include "base/message_loop/message_loop.h"
10 #include "base/strings/utf_string_conversions.h" 10 #include "base/strings/utf_string_conversions.h"
(...skipping 1353 matching lines...) Expand 10 before | Expand all | Expand 10 after
1364 } 1364 }
1365 1365
1366 //////////////////////////////////////////////////////////////////////////////// 1366 ////////////////////////////////////////////////////////////////////////////////
1367 // Widget, protected: 1367 // Widget, protected:
1368 1368
1369 internal::RootView* Widget::CreateRootView() { 1369 internal::RootView* Widget::CreateRootView() {
1370 return new internal::RootView(this); 1370 return new internal::RootView(this);
1371 } 1371 }
1372 1372
1373 void Widget::DestroyRootView() { 1373 void Widget::DestroyRootView() {
1374 NotifyWillRemoveView(root_view_.get());
1374 non_client_view_ = NULL; 1375 non_client_view_ = NULL;
1375 root_view_.reset(); 1376 root_view_.reset();
1376 } 1377 }
1377 1378
1378 void Widget::OnDragWillStart() {} 1379 void Widget::OnDragWillStart() {}
1379 1380
1380 void Widget::OnDragComplete() {} 1381 void Widget::OnDragComplete() {}
1381 1382
1382 //////////////////////////////////////////////////////////////////////////////// 1383 ////////////////////////////////////////////////////////////////////////////////
1383 // Widget, private: 1384 // Widget, private:
(...skipping 109 matching lines...) Expand 10 before | Expand all | Expand 10 after
1493 1494
1494 //////////////////////////////////////////////////////////////////////////////// 1495 ////////////////////////////////////////////////////////////////////////////////
1495 // internal::NativeWidgetPrivate, NativeWidget implementation: 1496 // internal::NativeWidgetPrivate, NativeWidget implementation:
1496 1497
1497 internal::NativeWidgetPrivate* NativeWidgetPrivate::AsNativeWidgetPrivate() { 1498 internal::NativeWidgetPrivate* NativeWidgetPrivate::AsNativeWidgetPrivate() {
1498 return this; 1499 return this;
1499 } 1500 }
1500 1501
1501 } // namespace internal 1502 } // namespace internal
1502 } // namespace views 1503 } // namespace views
OLDNEW
« no previous file with comments | « no previous file | ui/views/widget/widget_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698