Call scoped_refptr<T>::get() rather than relying on implicit "operator T*"

content/browser/web_contents/web_contents_screenshot_manager.cc

 // Copyright (c) 2013 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 "content/browser/web_contents/web_contents_screenshot_manager.h"
 
 #include "base/command_line.h"
 #include "base/threading/worker_pool.h"
 #include "content/browser/renderer_host/render_view_host_impl.h"
 #include "content/browser/web_contents/navigation_controller_impl.h"
@@ skipping 150 matching lines @@
                  unique_id,
                  screenshot));
 }
 
 int WebContentsScreenshotManager::GetScreenshotCount() const {
   int screenshot_count = 0;
   int entry_count = owner_->GetEntryCount();
   for (int i = 0; i < entry_count; ++i) {
     NavigationEntryImpl* entry =
         NavigationEntryImpl::FromNavigationEntry(owner_->GetEntryAtIndex(i));
-    if (entry->screenshot())
+    if (entry->screenshot().get())
       screenshot_count++;
   }
   return screenshot_count;
 }
 
 void WebContentsScreenshotManager::OnScreenshotEncodeComplete(
     int unique_id,
     scoped_refptr<ScreenshotData> screenshot) {
   NavigationEntryImpl* entry = NULL;
   int entry_count = owner_->GetEntryCount();
   for (int i = 0; i < entry_count; ++i) {
     NavigationEntry* iter = owner_->GetEntryAtIndex(i);
     if (iter->GetUniqueID() == unique_id) {
       entry = NavigationEntryImpl::FromNavigationEntry(iter);
       break;
     }
   }
   if (!entry)
     return;
   entry->SetScreenshotPNGData(screenshot->data());
   OnScreenshotSet(entry);
 }
 
 void WebContentsScreenshotManager::OnScreenshotSet(NavigationEntryImpl* entry) {
   PurgeScreenshotsIfNecessary();
 }
 
 bool WebContentsScreenshotManager::ClearScreenshot(NavigationEntryImpl* entry) {
-  if (!entry->screenshot())
+  if (!entry->screenshot().get())
     return false;
 
   entry->SetScreenshotPNGData(NULL);
   return true;
 }
 
 void WebContentsScreenshotManager::PurgeScreenshotsIfNecessary() {
   // Allow only a certain number of entries to keep screenshots.
   const int kMaxScreenshots = 10;
   int screenshot_count = GetScreenshotCount();
   if (screenshot_count < kMaxScreenshots)
     return;
 
   const int current = owner_->GetCurrentEntryIndex();
   const int num_entries = owner_->GetEntryCount();
   int available_slots = kMaxScreenshots;
-  if (NavigationEntryImpl::FromNavigationEntry(
-          owner_->GetEntryAtIndex(current))->screenshot()) {
+  if (NavigationEntryImpl::FromNavigationEntry(owner_->GetEntryAtIndex(current))
+          ->screenshot().get()) {
     --available_slots;
   }
 
   // Keep screenshots closer to the current navigation entry, and purge the ones
   // that are farther away from it. So in each step, look at the entries at
   // each offset on both the back and forward history, and start counting them
   // to make sure that the correct number of screenshots are kept in memory.
   // Note that it is possible for some entries to be missing screenshots (e.g.
   // when taking the screenshot failed for some reason). So there may be a state
   // where there are a lot of entries in the back history, but none of them has
   // any screenshot. In such cases, keep the screenshots for |kMaxScreenshots|
   // entries in the forward history list.
   int back = current - 1;
   int forward = current + 1;
   while (available_slots > 0 && (back >= 0 || forward < num_entries)) {
     if (back >= 0) {
       NavigationEntryImpl* entry = NavigationEntryImpl::FromNavigationEntry(
           owner_->GetEntryAtIndex(back));
-      if (entry->screenshot())
+      if (entry->screenshot().get())
         --available_slots;
       --back;
     }
 
     if (available_slots > 0 && forward < num_entries) {
       NavigationEntryImpl* entry = NavigationEntryImpl::FromNavigationEntry(
           owner_->GetEntryAtIndex(forward));
-      if (entry->screenshot())
+      if (entry->screenshot().get())
         --available_slots;
       ++forward;
     }
   }
 
   // Purge any screenshot at |back| or lower indices, and |forward| or higher
   // indices.
   while (screenshot_count > kMaxScreenshots && back >= 0) {
     NavigationEntryImpl* entry = NavigationEntryImpl::FromNavigationEntry(
         owner_->GetEntryAtIndex(back));
     if (ClearScreenshot(entry))
       --screenshot_count;
     --back;
   }
 
   while (screenshot_count > kMaxScreenshots && forward < num_entries) {
     NavigationEntryImpl* entry = NavigationEntryImpl::FromNavigationEntry(
         owner_->GetEntryAtIndex(forward));
     if (ClearScreenshot(entry))
       --screenshot_count;
     ++forward;
   }
   CHECK_GE(screenshot_count, 0);
   CHECK_LE(screenshot_count, kMaxScreenshots);
 }
 
 }  // namespace content