Fix unchecked pointer dereference that I stumbled across.

chrome/browser/external_tab_container.cc

 // Copyright (c) 2006-2008 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/external_tab_container.h"
 
 #include "app/win_util.h"
 #include "base/logging.h"
 #include "base/win_util.h"
 #include "chrome/browser/automation/automation_provider.h"
@@ skipping 250 matching lines @@
   if (!automation_)
     return;
 
   static const int kHttpClientErrorStart = 400;
   static const int kHttpServerErrorEnd = 510;
 
   switch (type.value) {
     case NotificationType::LOAD_STOP: {
         const LoadNotificationDetails* load =
             Details<LoadNotificationDetails>(details).ptr();
-        if (PageTransition::IsMainFrame(load->origin())) {
+        LOG_IF(WARNING, load == NULL) << "ExternalTabContainer received "

[amit, 2009/05/18 21:02:10]

I think this can be null for legitimate reasons, so no LOG is needed.

[robertshield, 2009/05/18 21:11:13]

Done.

+            << "null LoadNotificationDetails in Observe(LOAD_STOP).";
+        if (load != NULL && PageTransition::IsMainFrame(load->origin())) {
           automation_->Send(new AutomationMsg_TabLoaded(0, tab_handle_,
                                                         load->url()));
         }
         break;
       }
     case NotificationType::NAV_ENTRY_COMMITTED: {
         if (ignore_next_load_notification_) {
           ignore_next_load_notification_ = false;
           return;
         }
@@ skipping 142 matching lines @@
   if (!::IsWindow(parent_window)) {
     return NULL;
   }
   if (!IsExternalTabContainer(parent_window)) {
     return NULL;
   }
   ExternalTabContainer* container = reinterpret_cast<ExternalTabContainer*>(
       GetProp(parent_window, kWindowObjectKey));
   return container;
 }