Fix reference to potentially NULL TabContentsWrapper.

chrome/browser/autocomplete/autocomplete_edit.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/autocomplete/autocomplete_edit.h"
 
 #include <string>
 
 #include "base/basictypes.h"
 #include "base/metrics/histogram.h"
@@ skipping 1018 matching lines @@
     return false;
 
   TabContentsWrapper* tab = controller_->GetTabContentsWrapper();
 
   if (!tab)
     return false;
 
   if (user_input_in_progress() && popup_->IsOpen()) {
     return instant->Update(tab, match, view_->GetText(), UseVerbatimInstant(),
                            suggested_text);
   } else {

[Peter Kasting, 2011/12/01 21:03:38]

Nit: While here... no else after return.

[dominich, 2011/12/02 00:30:41]

Done.

     instant->Hide();
     return false;
   }
 }
 
 void AutocompleteEditModel::DoPrerender(const AutocompleteMatch& match) {
   // Do not prerender if the destination URL is the same as the current URL.
   if (match.destination_url == PermanentURL())
     return;
   TabContentsWrapper* tab = controller_->GetTabContentsWrapper();
+  if (!tab)
+    return;

[Peter Kasting, 2011/12/01 21:03:38]

Nit: It would be nice to write a comment here explaining when this can be NULL. 
(Also for DoInstant() above.)

[dominich, 2011/12/02 00:30:41]

Done.

   prerender::PrerenderManager* prerender_manager =
       prerender::PrerenderManagerFactory::GetForProfile(tab->profile());
   if (prerender_manager) {
     RenderViewHost* current_host = tab->tab_contents()->render_view_host();
     prerender_manager->AddPrerenderFromOmnibox(
         match.destination_url, current_host->session_storage_namespace());
   }
 }
 
 void AutocompleteEditModel::DoPreconnect(const AutocompleteMatch& match) {
@@ skipping 15 matching lines @@
 //  static
 bool AutocompleteEditModel::IsSpaceCharForAcceptingKeyword(wchar_t c) {
   switch (c) {
     case 0x0020:  // Space
     case 0x3000:  // Ideographic Space
       return true;
     default:
       return false;
   }
 }