[ChromeDriver] Send DOM.getDocument after each DOM.documentUpdated.

chrome/test/chromedriver/devtools_state.cc

Index: chrome/test/chromedriver/devtools_state.cc
diff --git a/chrome/test/chromedriver/devtools_state.cc b/chrome/test/chromedriver/devtools_state.cc
new file mode 100644
index 0000000000000000000000000000000000000000..c5752e191e5ba8abbb6b300d584381cdcb9edd90
--- /dev/null
+++ b/chrome/test/chromedriver/devtools_state.cc
@@ -0,0 +1,42 @@
+// Copyright (c) 2012 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 "chrome/test/chromedriver/devtools_state.h"
+
+#include <utility>
+
+#include "base/json/json_writer.h"
+#include "base/logging.h"
+#include "base/values.h"
+#include "chrome/test/chromedriver/devtools_client.h"
+#include "chrome/test/chromedriver/status.h"
+
+DevToolsState::DevToolsState(DevToolsClient* client)
+  : client_(client) {}
+
+DevToolsState::~DevToolsState() {}
+
+Status DevToolsState::Init() {
+  // Perform necessary configuration of the DevTools client.
+  // Fetch the root document node so that Inspector will push DOM node
+  // information to the client.
+  base::DictionaryValue params;
+  Status status = client_->SendCommand("DOM.getDocument", params);
+  if (status.IsError())
+    return status;
+  // Enable page domain notifications to allow tracking navigation state.
+  status = client_->SendCommand("Page.enable", params);

[kkania, 2013/01/30 18:50:29]

So actually, Page.enable and Runtime.enable only need to be done once per
devtools connection. So, we actually don't need to put these here.

I'm not sure it's best to put the DOM.getDocument in here, since it seems like
something the dom tracker should be responsible for.

[kkania, 2013/01/30 20:33:24]

Wait nevermind, this is only done once per connection. So this is ok. However, I
still think it makes more sense to do the DOM.getDocument as part of the DOM
tracker...

[craigdh, 2013/01/30 21:35:08]

Ok, I'll just fold this class's functionality into DomTracker.

+  if (status.IsError())
+    return status;
+  // Enable runtime events to allow tracking execution context creation.
+  return client_->SendCommand("Runtime.enable", params);
+}
+
+void DevToolsState::OnEvent(const std::string& method,
+                         const base::DictionaryValue& params) {
+  if (method == "DOM.documentUpdated") {
+    base::DictionaryValue params;
+    client_->SendCommand("DOM.getDocument", params);
+  }
+}