WTF/std normalization: replace WTF::Vector::removeLast with ::pop_back

WTF/std normalization: replace WTF::Vector::removeLast with ::pop_back

As discussed on platform-architecture-dev [0], there is interest in
normalizing the methods of WTF classes to better align with std classes.
This CL replaces removeLast() with pop_back(). There are no
functional changes.

[0] https://groups.google.com/a/chromium.org/d/topic/platform-architecture-dev/I7jnz4p1h84/discussion

BUG=662431
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2

Committed: https://crrev.com/c9e1380ae7c169501892b25f1ee82e04207e9208
Cr-Commit-Position: refs/heads/master@{#429930}

[pilgrim_google, 2016-10-28 19:15:13]

Description was changed from

==========
WTF/std normalization: replace WTF::Vector::removeLast with ::pop_back

WTF/std normalization: replace WTF::Vector::emplaceAppend with ::emplace_back

As discussed on platform-architecture-dev [0], there is interest in
normalizing the methods of WTF classes to better align with std classes.
This CL replaces removeLast() with pop_back(). There are no
functional changes.

[0]
https://groups.google.com/a/chromium.org/d/topic/platform-architecture-dev/I7...>

BUG=
==========

to

==========
WTF/std normalization: replace WTF::Vector::removeLast with ::pop_back

WTF/std normalization: replace WTF::Vector::emplaceAppend with ::emplace_back

As discussed on platform-architecture-dev [0], there is interest in
normalizing the methods of WTF classes to better align with std classes.
This CL replaces removeLast() with pop_back(). There are no
functional changes.

[0]
https://groups.google.com/a/chromium.org/d/topic/platform-architecture-dev/I7...>

BUG=
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

[pilgrim_google, 2016-10-28 19:23:32]

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

[commit-bot: I haz the power, 2016-10-28 19:24:01]

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

[pilgrim_google, 2016-10-28 19:37:06]

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

[commit-bot: I haz the power, 2016-10-28 19:37:43]

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-10-28 19:55:33]

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

[commit-bot: I haz the power, 2016-10-28 19:55:34]

Dry run: Try jobs failed on following builders:
  mac_chromium_compile_dbg_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)

[pilgrim_google, 2016-10-28 19:57:44]

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

[commit-bot: I haz the power, 2016-10-28 19:58:23]

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-10-28 20:56:06]

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

[commit-bot: I haz the power, 2016-10-28 20:56:07]

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

[pilgrim_google, 2016-11-03 19:10:42]

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

[commit-bot: I haz the power, 2016-11-03 19:11:27]

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

[pilgrim_google, 2016-11-03 20:04:18]

pilgrim@chromium.org changed reviewers:
+ esprehn@chromium.org, haraken@chromium.org, tkent@chromium.org

[commit-bot: I haz the power, 2016-11-03 23:15:13]

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

[commit-bot: I haz the power, 2016-11-03 23:15:13]

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

[haraken, 2016-11-03 23:49:34]

LGTM

Remove "WTF/std normalization: replace WTF::Vector::emplaceAppend with
::emplace_back" from the CL description.

Also please create a tracking bug.

[Yuta Kitamura, 2016-11-04 05:50:48]

yutak@chromium.org changed reviewers:
+ yutak@chromium.org

[Yuta Kitamura, 2016-11-04 05:50:49]

LGTM

In addition to what haraken said, please fix the URL in
the change description (extra characters in the end).

[pilgrim_google, 2016-11-04 16:13:41]

Description was changed from

==========
WTF/std normalization: replace WTF::Vector::removeLast with ::pop_back

WTF/std normalization: replace WTF::Vector::emplaceAppend with ::emplace_back

As discussed on platform-architecture-dev [0], there is interest in
normalizing the methods of WTF classes to better align with std classes.
This CL replaces removeLast() with pop_back(). There are no
functional changes.

[0]
https://groups.google.com/a/chromium.org/d/topic/platform-architecture-dev/I7...>

BUG=
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

to

==========
WTF/std normalization: replace WTF::Vector::removeLast with ::pop_back

