Cleanup ResourceDispatcher::PendingRequestInfo

Cleanup ResourceDispatcher::PendingRequestInfo

Some preliminary cleanups for RequestPeer lifetime fix.
- Changes hash_map<> to map<scoped_ptr<>>, remove default ctor
- Replaces some linked_ptr with scoped_ptr (crbug.com/556939)

BUG=507170, 556939
Committed: https://crrev.com/b946dc7afe1ad7a11a423d1519ad4f880b62a44c
Cr-Commit-Position: refs/heads/master@{#367782}

[kinuko, 2016-01-05 14:05:55]

Description was changed from

==========
Cleanup ResourceDispatcher::PendingRequestInfo

Some preliminary cleanups for RequestPeer lifetime fix.
- Changes hash_map<> to map<scoped_ptr<>>, remove default ctor
- Replaces some linked_ptr with scoped_ptr (crbug.com/556939)

BUG=507170, 556939
==========

to

==========
Cleanup ResourceDispatcher::PendingRequestInfo

Some preliminary cleanups for RequestPeer lifetime fix.
- Changes hash_map<> to map<scoped_ptr<>>, remove default ctor
- Replaces some linked_ptr with scoped_ptr (crbug.com/556939)

BUG=507170, 556939
==========

[kinuko, 2016-01-05 14:05:55]

kinuko@chromium.org changed reviewers:
+ yhirano@chromium.org

[kinuko, 2016-01-05 14:06:05]

PTL

[yhirano, 2016-01-06 04:07:48]

LGTM

https://codereview.chromium.org/1561563002/diff/1/content/child/resource_disp...
File content/child/resource_dispatcher.cc (right):

https://codereview.chromium.org/1561563002/diff/1/content/child/resource_disp...
content/child/resource_dispatcher.cc:458: const PendingRequestInfo& info =
*it->second.get();
Is ".get()" needed?

[kinuko, 2016-01-06 04:18:29]

Thanks!

https://codereview.chromium.org/1561563002/diff/1/content/child/resource_disp...
File content/child/resource_dispatcher.cc (right):

https://codereview.chromium.org/1561563002/diff/1/content/child/resource_disp...
content/child/resource_dispatcher.cc:458: const PendingRequestInfo& info =
*it->second.get();
On 2016/01/06 04:07:48, yhirano wrote:
> Is ".get()" needed?

Likely not, removed.

[kinuko, 2016-01-06 04:18:32]

The CQ bit was checked by kinuko@chromium.org

[kinuko, 2016-01-06 04:18:33]

The patchset sent to the CQ was uploaded after l-g-t-m from yhirano@chromium.org
Link to the patchset: https://codereview.chromium.org/1561563002/#ps20001
(title: "review fix")

[commit-bot: I haz the power, 2016-01-06 04:19:01]

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

[commit-bot: I haz the power, 2016-01-06 05:33:24]

Description was changed from

==========
Cleanup ResourceDispatcher::PendingRequestInfo

Some preliminary cleanups for RequestPeer lifetime fix.
- Changes hash_map<> to map<scoped_ptr<>>, remove default ctor
- Replaces some linked_ptr with scoped_ptr (crbug.com/556939)

BUG=507170, 556939
==========

to

==========
Cleanup ResourceDispatcher::PendingRequestInfo

Some preliminary cleanups for RequestPeer lifetime fix.
- Changes hash_map<> to map<scoped_ptr<>>, remove default ctor
- Replaces some linked_ptr with scoped_ptr (crbug.com/556939)

BUG=507170, 556939
==========

[commit-bot: I haz the power, 2016-01-06 05:33:25]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-01-06 05:34:10]

Description was changed from

==========
Cleanup ResourceDispatcher::PendingRequestInfo

Some preliminary cleanups for RequestPeer lifetime fix.
- Changes hash_map<> to map<scoped_ptr<>>, remove default ctor
- Replaces some linked_ptr with scoped_ptr (crbug.com/556939)

BUG=507170, 556939
==========

to

==========
Cleanup ResourceDispatcher::PendingRequestInfo

Some preliminary cleanups for RequestPeer lifetime fix.
- Changes hash_map<> to map<scoped_ptr<>>, remove default ctor
- Replaces some linked_ptr with scoped_ptr (crbug.com/556939)

BUG=507170, 556939

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

[commit-bot: I haz the power, 2016-01-06 05:34:11]

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