Make the embedder responsible for creating the WebFrame

public/web/WebFrame.h

Index: public/web/WebFrame.h
diff --git a/public/web/WebFrame.h b/public/web/WebFrame.h
index 211def68aed06bc968232765d3d6b6d4962f7ed1..7acf490dc67dfcf7a1a6a8e9ed42c6957292b38f 100644
--- a/public/web/WebFrame.h
+++ b/public/web/WebFrame.h
@@ -59,6 +59,7 @@ class WebDataSource;
 class WebDocument;
 class WebElement;
 class WebFormElement;
+class WebFrameClient;
 class WebHistoryItem;
 class WebInputElement;
 class WebPerformance;
@@ -92,6 +93,23 @@ public:
     };
     typedef unsigned RenderAsTextControls;
 
+    // Creates a WebFrame. Delete this WebFrame by calling WebFrame::close().
+    // It is valid to pass a null client pointer.
+    WEBKIT_EXPORT static WebFrame* create(WebFrameClient*);
+
+    // Same as create(WebFrameClient*) except the embedder may explicitly pass
+    // in the identifier for the WebFrame. This can be used with
+    // generateEmbedderIdentifier() if constructing the WebFrameClient for this
+    // frame requires the identifier.
+    //
+    // FIXME: Move the embedderIdentifier concept fully to the embedder and
+    // remove this factory method.
+    WEBKIT_EXPORT static WebFrame* create(WebFrameClient*, long long embedderIdentifier);
+
+    // Generates an identifier suitable for use with create() above.
+    // Never returns -1.
+    WEBKIT_EXPORT static long long generateEmbedderIdentifier();
+
     // Returns the number of live WebFrame objects, used for leak checking.
     WEBKIT_EXPORT static int instanceCount();
 
@@ -109,6 +127,9 @@ public:
     // the given element is not a frame, iframe or if the frame is empty.
     WEBKIT_EXPORT static WebFrame* fromFrameOwnerElement(const WebElement&);
 
+    // This method closes and deletes the WebFrame.
+    virtual void close() = 0;
+
 
     // Basic properties ---------------------------------------------------
 
@@ -124,7 +145,9 @@ public:
     virtual void setName(const WebString&) = 0;
 
     // A globally unique identifier for this frame.
-    virtual long long identifier() const = 0;
+    // FIXME: Convert users to embedderIdentifier() and remove identifier().
+    long long identifier() const { return embedderIdentifier(); }
+    virtual long long embedderIdentifier() const = 0;
 
     // The urls of the given combination types of favicon (if any) specified by
     // the document loaded in this frame. The iconTypesMask is a bit-mask of