Replicating WebFrame::uniqueName across renderers.

content/browser/frame_host/render_frame_host_manager.h

 // Copyright 2013 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.
 
 #ifndef CONTENT_BROWSER_FRAME_HOST_RENDER_FRAME_HOST_MANAGER_H_
 #define CONTENT_BROWSER_FRAME_HOST_RENDER_FRAME_HOST_MANAGER_H_
 
 #include <stdint.h>
 
 #include <list>
@@ skipping 416 matching lines @@
   // Clears the speculative members, returning the RenderFrameHost to the caller
   // for disposal.
   scoped_ptr<RenderFrameHostImpl> UnsetSpeculativeRenderFrameHost();
 
   // Notification methods to tell this RenderFrameHostManager that the frame it
   // is responsible for has started or stopped loading a document.
   void OnDidStartLoading();
   void OnDidStopLoading();
 
   // Send updated frame name to all frame proxies when the frame changes its
   // window.name property.

[alexmos, 2016/02/10 00:46:46]

nit: explain why this now also takes unique_name in the comment.

[Łukasz Anforowicz, 2016/02/10 22:10:51]

Done (I rewrote the comment).

-  void OnDidUpdateName(const std::string& name);
+  void OnDidUpdateName(const std::string& name, const std::string& unique_name);
 
   // Sends updated enforcement of strict mixed content checking to all
   // frame proxies when the frame changes its setting.
   void OnEnforceStrictMixedContentChecking(bool should_enforce);
 
   // Send updated origin to all frame proxies when the frame navigates to a new
   // origin.
   void OnDidUpdateOrigin(const url::Origin& origin);
 
   void EnsureRenderViewInitialized(RenderViewHostImpl* render_view_host,
@@ skipping 325 matching lines @@
   scoped_ptr<RenderFrameHostImpl> speculative_render_frame_host_;
 
   base::WeakPtrFactory<RenderFrameHostManager> weak_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(RenderFrameHostManager);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_FRAME_HOST_RENDER_FRAME_HOST_MANAGER_H_