Use emplace_back when adding an item to WebSocket stalled requests

Use emplace_back when adding an item to WebSocket stalled requests

This is a tiny readability improvement, and removes an old TODO.

By using emplace_back() when adding an item to stalled_request_queue_ in
net::WebSocketTransportClientSocketPool we remove an unnecessary local variable
declaration. There's probably a minor performance improvement too.

BUG=

Review-Url: https://codereview.chromium.org/2847073002
Cr-Commit-Position: refs/heads/master@{#473521}
Committed: https://chromium.googlesource.com/chromium/src/+/00bd27b320be81e0cb51d5c9a017d97449c5a5fa

[Adam Rice, 2017-04-28 11:53:33]

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

[commit-bot: I haz the power, 2017-04-28 11:53:47]

Dry run: CQ is trying da patch.

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

[Adam Rice, 2017-04-28 11:53:58]

ricea@chromium.org changed reviewers:
+ tyoshino@chromium.org

[Adam Rice, 2017-04-28 11:53:58]

Very low priority.

[commit-bot: I haz the power, 2017-04-28 12:52:59]

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

[commit-bot: I haz the power, 2017-04-28 12:53:00]

Dry run: This issue passed the CQ dry run.

[tyoshino (SeeGerritForStatus), 2017-05-12 10:21:20]

lgtm :)

[Adam Rice, 2017-05-19 06:36:51]

The CQ bit was checked by ricea@chromium.org

[commit-bot: I haz the power, 2017-05-19 06:37:32]

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-05-19 07:47:32]

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

[commit-bot: I haz the power, 2017-05-19 07:47:33]

Try jobs failed on following builders:
  win_chromium_x64_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)

[Adam Rice, 2017-05-22 05:48:19]

The CQ bit was checked by ricea@chromium.org

[commit-bot: I haz the power, 2017-05-22 05:48:33]

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-05-22 07:36:33]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1495432099252890, "parent_rev":
"176bf996a20bed1540ac1b2ef01079ee35c3e7b8", "commit_rev":
"00bd27b320be81e0cb51d5c9a017d97449c5a5fa"}

[commit-bot: I haz the power, 2017-05-22 07:36:46]

Description was changed from

==========
Use emplace_back when adding an item to WebSocket stalled requests

This is a tiny readability improvement, and removes an old TODO.

By using emplace_back() when adding an item to stalled_request_queue_ in
net::WebSocketTransportClientSocketPool we remove an unnecessary local variable
declaration. There's probably a minor performance improvement too.

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

to

==========
Use emplace_back when adding an item to WebSocket stalled requests

This is a tiny readability improvement, and removes an old TODO.

By using emplace_back() when adding an item to stalled_request_queue_ in
net::WebSocketTransportClientSocketPool we remove an unnecessary local variable
declaration. There's probably a minor performance improvement too.

BUG=

Review-Url: https://codereview.chromium.org/2847073002
Cr-Commit-Position: refs/heads/master@{#473521}
Committed:
https://chromium.googlesource.com/chromium/src/+/00bd27b320be81e0cb51d5c9a017...
==========

[commit-bot: I haz the power, 2017-05-22 07:36:47]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/00bd27b320be81e0cb51d5c9a017...