some more fixes for ignored scoped_ptr::release() calls

BUG=42904
TEST=bots

Review URL: http://codereview.chromium.org/1982001

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@46622 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 5 insertions, 5 deletions

diff --git a/chrome/browser/cocoa/location_bar_view_mac.mm b/chrome/browser/cocoa/location_bar_view_mac.mm
index 807c400..2d68c91 100644
--- a/chrome/browser/cocoa/location_bar_view_mac.mm
+++ b/chrome/browser/cocoa/location_bar_view_mac.mm
@@ -240,8 +240,8 @@ void LocationBarViewMac::OnAutocompleteAccept(const GURL& url,
     return;
   }
 
-  scoped_ptr<AlternateNavURLFetcher> fetcher(
-      new AlternateNavURLFetcher(alternate_nav_url));
+  AlternateNavURLFetcher* fetcher =
+      new AlternateNavURLFetcher(alternate_nav_url);
   // The AlternateNavURLFetcher will listen for the pending navigation
   // notification that will be issued as a result of the "open URL." It
   // will automatically install itself into that navigation controller.
@@ -249,10 +249,10 @@ void LocationBarViewMac::OnAutocompleteAccept(const GURL& url,
   if (fetcher->state() == AlternateNavURLFetcher::NOT_STARTED) {
     // I'm not sure this should be reachable, but I'm not also sure enough
     // that it shouldn't to stick in a NOTREACHED().  In any case, this is
-    // harmless; we can simply let the fetcher get deleted here and it will
-    // clean itself up properly.
+    // harmless.
+    delete fetcher;
   } else {
-    fetcher.release();  // The navigation controller will delete the fetcher.
+    // The navigation controller will delete the fetcher.
   }
 }