Convert //android_webview to use std::unique_ptr

Convert //android_webview to use std::unique_ptr

BUG=554298
Committed: https://crrev.com/08daa0770adc348a5e3852200352b2cd7f620d21
Cr-Commit-Position: refs/heads/master@{#384471}

[dcheng, 2016-03-31 23:53:16]

dcheng@chromium.org changed reviewers:
+ boliu@chromium.org

[dcheng, 2016-03-31 23:53:17]

It begins…

[boliu, 2016-04-01 00:49:42]

lgtm

found some existing incorrect includes. this CL doesn't make those any worse

And idea, is it worth removing scoped_ptr.h from android_webview/DEPS?

https://codereview.chromium.org/1852513003/diff/20001/android_webview/browser...
File android_webview/browser/net/input_stream_reader_unittest.cc (right):

https://codereview.chromium.org/1852513003/diff/20001/android_webview/browser...
android_webview/browser/net/input_stream_reader_unittest.cc:7: #include <memory>
old scoped_ptr include is actually not needed, so neither is this one

https://codereview.chromium.org/1852513003/diff/20001/android_webview/lib/aw_...
File android_webview/lib/aw_browser_dependency_factory_impl.h (right):

https://codereview.chromium.org/1852513003/diff/20001/android_webview/lib/aw_...
android_webview/lib/aw_browser_dependency_factory_impl.h:8: #include <memory>
ditto

https://codereview.chromium.org/1852513003/diff/20001/android_webview/native/...
File android_webview/native/aw_contents_io_thread_client_impl.h (right):

https://codereview.chromium.org/1852513003/diff/20001/android_webview/native/...
android_webview/native/aw_contents_io_thread_client_impl.h:10: #include <memory>
ditto..

Actually not quite. Might break aw_web_contents_delegate.cc

https://codereview.chromium.org/1852513003/diff/20001/android_webview/native/...
File android_webview/native/aw_debug.h (right):

https://codereview.chromium.org/1852513003/diff/20001/android_webview/native/...
android_webview/native/aw_debug.h:10: #include <memory>
ditto

[dcheng, 2016-04-01 00:58:01]

https://codereview.chromium.org/1852513003/diff/20001/android_webview/browser...
File android_webview/browser/net/input_stream_reader_unittest.cc (right):

https://codereview.chromium.org/1852513003/diff/20001/android_webview/browser...
android_webview/browser/net/input_stream_reader_unittest.cc:7: #include <memory>
On 2016/04/01 at 00:49:42, boliu wrote:
> old scoped_ptr include is actually not needed, so neither is this one

Done (this is all automated, and I'm not able to spotcheck everything, so thanks
for catching this)

https://codereview.chromium.org/1852513003/diff/20001/android_webview/lib/aw_...
File android_webview/lib/aw_browser_dependency_factory_impl.h (right):

https://codereview.chromium.org/1852513003/diff/20001/android_webview/lib/aw_...
android_webview/lib/aw_browser_dependency_factory_impl.h:8: #include <memory>
On 2016/04/01 at 00:49:42, boliu wrote:
> ditto

Done.

https://codereview.chromium.org/1852513003/diff/20001/android_webview/native/...
File android_webview/native/aw_contents_io_thread_client_impl.h (right):

https://codereview.chromium.org/1852513003/diff/20001/android_webview/native/...
android_webview/native/aw_contents_io_thread_client_impl.h:10: #include <memory>
On 2016/04/01 at 00:49:42, boliu wrote:
> ditto..
> 
> Actually not quite. Might break aw_web_contents_delegate.cc

Done. We'll see, if it breaks, aw_web_contents_delegate.cc should be fixed.

https://codereview.chromium.org/1852513003/diff/20001/android_webview/native/...
File android_webview/native/aw_debug.h (right):

https://codereview.chromium.org/1852513003/diff/20001/android_webview/native/...
android_webview/native/aw_debug.h:10: #include <memory>
On 2016/04/01 at 00:49:42, boliu wrote:
> ditto

Done.

[dcheng, 2016-04-01 01:00:13]

The CQ bit was checked by dcheng@chromium.org

[dcheng, 2016-04-01 01:00:14]

The patchset sent to the CQ was uploaded after l-g-t-m from boliu@chromium.org
Link to the patchset: https://codereview.chromium.org/1852513003/#ps50001
(title: "git is hard")

[commit-bot: I haz the power, 2016-04-01 01:00:56]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1852513003/50001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1852513003/50001

[commit-bot: I haz the power, 2016-04-01 01:47:30]

Committed patchset #4 (id:50001)

[commit-bot: I haz the power, 2016-04-01 01:48:34]

Description was changed from

==========
Convert //android_webview to use std::unique_ptr

BUG=554298
==========

to

==========
Convert //android_webview to use std::unique_ptr

BUG=554298

Committed: https://crrev.com/08daa0770adc348a5e3852200352b2cd7f620d21
Cr-Commit-Position: refs/heads/master@{#384471}
==========

[commit-bot: I haz the power, 2016-04-01 01:48:35]

Patchset 4 (id:??) landed as
https://crrev.com/08daa0770adc348a5e3852200352b2cd7f620d21
Cr-Commit-Position: refs/heads/master@{#384471}