Remove implicit scoped_refptr operator T*

chrome/browser/history/history.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.
 
 // The history system runs on a background thread so that potentially slow
 // database operations don't delay the browser. This backend processing is
 // represented by HistoryBackend. The HistoryService's job is to dispatch to
 // that thread.
 //
 // Main thread                       History thread
@@ skipping 175 matching lines @@
   // callers of this assume that if the backend isn't yet loaded it will be
   // soon, so they will either listen for notifications or just retry this call
   // later.  If we've purged the backend, we haven't necessarily restarted it
   // loading by now, so we need to trigger the load in order to maintain that
   // expectation.
   LoadBackendIfNecessary();
   return backend_loaded_;
 }
 
 void HistoryService::UnloadBackend() {
-  if (!history_backend_)
+  if (!history_backend_.get())
     return;  // Already unloaded.
 
   // Get rid of the in-memory backend.
   in_memory_backend_.reset();
 
   // Give the InMemoryURLIndex a chance to shutdown.
   if (in_memory_url_index_.get())
     in_memory_url_index_->ShutDown();
 
   // The backend's destructor must run on the history thread since it is not
@@ skipping 628 matching lines @@
   // Allow all about: and chrome: URLs except about:blank, since the user may
   // like to see "chrome://memory/", etc. in their history and autocomplete.
   if (url == GURL(chrome::kAboutBlankURL))
     return false;
 
   return true;
 }
 
 void HistoryService::SetInMemoryBackend(int backend_id,
     history::InMemoryHistoryBackend* mem_backend) {
-  if (!history_backend_ || current_backend_id_ != backend_id) {
+  if (!history_backend_.get() || current_backend_id_ != backend_id) {
     VLOG(1) << "Message from obsolete backend";
     // Cleaning up the memory backend.
     delete mem_backend;
     return;
   }
   DCHECK(!in_memory_backend_.get()) << "Setting mem DB twice";
   in_memory_backend_.reset(mem_backend);
 
   // The database requires additional initialization once we own it.
   in_memory_backend_->AttachToHistoryService(profile_);
 }
 
 void HistoryService::NotifyProfileError(int backend_id,
                                         sql::InitStatus init_status) {
-  if (!history_backend_ || current_backend_id_ != backend_id) {
+  if (!history_backend_.get() || current_backend_id_ != backend_id) {
     VLOG(1) << "Message from obsolete backend";
     return;
   }
   ShowProfileErrorDialog(
       (init_status == sql::INIT_FAILURE) ?
       IDS_COULDNT_OPEN_PROFILE_ERROR : IDS_PROFILE_TOO_NEW_ERROR);
 }
 
 void HistoryService::DeleteURL(const GURL& url) {
   // We will update the visited links when we observe the delete notifications.
@@ skipping 36 matching lines @@
 
   // The details object just contains the pointer to the object that the
   // backend has allocated for us. The receiver of the notification will cast
   // this to the proper type.
   content::Details<history::HistoryDetails> det(details);
 
   content::NotificationService::current()->Notify(type, source, det);
 }
 
 void HistoryService::LoadBackendIfNecessary() {
-  if (!thread_ || history_backend_)
+  if (!thread_ || history_backend_.get())
     return;  // Failed to init, or already started loading.
 
   ++current_backend_id_;
   scoped_refptr<HistoryBackend> backend(
       new HistoryBackend(history_dir_,
                          current_backend_id_,
                          new BackendDelegate(this, profile_),
                          bookmark_service_));
   history_backend_.swap(backend);
 
   // There may not be a profile when unit testing.
   std::string languages;
   if (profile_) {
     PrefService* prefs = profile_->GetPrefs();
     languages = prefs->GetString(prefs::kAcceptLanguages);
   }
   ScheduleAndForget(PRIORITY_UI, &HistoryBackend::Init, languages, no_db_);
 }
 
 void HistoryService::OnDBLoaded(int backend_id) {
-  if (!history_backend_ || current_backend_id_ != backend_id) {
+  if (!history_backend_.get() || current_backend_id_ != backend_id) {
     VLOG(1) << "Message from obsolete backend";
     return;
   }
   backend_loaded_ = true;
   content::NotificationService::current()->Notify(
       chrome::NOTIFICATION_HISTORY_LOADED,
       content::Source<Profile>(profile_),
       content::Details<HistoryService>(this));
   if (thread_ && profile_) {
     // We don't want to force creation of TopSites.
     history::TopSites* ts = profile_->GetTopSitesWithoutCreating();
     if (ts)
       ts->HistoryLoaded();
   }
 }
 
 bool HistoryService::GetRowForURL(const GURL& url, history::URLRow* url_row) {
   history::URLDatabase* db = InMemoryDatabase();
   return db && (db->GetRowForURL(url, url_row) != 0);
 }
 
 void HistoryService::StartTopSitesMigration(int backend_id) {
-  if (!history_backend_ || current_backend_id_ != backend_id) {
+  if (!history_backend_.get() || current_backend_id_ != backend_id) {
     VLOG(1) << "Message from obsolete backend";
     return;
   }
   needs_top_sites_migration_ = true;
   if (thread_ && profile_) {
     // We don't want to force creation of TopSites.
     history::TopSites* ts = profile_->GetTopSitesWithoutCreating();
     if (ts)
       ts->MigrateFromHistory();
   }
@@ skipping 12 matching lines @@
 void HistoryService::RemoveVisitDatabaseObserver(
     history::VisitDatabaseObserver* observer) {
   visit_database_observers_->RemoveObserver(observer);
 }
 
 void HistoryService::NotifyVisitDBObserversOnAddVisit(
     const history::BriefVisitInfo& info) {
   visit_database_observers_->Notify(
       &history::VisitDatabaseObserver::OnAddVisit, info);
 }