New link rel=prerender api, using WebKit::WebPrerenderingPlatform

chrome/browser/prerender/prerender_link_manager_factory.cc

Index: chrome/browser/prerender/prerender_link_manager_factory.cc
diff --git a/chrome/browser/prerender/prerender_link_manager_factory.cc b/chrome/browser/prerender/prerender_link_manager_factory.cc
new file mode 100644
index 0000000000000000000000000000000000000000..830cacf5ec47010ecdf35532c8a74b750c94e107
--- /dev/null
+++ b/chrome/browser/prerender/prerender_link_manager_factory.cc
@@ -0,0 +1,48 @@
+// Copyright (c) 2012 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "chrome/browser/prerender/prerender_link_manager_factory.h"
+
+#include "chrome/browser/prerender/prerender_link_manager.h"
+#include "chrome/browser/prerender/prerender_manager.h"
+#include "chrome/browser/prerender/prerender_manager_factory.h"
+#include "chrome/browser/profiles/profile.h"
+#include "chrome/browser/profiles/profile_dependency_manager.h"
+
+namespace prerender {
+
+// static
+PrerenderLinkManager* PrerenderLinkManagerFactory::GetForProfile(
+    Profile* profile) {
+  return static_cast<PrerenderLinkManager*>(
+      GetInstance()->GetServiceForProfile(profile, true));
+}
+
+// static
+PrerenderLinkManagerFactory* PrerenderLinkManagerFactory::GetInstance() {
+  return Singleton<PrerenderLinkManagerFactory>::get();
+}
+
+PrerenderLinkManagerFactory::PrerenderLinkManagerFactory()
+    : ProfileKeyedServiceFactory("PrerenderLinkmanager",
+                                 ProfileDependencyManager::GetInstance()) {
+  DependsOn(prerender::PrerenderManagerFactory::GetInstance());
+}
+
+ProfileKeyedService* PrerenderLinkManagerFactory::BuildServiceInstanceFor(
+    Profile* profile) const {
+  PrerenderManager* prerender_manager =
+      PrerenderManagerFactory::GetForProfile(profile);
+  if (!prerender_manager)

[dominich, 2012/04/26 22:50:58]

|| !prerender_manager.IsEnabled() ?

[gavinp, 2012/04/27 20:31:32]

Isn't that redundant with the check on PrerenderManager::IsPrerenderingPossible
from the PrerenderManagerFactory::BuildServiceInstanceFor ?

If it isn't redundant to it, should we fix
PrerenderManagerFactory::BuildServiceInstanceFor instead?

[dominich, 2012/04/27 21:52:01]

You're probably right, but I recently was burned by not checking for IsEnabled
when Omnibox Prerendering so I'm cautious. Feel free to do the work of checking
my assertion and responding accordingly :)

[gavinp, 2012/04/27 23:31:45]

Good point.  I'll follow up.

[cbentzel, 2012/05/02 15:54:22]

I centralized the IsEnabled check in PrerenderManager, rather than callers. 

+    return NULL;
+  PrerenderLinkManager* prerender_link_manager =
+      new PrerenderLinkManager(prerender_manager);
+  return prerender_link_manager;
+}
+
+bool PrerenderLinkManagerFactory::ServiceHasOwnInstanceInIncognito() {
+  return true;
+}
+
+}  // namespace prerender