Fix crash due to erroneous use of |controller_|....

chrome/browser/google/google_url_tracker.cc

 // Copyright (c) 2010 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/google/google_url_tracker.h"
 
 #include <vector>
 
 #include "app/l10n_util.h"
 #include "base/command_line.h"
@@ skipping 295 matching lines @@
                                const NotificationDetails& details) {
   switch (type.value) {
     case NotificationType::DEFAULT_REQUEST_CONTEXT_AVAILABLE:
       registrar_.Remove(this,
                         NotificationType::DEFAULT_REQUEST_CONTEXT_AVAILABLE,
                         NotificationService::AllSources());
       request_context_available_ = true;
       StartFetchIfDesirable();
       break;
 
-    case NotificationType::NAV_ENTRY_PENDING:
-      OnNavigationPending(source, controller_->pending_entry()->url());
+    case NotificationType::NAV_ENTRY_PENDING: {
+      NavigationController* controller =
+          Source<NavigationController>(source).ptr();
+      OnNavigationPending(source, controller->pending_entry()->url());
       break;
+    }

[Ilya Sherman, 2010/11/15 19:40:51]

nit: Please either add braces to the other case statements or remove these

 
     case NotificationType::NAV_ENTRY_COMMITTED:
     case NotificationType::TAB_CLOSED:
       OnNavigationCommittedOrTabClosed(
           Source<NavigationController>(source).ptr()->tab_contents(),
           type.value);
       break;
 
     default:
       NOTREACHED() << "Unknown notification received:" << type.value;
@@ skipping 42 matching lines @@
 }
 
 void GoogleURLTracker::ShowGoogleURLInfoBarIfNecessary(
     TabContents* tab_contents) {
   if (!need_to_prompt_)
     return;
   DCHECK(!fetched_google_url_.is_empty());
 
   infobar_ = (*infobar_creator_)(tab_contents, this, fetched_google_url_);
 }