Remove ScopedVector from proxy_resolver_factory_mojo_unittest

Remove ScopedVector from proxy_resolver_factory_mojo_unittest

Replace ScopedVector with std::vector<scoped_ptr> in
proxy_resolver_factory_mojo_unittest. Also use some std::move isntead of Pass().

BUG=554289
Committed: https://crrev.com/6a847e3597891af23b78ef4088d19f1a773d7e69
Cr-Commit-Position: refs/heads/master@{#364007}

[Olli Raula, 2015-12-07 16:22:28]

Description was changed from

==========
Remove ScopedVector from proxy_resolver_factory_mojo_unittest

Replace ScopedVector with std::vector<scoped_ptr> from
proxy_resolver_factory_mojo_unittest

BUG=554289
==========

to

==========
Remove ScopedVector from proxy_resolver_factory_mojo_unittest

Replace ScopedVector with std::vector<scoped_ptr> in
proxy_resolver_factory_mojo_unittest

BUG=554289
==========

[Olli Raula, 2015-12-08 09:08:19]

olli.raula@intel.com changed reviewers:
+ eroman@chromium.org

[Olli Raula, 2015-12-08 09:08:32]

Could you review?

[eroman, 2015-12-08 19:50:17]

lgtm

https://codereview.chromium.org/1509513002/diff/1/net/proxy/proxy_resolver_fa...
File net/proxy/proxy_resolver_factory_mojo_unittest.cc (left):

https://codereview.chromium.org/1509513002/diff/1/net/proxy/proxy_resolver_fa...
net/proxy/proxy_resolver_factory_mojo_unittest.cc:441: new
interfaces::ProxyResolverFactoryRequestClientPtr(client.Pass()));
strictly speaking these changes from .Pass() --> std::move() are not part of the
CL description (or necessary for this refactor).

That said should be fine, since our style guide now permits move()

[Olli Raula, 2015-12-09 07:10:58]

Description was changed from

==========
Remove ScopedVector from proxy_resolver_factory_mojo_unittest

Replace ScopedVector with std::vector<scoped_ptr> in
proxy_resolver_factory_mojo_unittest

BUG=554289
==========

to

==========
Remove ScopedVector from proxy_resolver_factory_mojo_unittest

Replace ScopedVector with std::vector<scoped_ptr> in
proxy_resolver_factory_mojo_unittest. Also use some std::move isntead of Pass().

BUG=554289
==========

[Olli Raula, 2015-12-09 07:11:17]

The CQ bit was checked by olli.raula@intel.com

[commit-bot: I haz the power, 2015-12-09 07:12:02]

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

[commit-bot: I haz the power, 2015-12-09 08:31:05]

Description was changed from

==========
Remove ScopedVector from proxy_resolver_factory_mojo_unittest

Replace ScopedVector with std::vector<scoped_ptr> in
proxy_resolver_factory_mojo_unittest. Also use some std::move isntead of Pass().

BUG=554289
==========

to

==========
Remove ScopedVector from proxy_resolver_factory_mojo_unittest

Replace ScopedVector with std::vector<scoped_ptr> in
proxy_resolver_factory_mojo_unittest. Also use some std::move isntead of Pass().

BUG=554289
==========

[commit-bot: I haz the power, 2015-12-09 08:31:06]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-12-09 08:32:01]

Description was changed from

==========
Remove ScopedVector from proxy_resolver_factory_mojo_unittest

Replace ScopedVector with std::vector<scoped_ptr> in
proxy_resolver_factory_mojo_unittest. Also use some std::move isntead of Pass().

BUG=554289
==========

to

==========
Remove ScopedVector from proxy_resolver_factory_mojo_unittest

Replace ScopedVector with std::vector<scoped_ptr> in
proxy_resolver_factory_mojo_unittest. Also use some std::move isntead of Pass().

BUG=554289

Committed: https://crrev.com/6a847e3597891af23b78ef4088d19f1a773d7e69
Cr-Commit-Position: refs/heads/master@{#364007}
==========

[commit-bot: I haz the power, 2015-12-09 08:32:02]

Patchset 1 (id:??) landed as
https://crrev.com/6a847e3597891af23b78ef4088d19f1a773d7e69
Cr-Commit-Position: refs/heads/master@{#364007}