OOPIF: Use the same uniqueName when swapping between local and remote.

third_party/WebKit/Source/web/WebRemoteFrameImpl.cpp

 // Copyright 2014 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.
 
 #include "config.h"
 #include "web/WebRemoteFrameImpl.h"
 
 #include "core/frame/FrameView.h"
 #include "core/frame/RemoteFrame.h"
 #include "core/frame/Settings.h"
@@ skipping 67 matching lines @@
 {
 #if ENABLE(OILPAN)
     m_selfKeepAlive.clear();
 #else
     deref();
 #endif
 }
 
 WebString WebRemoteFrameImpl::uniqueName() const
 {
-    ASSERT_NOT_REACHED();
-    return WebString();
+    // FIXME: Why aren't these name methods allowed in WebRemoteFrame?
+    ASSERT(frame());
+    return frame()->tree().uniqueName();

[Charlie Reis, 2015/11/06 23:04:48]

I'm not sure why name and uniqueName aren't allowed on WebRemoteFrame.  We
already have setReplicatedName, which kind of gets around these
ASSERT_NOT_REACHEDs.  Names might be something that belong on Frame rather than
LocalFrame?

[dcheng, 2015/11/25 18:14:51]

Actually, let's try to limit the API surface of WebRemoteFrame. Since we're
implementing this in web/, we can just reach directly into blink::Frame::tree()
to get the name and uniqueName.

[Charlie Reis, 2015/11/25 18:22:43]

Ok, will do.

 }
 
 WebString WebRemoteFrameImpl::assignedName() const
 {
     ASSERT_NOT_REACHED();
     return WebString();
 }
 
 void WebRemoteFrameImpl::setName(const WebString&)
 {
@@ skipping 709 matching lines @@
     : WebRemoteFrame(scope)
     , m_frameClient(RemoteFrameClientImpl::create(this))
     , m_client(client)
 #if ENABLE(OILPAN)
     , m_selfKeepAlive(this)
 #endif
 {
 }
 
 } // namespace blink