Remove ScopedVector from components/sessions.

Remove ScopedVector from components/sessions.

BUG=554289, 561329
Committed: https://crrev.com/51f1cd9460c0e166805ceb6d47a4380408c9fae6
Cr-Commit-Position: refs/heads/master@{#441162}

[Avi (use Gerrit), 2016-12-22 20:49:12]

The CQ bit was checked by avi@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-12-22 20:49:41]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-12-22 21:16:47]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-12-22 21:16:48]

Dry run: Try jobs failed on following builders:
  ios-device on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device/builds...)
  ios-simulator on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator/bui...)

[Avi (use Gerrit), 2016-12-31 21:50:58]

The CQ bit was checked by avi@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-12-31 21:51:03]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-12-31 21:59:33]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-12-31 21:59:34]

Dry run: Try jobs failed on following builders:
  linux_chromium_chromeos_ozone_rel_ng on master.tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_chromeos_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[Avi (use Gerrit), 2016-12-31 22:06:27]

The CQ bit was checked by avi@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-12-31 22:06:36]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-12-31 22:48:36]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-12-31 22:48:37]

Dry run: This issue passed the CQ dry run.

[Avi (use Gerrit), 2016-12-31 22:57:48]

avi@chromium.org changed reviewers:
+ eugenebut@chromium.org, skuhne@chromium.org, sky@chromium.org

[Avi (use Gerrit), 2016-12-31 22:57:49]

sky/skuhne: sessions
eugenebut: ios

[Eugene But (OOO till 7-30), 2017-01-03 17:25:39]

ios lgtm

[sky, 2017-01-03 17:29:34]

LGTM

https://codereview.chromium.org/2600583002/diff/40001/chrome/browser/sessions...
File chrome/browser/sessions/session_service.h (right):

https://codereview.chromium.org/2600583002/diff/40001/chrome/browser/sessions...
chrome/browser/sessions/session_service.h:14: #include
"base/memory/scoped_vector.h"
Remove include?

[Avi (use Gerrit), 2017-01-03 17:34:03]

https://codereview.chromium.org/2600583002/diff/40001/chrome/browser/sessions...
File chrome/browser/sessions/session_service.h (right):

https://codereview.chromium.org/2600583002/diff/40001/chrome/browser/sessions...
chrome/browser/sessions/session_service.h:14: #include
"base/memory/scoped_vector.h"
On 2017/01/03 17:29:34, sky (OOO) wrote:
> Remove include?

Done.

[Avi (use Gerrit), 2017-01-03 17:36:00]

The CQ bit was checked by avi@chromium.org

[Avi (use Gerrit), 2017-01-03 17:36:01]

The patchset sent to the CQ was uploaded after l-g-t-m from
eugenebut@chromium.org, sky@chromium.org
Link to the patchset: https://codereview.chromium.org/2600583002/#ps60001
(title: "include")

[commit-bot: I haz the power, 2017-01-03 17:36:16]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-01-03 18:23:42]

CQ is committing da patch.

Bot data: {"patchset_id": 60001, "attempt_start_ts": 1483464960497970,
"parent_rev": "b54db473de500ca09853cfd26fd6ed3ec4358aea", "commit_rev":
"2d8bc1701ba2dc7475faf694e984986d26fe0a15"}

[commit-bot: I haz the power, 2017-01-03 18:24:11]

Description was changed from

==========
Remove ScopedVector from components/sessions.

BUG=554289, 561329
==========

to

==========
Remove ScopedVector from components/sessions.

BUG=554289, 561329

Review-Url: https://codereview.chromium.org/2600583002
==========

[commit-bot: I haz the power, 2017-01-03 18:24:12]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2017-01-03 18:27:39]

Description was changed from

==========
Remove ScopedVector from components/sessions.

BUG=554289, 561329

Review-Url: https://codereview.chromium.org/2600583002
==========

to

==========
Remove ScopedVector from components/sessions.

BUG=554289, 561329

Committed: https://crrev.com/51f1cd9460c0e166805ceb6d47a4380408c9fae6
Cr-Commit-Position: refs/heads/master@{#441162}
==========

[commit-bot: I haz the power, 2017-01-03 18:27:40]

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