TEMP DO NOT LAND

TEMP DO NOT LAND

patch from issue 2397563002 at patchset 1 (http://crrev.com/2397563002#ps1)

BUG=

[ssid, 2016-10-14 18:52:24]

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

[ssid, 2016-10-14 18:52:25]

few thoughts. Let me know if it helps.

https://codereview.chromium.org/2414883005/diff/20001/net/http/http_network_s...
File net/http/http_network_session.cc (right):

https://codereview.chromium.org/2414883005/diff/20001/net/http/http_network_s...
net/http/http_network_session.cc:383: base::trace_event::MemoryAllocatorDump*
dump) const {
Maybe pass ProcessMemoryDump* here, with a root name as
"net/url_request_context_X/media_X"

You can add extra id from here and create allocator dump:

name = root_name + "/http_request_X"
pmd->createDump(name);

https://codereview.chromium.org/2414883005/diff/20001/net/http/http_network_s...
net/http/http_network_session.cc:395:
normal_socket_pool_manager_->PopulateAllocatorDump(http_network_session_dump);
The code could be:

name = root_name + "/http_request_X"
dump = pmd->createDump(name);
dump->AddScalar(size);

target_name = "net/url_request_context/all_http_sessions/session_X"
target_dump = pmd->GetAllocatorDump(target_name);
if (!target_dump) {
  target_dmup = pmd->GetAllocatorDump(target_name)
  target_dump->AddScalar(size);
}
// This will take the effective_size away from the 
// "all_http_sesisons" pool and give it to the media/ dump.
pmd->AddOwnershipEdge(dump, target_dump);

[xunjieli, 2016-11-19 04:43:34]

Description was changed from

==========
TEMP DO NOT LAND

patch from issue 2397563002 at patchset 1 (http://crrev.com/2397563002#ps1)

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

to

==========
TEMP DO NOT LAND

patch from issue 2397563002 at patchset 1 (http://crrev.com/2397563002#ps1)

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

[xunjieli, 2016-11-19 04:43:34]

xunjieli@chromium.org changed reviewers:
- ssid@chromium.org