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

sky/engine/core/loader/EmptyClients.h

 /*
  * Copyright (C) 2006 Eric Seidel (eric@webkit.org)
  * Copyright (C) 2008, 2009, 2010, 2011, 2012 Apple Inc. All rights reserved.
  * Copyright (C) 2010 Nokia Corporation and/or its subsidiary(-ies).
  * Copyright (C) 2012 Samsung Electronics. 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
@@ skipping 112 matching lines @@
     virtual void dispatchDidReceiveTitle(const String&) override { }
     virtual void dispatchDidFailLoad(const ResourceError&) override { }
 
     virtual NavigationPolicy decidePolicyForNavigation(const ResourceRequest&, Document*, NavigationPolicy, bool isTransitionNavigation) override;
 
     virtual void didStartLoading(LoadStartType) override { }
     virtual void progressEstimateChanged(double) override { }
     virtual void didStopLoading() override { }
 
     virtual void loadURLExternally(const ResourceRequest&, NavigationPolicy, const String& = String()) override { }
-    virtual void createView(const KURL&) override {}
+    virtual uint32_t createChildFrame(const KURL&) override { return static_cast<uint32_t>(-1); }

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

-1 is a strange uint32_t.  Maybe std::numeric_limits<uint32_t>::max() ?

Actually, I think 0 is a reserved view ID for this purpose.

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

view.cc initializes id_ to -1 when it's not defined. But I'm using objects now
instead.

+    virtual void initializeChildFrame(uint32_t, const IntRect&) override { }
 
     virtual void transitionToCommittedForNewPage() override { }
 
     virtual void selectorMatchChanged(const Vector<String>&, const Vector<String>&) override { }
 
     virtual void documentElementAvailable() override { }
 
     virtual void didCreateScriptContext(v8::Handle<v8::Context>, int extensionGroup, int worldId) override { }
     virtual void willReleaseScriptContext(v8::Handle<v8::Context>, int worldId) override { }
 };
@@ skipping 42 matching lines @@
     virtual bool canPaste(LocalFrame*, bool defaultValue) const override { return defaultValue; }
 
     virtual bool handleKeyboardEvent() override { return false; }
 };
 
 void fillWithEmptyClients(Page::PageClients&);
 
 }
 
 #endif // EmptyClients_h