Push updates to inspector when windows are added, destroyed or reorganized

ash/common/devtools/ash_devtools_unittest.cc

Index: ash/common/devtools/ash_devtools_unittest.cc
diff --git a/ash/common/devtools/ash_devtools_unittest.cc b/ash/common/devtools/ash_devtools_unittest.cc
index 65d77bec087f488460e1238fd6429b9a3ef12c1b..4bdcb034d756fc6d0cd081d6054616e722f78b93 100644
--- a/ash/common/devtools/ash_devtools_unittest.cc
+++ b/ash/common/devtools/ash_devtools_unittest.cc
@@ -146,6 +146,8 @@ TEST_F(AshDevToolsTest, GetDocumentWithWindowWidgetView) {
   ASSERT_TRUE(widget_children->get(0)->getChildren(default_children));
   Compare(child_view,
           widget_children->get(0)->getChildren(default_children)->get(1));
+  // TODO(mhashmi): Remove this call and mock FrontendChannel
+  dom_agent()->disable();

[sadrul, 2016/11/04 15:39:17]

When |widget| is destroyed, it should clean up the state from the appropriate
callbacks, right? Why is it necessary to explicitly disable() here?

[Sarmad Hashmi, 2016/11/04 16:03:50]

The tear down of the |windows| on windows trybots causes the test to crash in
the generated files. Specifically, the backtrace crashes inside the Frontend
methods, which are all generated. The crash is very similar to the one we saw
before for the tests. I'll investigate more a bit later, but the tests work if
we disable the agents since the Frontend methods are no longer called on tear
down.

 }
 
 }  // namespace ash