[DevTools] Make DevToolsSession own it's DevToolsMessageChunkProcessor

content/browser/devtools/render_frame_devtools_agent_host.h

Index: content/browser/devtools/render_frame_devtools_agent_host.h
diff --git a/content/browser/devtools/render_frame_devtools_agent_host.h b/content/browser/devtools/render_frame_devtools_agent_host.h
index 675990f59986d0957e2836d8e8b8f33669b3b943..3d8b3c397691d39da628f6bfb1625044827bf458 100644
--- a/content/browser/devtools/render_frame_devtools_agent_host.h
+++ b/content/browser/devtools/render_frame_devtools_agent_host.h
@@ -171,22 +171,18 @@ class CONTENT_EXPORT RenderFrameDevToolsAgentHost
 
   // The active host we are talking to.
   RenderFrameHostImpl* frame_host_ = nullptr;
+  std::unordered_set<NavigationHandleImpl*> navigation_handles_;

[caseq, 2017/06/13 19:52:26]

flat_set? ;)

[dgozman, 2017/06/16 17:58:46]

Done.

+  bool render_frame_alive_ = false;
+
+  // These messages were queued after suspending, not sent to the agent,
+  // and will be sent after resuming.
   struct Message {
     int session_id;
+    int call_id;
     std::string method;
     std::string message;
   };
-  // Chunk processor's state cookie always corresponds to a state before
-  // any of the suspended or waiting for response messages have been handled.
-  DevToolsMessageChunkProcessor chunk_processor_;
-  std::unordered_set<NavigationHandleImpl*> navigation_handles_;
-  bool render_frame_alive_ = false;
-  // These messages were sent before suspending, but their result have not been
-  // received yet, and state cookie has not been updated.
-  std::map<int, Message> waiting_for_response_messages_;
-  // These messages were queued after suspending, not sent to the agent,
-  // and will be sent after resuming.
-  std::map<int, Message> suspended_messages_;
+  std::vector<Message> suspended_messages_;
 
   // The FrameTreeNode associated with this agent.
   FrameTreeNode* frame_tree_node_;