WTF/std normalization: replace WTF::Vector::emplaceAppend with ::emplace_back

As discussed on platform-architecture-dev [0], there is interest in
normalizing the methods of WTF classes to better align with std classes.
This CL replaces removeLast() with pop_back(). There are no
functional changes.

[0]
https://groups.google.com/a/chromium.org/d/topic/platform-architecture-dev/I7...>

BUG=662431
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

[pilgrim_google, 2016-11-04 16:14:14]

Description was changed from

==========
WTF/std normalization: replace WTF::Vector::removeLast with ::pop_back

WTF/std normalization: replace WTF::Vector::emplaceAppend with ::emplace_back

As discussed on platform-architecture-dev [0], there is interest in
normalizing the methods of WTF classes to better align with std classes.
This CL replaces removeLast() with pop_back(). There are no
functional changes.

[0]
https://groups.google.com/a/chromium.org/d/topic/platform-architecture-dev/I7...>

BUG=662431
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

to

==========
WTF/std normalization: replace WTF::Vector::removeLast with ::pop_back

WTF/std normalization: replace WTF::Vector::emplaceAppend with ::emplace_back

As discussed on platform-architecture-dev [0], there is interest in
normalizing the methods of WTF classes to better align with std classes.
This CL replaces removeLast() with pop_back(). There are no
functional changes.

[0]
https://groups.google.com/a/chromium.org/d/topic/platform-architecture-dev/I7...

BUG=662431
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

[pilgrim_google, 2016-11-04 16:14:46]

Description was changed from

==========
WTF/std normalization: replace WTF::Vector::removeLast with ::pop_back

WTF/std normalization: replace WTF::Vector::emplaceAppend with ::emplace_back

As discussed on platform-architecture-dev [0], there is interest in
normalizing the methods of WTF classes to better align with std classes.
This CL replaces removeLast() with pop_back(). There are no
functional changes.

[0]
https://groups.google.com/a/chromium.org/d/topic/platform-architecture-dev/I7...

BUG=662431
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

to

==========
WTF/std normalization: replace WTF::Vector::removeLast with ::pop_back

As discussed on platform-architecture-dev [0], there is interest in
normalizing the methods of WTF classes to better align with std classes.
This CL replaces removeLast() with pop_back(). There are no
functional changes.

[0]
https://groups.google.com/a/chromium.org/d/topic/platform-architecture-dev/I7...

BUG=662431
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

[pilgrim_google, 2016-11-04 16:14:55]

The CQ bit was checked by pilgrim@chromium.org

[commit-bot: I haz the power, 2016-11-04 16:15:39]

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

[pilgrim_google, 2016-11-04 16:16:35]

On 2016/11/04 at 05:50:49, yutak wrote:
> LGTM
> 
> In addition to what haraken said, please fix the URL in
> the change description (extra characters in the end).

Done and done!

[commit-bot: I haz the power, 2016-11-04 17:23:15]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2016-11-04 17:36:47]

Description was changed from

==========
WTF/std normalization: replace WTF::Vector::removeLast with ::pop_back

As discussed on platform-architecture-dev [0], there is interest in
normalizing the methods of WTF classes to better align with std classes.
This CL replaces removeLast() with pop_back(). There are no
functional changes.

[0]
https://groups.google.com/a/chromium.org/d/topic/platform-architecture-dev/I7...

BUG=662431
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

to

==========
WTF/std normalization: replace WTF::Vector::removeLast with ::pop_back

As discussed on platform-architecture-dev [0], there is interest in
normalizing the methods of WTF classes to better align with std classes.
This CL replaces removeLast() with pop_back(). There are no
functional changes.

[0]
https://groups.google.com/a/chromium.org/d/topic/platform-architecture-dev/I7...

BUG=662431
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2

Committed: https://crrev.com/c9e1380ae7c169501892b25f1ee82e04207e9208
Cr-Commit-Position: refs/heads/master@{#429930}
==========

[commit-bot: I haz the power, 2016-11-04 17:36:49]

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