Add WebPrescientNetworking to trigger preconnect from Blink.

Source/Platform/chromium/public/WebPrescientNetworking.h

 /*
- * Copyright (C) 2012 Google Inc. All rights reserved.
+ * Copyright (C) 2013 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:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
  * copyright notice, this list of conditions and the following disclaimer
  * in the documentation and/or other materials provided with the
  * distribution.
  *     * Neither the name of Google Inc. nor the names of its
  * contributors may be used to endorse or promote products derived from
  * this software without specific prior written permission.
  *
  * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
  * "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
  * LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
  * A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
  * OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
  * SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
  * LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
  * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
  * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
  * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
-#ifndef WebPrerenderingSupport_h
-#define WebPrerenderingSupport_h
+#ifndef WebPrescientNetworking_h
+#define WebPrescientNetworking_h
 
 #include "WebCommon.h"
 
 namespace WebKit {
 
-class WebPrerender;
+class WebURL;
 
-class WebPrerenderingSupport {
+// FIXME (kouhei): Passing preconnect motivation to net is layering vioration.

[abarth-chromium, 2013/05/14 06:18:56]

" (kouhei)" <-- you should delete this part.  We don't put user names in FIXME
comments in Blink.

net -> "the platform".  Blink has no knowledge that a "net" module exists.

[kouhei (in TOK), 2013/05/14 06:34:07]

Done.

+// However, this is done temporarily to perform a Finch field trial to enable
+// preconnect in different triggers one at a time. This enum can be safely
+// removed after the field trial is done.
+enum WebPreconnectMotivation {
+    WebPreconnectMotivationLinkMouseDown,
+    WebPreconnectMotivationLinkMouseOver,
+    WebPreconnectMotivationLinkTapUnconfirmed,
+    WebPreconnectMotivationLinkTapDown,
+};
+
+class WebPrescientNetworking {
 public:
-    WEBKIT_EXPORT static void initialize(WebPrerenderingSupport*);
+    WEBKIT_EXPORT static void initialize(WebPrescientNetworking*);
     WEBKIT_EXPORT static void shutdown();
-    WEBKIT_EXPORT static WebPrerenderingSupport* current();
+    WEBKIT_EXPORT static WebPrescientNetworking* current();

[abarth-chromium, 2013/05/14 06:18:56]

None of these methods are needed.  They should all be removed.

Instead, we should get the WebPrescientNetworking instance from
WebKit::Platform::current().

[kouhei (in TOK), 2013/05/14 06:34:07]

Done.

 
-    // A prerender link element is added when it is inserted into a document.
-    virtual void add(const WebPrerender&) = 0;
-
-    // A prerender is canceled when it is removed from a document.
-    virtual void cancel(const WebPrerender&) = 0;
-
-    // A prerender is abandoned when it's navigated away from or suspended in the page cache. This
-    // is a weaker signal than cancel(), since the launcher hasn't indicated that the prerender isn't
-    // wanted, and we may end up using it after, for instance, a short redirect chain.
-    virtual void abandon(const WebPrerender&) = 0;
+    // When a page navigation is speculated, preconnect is triggered to hide
+    // session initialization latency to the server providing the page resource.
+    virtual void preconnect(const WebURL&, WebPreconnectMotivation) { }
 
 protected:
-    WebPrerenderingSupport() { }
-    virtual ~WebPrerenderingSupport() { }
+    WebPrescientNetworking() { }

[abarth-chromium, 2013/05/14 06:18:56]

No constructor is needed.

[kouhei (in TOK), 2013/05/14 06:34:07]

Done.

+    virtual ~WebPrescientNetworking() { }
 
 private:
-    static WebPrerenderingSupport* s_platform;
+    static WebPrescientNetworking* s_platform;

[abarth-chromium, 2013/05/14 06:18:56]

Please remove the static state from this class.  Instead, this object should be
retrieved from the WebKit::Platform::current static.

[kouhei (in TOK), 2013/05/14 06:34:07]

Done.

 };
 
 } // namespace WebKit
 
-#endif // WebPrerenderingSupport_h
+#endif // WebPrescientNetworking_h