Pipe the devTools FrameId from blink into the browser for headless

content/browser/frame_host/render_frame_host_impl.h

Index: content/browser/frame_host/render_frame_host_impl.h
diff --git a/content/browser/frame_host/render_frame_host_impl.h b/content/browser/frame_host/render_frame_host_impl.h
index d4415a7ce0d084e2c8679a5b741ae9c1cb77947c..e30705a1e52203623c40bd574b69a5fd4829dc46 100644
--- a/content/browser/frame_host/render_frame_host_impl.h
+++ b/content/browser/frame_host/render_frame_host_impl.h
@@ -627,6 +627,8 @@ class CONTENT_EXPORT RenderFrameHostImpl
     return has_focused_editable_element_;
   }
 
+  const std::string& devtools_frame_id() const { return devtools_frame_id_; }
+
   // Cancels any blocked request for the frame and its subframes.
   void CancelBlockedRequestsForFrame();
 
@@ -781,6 +783,7 @@ class CONTENT_EXPORT RenderFrameHostImpl
   void OnFocusedNodeChanged(bool is_editable_element,
                             const gfx::Rect& bounds_in_frame_widget);
   void OnSetHasReceivedUserGesture();
+  void OnSetDevToolsFrameId(const std::string& devtools_frame_id);
 
 #if BUILDFLAG(USE_EXTERNAL_POPUP_MENU)
   void OnShowPopup(const FrameHostMsg_ShowPopup_Params& params);
@@ -1178,6 +1181,8 @@ class CONTENT_EXPORT RenderFrameHostImpl
   std::unique_ptr<service_manager::InterfaceRegistry> java_interface_registry_;
 #endif
 
+  std::string devtools_frame_id_;

[Charlie Reis, 2017/04/24 20:26:42]

dgozman@: Is this something that's RFH specific, or is it stable for the
lifetime of the FrameTreeNode?  (Is RFH the right place for it?)

[dgozman, 2017/04/25 22:04:48]

This is currently per-{Local,Render}Frame so RFH is a right place.

+
   // NOTE: This must be the last member.
   base::WeakPtrFactory<RenderFrameHostImpl> weak_ptr_factory_;