Add a separate call to initialize the bounds for a sky <iframe> element.

sky/engine/web/FrameLoaderClientImpl.cpp

 /*
  * Copyright (C) 2009, 2012 Google Inc. All rights reserved.
  * Copyright (C) 2011 Apple 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.
@@ skipping 233 matching lines @@
 void FrameLoaderClientImpl::loadURLExternally(const ResourceRequest& request, NavigationPolicy policy, const String& suggestedName)
 {
     if (m_webFrame->client()) {
         ASSERT(m_webFrame->frame()->document());
         WrappedResourceRequest webreq(request);
         m_webFrame->client()->loadURLExternally(
             m_webFrame, webreq, static_cast<WebNavigationPolicy>(policy), suggestedName);
     }
 }
 
-void FrameLoaderClientImpl::createView(const KURL& url)
+uint32_t FrameLoaderClientImpl::createChildFrame(const KURL& url)
 {
     if (m_webFrame->client()) {
-        m_webFrame->client()->createChildView(url);
+        return m_webFrame->client()->createChildFrame(url);
+    }
+    return -1;

[abarth-chromium, 2014/11/07 21:25:50]

We shouldn't ever get into this situation.  Maybe ASSERT_NOT_REACHED?

[abarth-chromium, 2014/11/07 21:25:50]

We shouldn't ever get into this situation.  Maybe ASSERT_NOT_REACHED?

[Matt Perry, 2014/11/10 23:00:56]

Done.

+}
+
+void FrameLoaderClientImpl::initializeChildFrame(uint32_t frame_id, const IntRect& bounds)
+{
+    if (m_webFrame->client()) {
+        return m_webFrame->client()->initializeChildFrame(frame_id, bounds);
     }
 }
 
 void FrameLoaderClientImpl::selectorMatchChanged(const Vector<String>& addedSelectors, const Vector<String>& removedSelectors)
 {
     if (WebFrameClient* client = m_webFrame->client())
         client->didMatchCSS(m_webFrame, WebVector<WebString>(addedSelectors), WebVector<WebString>(removedSelectors));
 }
 
 // Called when the FrameLoader goes into a state in which a new page load
 // will occur.
 void FrameLoaderClientImpl::transitionToCommittedForNewPage()
 {
     m_webFrame->createFrameView();
 }
 
 void FrameLoaderClientImpl::didLoseWebGLContext(int arbRobustnessContextLostReason)
 {
     if (m_webFrame->client())
         m_webFrame->client()->didLoseWebGLContext(m_webFrame, arbRobustnessContextLostReason);
 }
 
 void FrameLoaderClientImpl::dispatchDidChangeManifest()
 {
     if (m_webFrame->client())
         m_webFrame->client()->didChangeManifest(m_webFrame);
 }
 
 } // namespace blink