Convert //content from scoped_ptr to std::unique_ptr

extensions/shell/browser/shell_url_request_context_getter.h

Index: extensions/shell/browser/shell_url_request_context_getter.h
diff --git a/extensions/shell/browser/shell_url_request_context_getter.h b/extensions/shell/browser/shell_url_request_context_getter.h
index ca678eadf7ba6a39e3737787c3a24e3709aaadf6..943a133905f226db7e350c432b910ba98eb64a21 100644
--- a/extensions/shell/browser/shell_url_request_context_getter.h
+++ b/extensions/shell/browser/shell_url_request_context_getter.h
@@ -41,12 +41,12 @@ class ShellURLRequestContextGetter :
       InfoMap* extension_info_map);
 
   // content::ShellURLRequestContextGetter implementation.
-  scoped_ptr<net::NetworkDelegate> CreateNetworkDelegate() override;

[dcheng, 2016/04/09 06:12:47]

I opted to change this to use std::unique_ptr: this method is inherited from a
content/public interface, so in some sense, #including this header is a detail
of the interface.

Let me know how you feel about this.

[dcheng, 2016/04/09 06:13:50]

"this header" == scoped_ptr.h. To clarify, this is a IWYU fix: usually I fix
those by just adding headers, but I didn't for the reason previously stated.

[Avi (use Gerrit), 2016/04/09 15:46:19]

Fine.

+  std::unique_ptr<net::NetworkDelegate> CreateNetworkDelegate() override;
 
-protected:
- ~ShellURLRequestContextGetter() override;
+ protected:
+  ~ShellURLRequestContextGetter() override;
 
-private:
+ private:
   content::BrowserContext* browser_context_;
   InfoMap* extension_info_map_;