Move unique name generation and tracking into //content.

third_party/WebKit/public/web/WebFrameClient.h

 /*
  * Copyright (C) 2011, 2012 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 162 matching lines @@
   // pending navigation's URL, because a URL spoof is possible.
   virtual void didAccessInitialDocument() {}
 
   // Request the creation of a new child frame. Embedders may return nullptr
   // to prevent the new child frame from being attached. Otherwise, embedders
   // should create a new WebLocalFrame, insert it into the frame tree, and
   // return the created frame.
   virtual WebLocalFrame* createChildFrame(WebLocalFrame* parent,
                                           WebTreeScopeType,
                                           const WebString& name,
-                                          const WebString& uniqueName,
+                                          const WebString& fallbackName,

[dcheng, 2017/03/01 23:57:26]

Note: I'm 95% sure this isn't actually needed: when creating a child frame, we
already set name to the fallback name. But this is already a pretty large
change, so I'll pursue this in followups.

                                           WebSandboxFlags sandboxFlags,
                                           const WebFrameOwnerProperties&) {
     return nullptr;
   }
 
   // This frame has set its opener to another frame, or disowned the opener
   // if opener is null. See http://html.spec.whatwg.org/#dom-opener.
   virtual void didChangeOpener(WebFrame*) {}
 
   // Specifies the reason for the detachment.
   enum class DetachType { Remove, Swap };
 
   // This frame has been detached. Embedders should release any resources
   // associated with this frame. If the DetachType is Remove, the frame should
   // also be removed from the frame tree; otherwise, if the DetachType is
   // Swap, the frame is being replaced in-place by WebFrame::swap().
   virtual void frameDetached(WebLocalFrame*, DetachType);
 
   // This frame has become focused.
   virtual void frameFocused() {}
 
   // A provisional load is about to commit.
   virtual void willCommitProvisionalLoad(WebLocalFrame*) {}
 
   // This frame's name has changed.
-  virtual void didChangeName(const WebString& name,
-                             const WebString& uniqueName) {}
+  virtual void didChangeName(const WebString& name) {}
 
   // This frame has set an insecure request policy.
   virtual void didEnforceInsecureRequestPolicy(WebInsecureRequestPolicy) {}
 
   // This frame has been updated to a unique origin, which should be
   // considered potentially trustworthy if
   // |isPotentiallyTrustworthyUniqueOrigin| is true. TODO(estark):
   // this method only exists to support dynamic sandboxing via a CSP
   // delivered in a <meta> tag. This is not supposed to be allowed per
   // the CSP spec and should be ripped out. https://crbug.com/594645
@@ skipping 533 matching lines @@
   // Overwrites the given URL to use an HTML5 embed if possible.
   // An empty URL is returned if the URL is not overriden.
   virtual WebURL overrideFlashEmbedWithHTML(const WebURL& url) {
     return WebURL();
   }
 };
 
 }  // namespace blink
 
 #endif