**NOTFORLANDING** New link rel=prerender API

chrome/browser/prerender/prerender_link_manager_factory.cc

+// 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) {
+  if (!PrerenderManager::IsPrerenderingPossible())

[cbentzel, 2012/04/24 15:42:59]

This check doesn't seem necessary - kind of covered by the
PrerenderManagerFactory::GetForProfile() which is called in
BuildServiceInstanceFor

[gavinp, 2012/04/26 23:55:39]

Done.

+    return NULL;
+  return static_cast<PrerenderLinkManager*>(
+      GetInstance()->GetServiceForProfile(profile,true));
+}
+
+// static
+PrerenderLinkManagerFactory* PrerenderLinkManagerFactory::GetInstance() {
+  return Singleton<PrerenderLinkManagerFactory>::get();
+}
+
+PrerenderLinkManagerFactory::PrerenderLinkManagerFactory()
+    : ProfileKeyedServiceFactory("PrerenderLinkmanager",

[cbentzel, 2012/04/24 15:42:59]

It seems like this factory should depend on PrerenderManagerFactory, so the
PrerenderLinkManager is created after the PrerenderManager for a profile. You
can just use DependsOn to do this.

[gavinp, 2012/04/26 23:55:39]

Done.

+                                 ProfileDependencyManager::GetInstance()) {
+}
+
+ProfileKeyedService* PrerenderLinkManagerFactory::BuildServiceInstanceFor(
+    Profile* profile) const {
+  PrerenderManager* prerender_manager =
+      PrerenderManagerFactory::GetForProfile(profile);
+  if (!prerender_manager)
+    return NULL;
+  PrerenderLinkManager* prerender_link_manager =
+      new PrerenderLinkManager(prerender_manager);
+  return prerender_link_manager;
+}
+
+bool PrerenderLinkManagerFactory::ServiceHasOwnInstanceInIncognito() {
+  return true;
+}
+
+}

[cbentzel, 2012/04/24 15:42:59]

Nit: // namespace prerender