Navigation transitions: Initiate navigation transition and send serialized markup to Chrome

public/web/WebFrameClient.h

Index: public/web/WebFrameClient.h
diff --git a/public/web/WebFrameClient.h b/public/web/WebFrameClient.h
index ea97506265bad7b436a2303c62dd1747269e825e..d4e11e36ac902a87d0f0e78a0c8929ce97bcd0e4 100644
--- a/public/web/WebFrameClient.h
+++ b/public/web/WebFrameClient.h
@@ -180,6 +180,14 @@ public:
     // The client may choose to alter the navigation policy.  Otherwise,
     // defaultPolicy should just be returned.
     virtual WebNavigationPolicy decidePolicyForNavigation(
+        WebLocalFrame* frame, WebDataSource::ExtraData* extraData, const WebURLRequest& urlRequest, WebNavigationType navigationType,
+        WebNavigationPolicy defaultPolicy, bool isRedirect, bool isTransitionNavigation)
+    {
+        return decidePolicyForNavigation(frame, extraData, urlRequest, navigationType, defaultPolicy, isRedirect);
+    }
+
+    // DEPRECATED
+    virtual WebNavigationPolicy decidePolicyForNavigation(
         WebLocalFrame*, WebDataSource::ExtraData*, const WebURLRequest&, WebNavigationType,
         WebNavigationPolicy defaultPolicy, bool isRedirect) { return defaultPolicy; }

[abarth-chromium, 2014/06/13 17:18:31]

This sort of change is begging for a struct instead of a long list of changing
parameters.  Using a struct will also make it easier to add and remove
parameters in the future.

[oystein (OOO til 10th of July), 2014/06/18 00:04:09]

Done! I made the URL the only constructor parameter; seemed to make the most
semantic sense, but I can add all of them if need be.

 
@@ -211,6 +219,8 @@ public:
     virtual void didCreateDataSource(WebLocalFrame*, WebDataSource*) { }
 
     // A new provisional load has been started.
+    virtual void didStartProvisionalLoad(WebLocalFrame* localFrame, bool isTransitionNavigation) { didStartProvisionalLoad(localFrame); }
+    // DEPRECATED
     virtual void didStartProvisionalLoad(WebLocalFrame*) { }
 
     // The provisional load was redirected via a HTTP 3xx response.