Remove ScopedVector from extensions/api/dial.

Remove ScopedVector from extensions/api/dial.

BUG=554289
R=justinlin@chromium.org

Committed: https://crrev.com/026b848a61b86043b471d694303684781741ed17
Cr-Commit-Position: refs/heads/master@{#360506}

[Igor Vayzert, 2015-11-17 03:33:19]

Description was changed from

==========
Remove ScopedVector from extensions/api/dial.

BUG=554289

R=justinlin@chromium.org
==========

to

==========
Remove ScopedVector from extensions/api/dial.

BUG=554289

R=justinlin@chromium.org
==========

[Igor Vayzert, 2015-11-17 03:33:19]

weisert@yandex-team.ru changed reviewers:
+ mfoltz@chromium.org

[Igor Vayzert, 2015-11-17 17:05:56]

PTAL

[mark a. foltz, 2015-11-17 22:34:00]

mfoltz@chromium.org changed reviewers:
- justinlin@chromium.org

[mark a. foltz, 2015-11-17 22:37:38]

https://codereview.chromium.org/1452513002/diff/1/chrome/browser/extensions/a...
File chrome/browser/extensions/api/dial/dial_service.cc (right):

https://codereview.chromium.org/1452513002/diff/1/chrome/browser/extensions/a...
chrome/browser/extensions/api/dial/dial_service.cc:555: auto iter =
dial_sockets_.begin();
Can this be declared in the for() ?

https://codereview.chromium.org/1452513002/diff/1/chrome/browser/extensions/a...
chrome/browser/extensions/api/dial/dial_service.cc:619: auto iter =
dial_sockets_.begin();
Ditto

[Igor Vayzert, 2015-11-18 03:12:56]

https://codereview.chromium.org/1452513002/diff/1/chrome/browser/extensions/a...
File chrome/browser/extensions/api/dial/dial_service.cc (right):

https://codereview.chromium.org/1452513002/diff/1/chrome/browser/extensions/a...
chrome/browser/extensions/api/dial/dial_service.cc:555: auto iter =
dial_sockets_.begin();
On 2015/11/17 22:37:38, mark a. foltz wrote:
> Can this be declared in the for() ?

Done.

https://codereview.chromium.org/1452513002/diff/1/chrome/browser/extensions/a...
chrome/browser/extensions/api/dial/dial_service.cc:619: auto iter =
dial_sockets_.begin();
On 2015/11/17 22:37:38, mark a. foltz wrote:
> Ditto

Done.

[mark a. foltz, 2015-11-18 20:22:14]

lgtm

[Igor Vayzert, 2015-11-19 02:08:06]

The CQ bit was checked by weisert@yandex-team.ru

[commit-bot: I haz the power, 2015-11-19 02:10:34]

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

[commit-bot: I haz the power, 2015-11-19 03:05:07]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-11-19 03:06:02]

Patchset 2 (id:??) landed as
https://crrev.com/026b848a61b86043b471d694303684781741ed17
Cr-Commit-Position: refs/heads/master@{#360506}