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

Side by Side Diff: components/mus/public/cpp/lib/window_tree_client_impl.cc

Issue 1984293002: Random fixes for mash bridge classes (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: comment Created 4 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 unified diff | Download patch
« no previous file with comments | « no previous file | components/mus/public/cpp/window_tree_connection_observer.h » ('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 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 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 "components/mus/public/cpp/lib/window_tree_client_impl.h" 5 #include "components/mus/public/cpp/lib/window_tree_client_impl.h"
6 6
7 #include <stddef.h> 7 #include <stddef.h>
8 8
9 #include <string> 9 #include <string>
10 #include <utility> 10 #include <utility>
(...skipping 148 matching lines...) Expand 10 before | Expand all | Expand 10 after
159 } 159 }
160 } 160 }
161 161
162 // Delete the non-owned windows last. In the typical case these are roots. The 162 // Delete the non-owned windows last. In the typical case these are roots. The
163 // exception is the window manager and embed roots, which may know about 163 // exception is the window manager and embed roots, which may know about
164 // other random windows that it doesn't own. 164 // other random windows that it doesn't own.
165 // NOTE: we manually delete as we're a friend. 165 // NOTE: we manually delete as we're a friend.
166 while (!tracker.windows().empty()) 166 while (!tracker.windows().empty())
167 delete tracker.windows().front(); 167 delete tracker.windows().front();
168 168
169 FOR_EACH_OBSERVER(WindowTreeConnectionObserver, observers_,
170 OnWillDestroyConnection(this));
171
169 delegate_->OnConnectionLost(this); 172 delegate_->OnConnectionLost(this);
170 } 173 }
171 174
172 void WindowTreeClientImpl::ConnectViaWindowTreeFactory( 175 void WindowTreeClientImpl::ConnectViaWindowTreeFactory(
173 shell::Connector* connector) { 176 shell::Connector* connector) {
174 // Clients created with no root shouldn't delete automatically. 177 // Clients created with no root shouldn't delete automatically.
175 delete_on_no_roots_ = false; 178 delete_on_no_roots_ = false;
176 179
177 // The connection id doesn't really matter, we use 101 purely for debugging. 180 // The connection id doesn't really matter, we use 101 purely for debugging.
178 connection_id_ = 101; 181 connection_id_ = 101;
(...skipping 920 matching lines...) Expand 10 before | Expand all | Expand 10 after
1099 1102
1100 void WindowTreeClientImpl::SetUnderlaySurfaceOffsetAndExtendedHitArea( 1103 void WindowTreeClientImpl::SetUnderlaySurfaceOffsetAndExtendedHitArea(
1101 Window* window, 1104 Window* window,
1102 const gfx::Vector2d& offset, 1105 const gfx::Vector2d& offset,
1103 const gfx::Insets& hit_area) { 1106 const gfx::Insets& hit_area) {
1104 window_manager_internal_client_->SetUnderlaySurfaceOffsetAndExtendedHitArea( 1107 window_manager_internal_client_->SetUnderlaySurfaceOffsetAndExtendedHitArea(
1105 server_id(window), offset.x(), offset.y(), mojo::Insets::From(hit_area)); 1108 server_id(window), offset.x(), offset.y(), mojo::Insets::From(hit_area));
1106 } 1109 }
1107 1110
1108 } // namespace mus 1111 } // namespace mus
OLDNEW
« no previous file with comments | « no previous file | components/mus/public/cpp/window_tree_connection_observer.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698