exhibit 1: explicit null checks

chrome/browser/instant/instant_controller.cc

 // Copyright (c) 2012 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/i18n/case_conversion.h"
 #include "base/metrics/histogram.h"
 #include "chrome/browser/autocomplete/autocomplete_match.h"
@@ skipping 272 matching lines @@
     // Add the page to history.
     preview->history_tab_helper()->UpdateHistoryForNavigation(last_navigation);
 
     // Update the page title.
     preview->history_tab_helper()->UpdateHistoryPageTitle(*entry);
 
     // Update the favicon.
     FaviconService* favicon_service =
         preview->profile()->GetFaviconService(Profile::EXPLICIT_ACCESS);
     if (favicon_service && entry->GetFavicon().valid &&
-        entry->GetFavicon().image.IsEmpty()) {
+        !entry->GetFavicon().image.IsEmpty()) {
       std::vector<unsigned char> image_data;
       // TODO: Add all variants once the history service supports it.
       gfx::PNGCodec::EncodeBGRASkBitmap(
-          entry->GetFavicon().image.AsBitmap(), false, &image_data);
+          *entry->GetFavicon().image.ToSkBitmap(), false, &image_data);
       favicon_service->SetFavicon(entry->GetURL(),
                                   entry->GetFavicon().url,
                                   image_data,
                                   history::FAVICON);
     }
   }
 
   // Add a fake history entry with a non-Instant search URL, so that search
   // terms extraction (for autocomplete history matches) works.
   HistoryService* history = HistoryServiceFactory::GetForProfile(
@@ skipping 286 matching lines @@
     return false;
   }
 
   return true;
 }
 
 bool InstantController::IsOutOfDate() const {
   return !last_active_tab_ ||
          last_active_tab_ != delegate_->GetActiveTabContents();
 }