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

sky/engine/core/loader/FrameLoaderClient.h

 /*
  * Copyright (C) 2006, 2007, 2008, 2009, 2010, 2011, 2012 Apple Inc. All rights reserved.
  * Copyright (C) 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:
  *
  * 1.  Redistributions of source code must retain the above copyright
  *     notice, this list of conditions and the following disclaimer.
@@ skipping 31 matching lines @@
 namespace blink {
 
     class Color;
     class Document;
     class DOMWindowExtension;
     class DOMWrapperWorld;
     class Element;
     class FetchRequest;
     class FrameLoader;
     class FrameNetworkingContext;
+    class IntRect;
     class IntSize;
     class KURL;
     class LocalFrame;
     class Page;
     class ResourceError;
     class ResourceHandle;
     class ResourceRequest;
     class ResourceResponse;
     class SharedBuffer;
     class Widget;
@@ skipping 19 matching lines @@
         virtual void dispatchAddNavigationTransitionData(const String& origin, const String& selector, const String& markup) { }
         virtual void dispatchWillRequestResource(FetchRequest*) { }
 
         virtual void didStartLoading(LoadStartType) = 0;
         virtual void progressEstimateChanged(double progressEstimate) = 0;
         virtual void didStopLoading() = 0;
 
         virtual void loadURLExternally(const ResourceRequest&, NavigationPolicy, const String& suggestedName = String()) = 0;
 
         // TODO(mpcomplete): return surface ID?
-        virtual void createView(const KURL&) = 0;
+        virtual uint32_t createChildFrame(const KURL&) = 0;
+        virtual void initializeChildFrame(uint32_t, const IntRect&) = 0;

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

Seems like createChildFrame should return an object that has the rest of these
functions hung off it.  Otherwise we're going to spam FrameLoaderClient with a
bunch of functions that take a uint32_t as a fake |this| parameter.

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

Done.

 
         // Transmits the change in the set of watched CSS selectors property
         // that match any element on the frame.
         virtual void selectorMatchChanged(const Vector<String>& addedSelectors, const Vector<String>& removedSelectors) = 0;
 
         virtual void transitionToCommittedForNewPage() = 0;
 
 
         virtual void documentElementAvailable() = 0;
 
@@ skipping 10 matching lines @@
         virtual void dispatchDidChangeResourcePriority(unsigned long identifier, ResourceLoadPriority, int intraPriorityValue) { }
 
         virtual void dispatchDidChangeManifest() { }
 
         virtual bool isFrameLoaderClientImpl() const { return false; }
     };
 
 } // namespace blink
 
 #endif // FrameLoaderClient_h