Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(212)

Unified Diff: chrome/browser/autocomplete/autocomplete_edit.cc

Issue 8745016: Fix reference to potentially NULL TabContentsWrapper. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 9 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/autocomplete/autocomplete_edit.cc
diff --git a/chrome/browser/autocomplete/autocomplete_edit.cc b/chrome/browser/autocomplete/autocomplete_edit.cc
index 6c971b8243805dcc17ac28307c5952d3da988527..5f99cf9263b8c22a1600d8d2b200c361473217e9 100644
--- a/chrome/browser/autocomplete/autocomplete_edit.cc
+++ b/chrome/browser/autocomplete/autocomplete_edit.cc
@@ -1047,6 +1047,8 @@ void AutocompleteEditModel::DoPrerender(const AutocompleteMatch& match) {
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 expl
dominich 2011/12/02 00:30:41 Done.
prerender::PrerenderManager* prerender_manager =
prerender::PrerenderManagerFactory::GetForProfile(tab->profile());
if (prerender_manager) {
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698