Added prerendering to omnibox.

chrome/browser/instant/instant_controller.cc

 // Copyright (c) 2011 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/instant/instant_controller.h"
 
 #include "base/command_line.h"
 #include "base/message_loop.h"
 #include "base/metrics/histogram.h"
 #include "build/build_config.h"
 #include "chrome/browser/autocomplete/autocomplete_match.h"
 #include "chrome/browser/instant/instant_delegate.h"
 #include "chrome/browser/instant/instant_loader.h"
 #include "chrome/browser/instant/instant_loader_manager.h"
 #include "chrome/browser/instant/promo_counter.h"
 #include "chrome/browser/platform_util.h"
 #include "chrome/browser/prefs/pref_service.h"
+#include "chrome/browser/prerender/prerender_manager.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/search_engines/template_url.h"
 #include "chrome/browser/search_engines/template_url_service.h"
 #include "chrome/browser/search_engines/template_url_service_factory.h"
 #include "chrome/browser/ui/blocked_content/blocked_content_tab_helper.h"
 #include "chrome/browser/ui/tab_contents/tab_contents_wrapper.h"
 #include "chrome/common/chrome_switches.h"
 #include "chrome/common/pref_names.h"
 #include "chrome/common/url_constants.h"
 #include "content/browser/renderer_host/render_widget_host_view.h"
@@ skipping 134 matching lines @@
   const TemplateURL* template_url = NULL;
 
   if (url.is_empty() || !url.is_valid()) {
     // Assume we were invoked with GURL() and should destroy all.
     DestroyPreviewContents();
     return;
   }
 
   if (!ShouldShowPreviewFor(match, &template_url)) {
     DestroyPreviewContentsAndLeaveActive();
+    // Start Prerender of this page instead.
+    CommandLine* cl = CommandLine::ForCurrentProcess();

[sky, 2011/06/20 18:15:54]

Seems like you only want to do this if you hit the last if in
ShouldShouldPreviewFor.

[dominich, 2011/06/20 18:40:00]

You're right. I didn't think it would matter that much and this feels like the
right place, but prerendering when the host is blacklisted or it's a javascript
scheme probably isn't a good idea.

Moved.

+    if (cl->HasSwitch(switches::kPrerenderFromOmnibox)) {
+      prerender::PrerenderManager* prerender_manager =
+          tab_contents_->profile()->GetPrerenderManager();
+      if (prerender_manager)
+        prerender_manager->AddPrerenderWithNoTag(match.destination_url);
+    }
     return;
   }
 
   if (!loader_manager_.get())
     loader_manager_.reset(new InstantLoaderManager(this));
 
   if (!is_active_) {
     is_active_ = true;
     delegate_->PrepareForInstant();
   }
@@ skipping 547 matching lines @@
   const TemplateURL* template_url = match.template_url;
   if (match.type == AutocompleteMatch::SEARCH_WHAT_YOU_TYPED ||
       match.type == AutocompleteMatch::SEARCH_HISTORY ||
       match.type == AutocompleteMatch::SEARCH_SUGGEST) {
     TemplateURLService* model = TemplateURLServiceFactory::GetForProfile(
         tab_contents_->profile());
     template_url = model ? model->GetDefaultSearchProvider() : NULL;
   }
   return template_url;
 }