[exp] ash: sysui for mash

components/mus/public/cpp/lib/window_tree_client_impl.cc

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "components/mus/public/cpp/lib/window_tree_client_impl.h"
 
 #include <stddef.h>
 #include <utility>
 
 #include "base/bind.h"
@@ skipping 543 matching lines @@
   WindowPrivate(window).LocalDestroy();
 }
 
 void WindowTreeClientImpl::OnTopLevelCreated(uint32_t change_id,
                                              mojom::WindowDataPtr data) {
   // The server ack'd the top level window we created and supplied the state
   // of the window at the time the server created it. For properties we do not
   // have changes in flight for we can update them immediately. For properties
   // with changes in flight we set the revert value from the server.
 
+  if (!in_flight_map_.count(change_id))  // XXX(sad): ??

[sky, 2016/02/03 22:43:48]

This early out indicates something is wrong. How are you hitting this?

[sadrul, 2016/02/04 00:03:42]

I have no idea why I am hitting this, but I hit this when I start up with the
rest of the patch. I haven't dug into it yet.

+    return;
   scoped_ptr<InFlightChange> change(std::move(in_flight_map_[change_id]));
   in_flight_map_.erase(change_id);
   DCHECK(change);
 
   Window* window = change->window();
   WindowPrivate window_private(window);
 
   // Drawn state and ViewportMetrics always come from the server (they can't
   // be modified locally).
   window_private.set_drawn(data->drawn);
@@ skipping 353 matching lines @@
 
 void WindowTreeClientImpl::SetUnderlaySurfaceOffsetAndExtendedHitArea(
     Window* window,
     const gfx::Vector2d& offset,
     const gfx::Insets& hit_area) {
   window_manager_internal_client_->SetUnderlaySurfaceOffsetAndExtendedHitArea(
       window->id(), offset.x(), offset.y(), mojo::Insets::From(hit_area));
 }
 
 }  // namespace mus