Remove includes of message headers in headers.

chrome/browser/debugger/devtools_manager.h

Index: chrome/browser/debugger/devtools_manager.h
===================================================================
--- chrome/browser/debugger/devtools_manager.h	(revision 73994)
+++ chrome/browser/debugger/devtools_manager.h	(working copy)
@@ -12,7 +12,6 @@
 #include "base/ref_counted.h"
 #include "chrome/browser/debugger/devtools_client_host.h"
 #include "chrome/browser/debugger/devtools_toggle_action.h"
-#include "chrome/common/devtools_messages.h"
 #include "webkit/glue/resource_loader_bridge.h"
 
 namespace IPC {
@@ -28,6 +27,8 @@
 
 using webkit_glue::ResourceLoaderBridge;
 
+typedef std::map<std::string, std::string> DevToolsRuntimeProperties;

[Tom Sepez, 2011/02/08 21:21:21]

Too bad we can't get this from a header; if the implementation of this changes
down the road, this will break.

[jam, 2011/02/08 21:38:39]

I agree, this is why it's nice to avoid typedefs in the message headers.  I
didn't see any headers for automation related structs, so that's why I copied
the typedef

+
 // This class is a singleton that manages DevToolsClientHost instances and
 // routes messages between developer tools clients and agents.
 class DevToolsManager : public DevToolsClientHost::CloseListener,