DevTools: ResourceTreeModel should not report resources twice

DevTools: ResourceTreeModel should not report resources twice

Today, if one loads the page 'http://example.com', devtools will
lack the main resource and thus will create one out of frame pyaload.

However, in case of loading 'http://example.com/index.html', this is
not necessary and results in reporting main resource twice.

This patch fixes the issue, creating a fake resource only if it
is not reported among frame resources.

R=dgozman
BUG=670180
Review-Url: https://codereview.chromium.org/2897713002
Cr-Commit-Position: refs/heads/master@{#473307}
Committed: https://chromium.googlesource.com/chromium/src/+/ad816ae2af03119667c954d52cfcac88d084cf1a

[lushnikov, 2017-05-19 18:37:08]

Description was changed from

==========
DevTools: ResourceTreeModel should not report resources twice

Today, in case of loading page 'http://example.com', the devtools will
lack the main resource and thus will create a 'fake' resource out of
frame pyaload.

However, in case of loading 'http://example.com/index.html' this is
not necessary and results in reporting a 'dangling' resource.

This patch fixes the issue, creating a fake resource only if it
is not reported among frame resources.

R=dgozman
BUG=670180
==========

to

==========
DevTools: ResourceTreeModel should not report resources twice

Today, in one loads page 'http://example.com', the devtools will
lack the main resource and thus will create one out of frame pyaload.

However, in case of loading 'http://example.com/index.html', this is
not necessary and results in reporting main resource twice.

This patch fixes the issue, creating a fake resource only if it
is not reported among frame resources.

R=dgozman
BUG=670180
==========

[lushnikov, 2017-05-19 18:37:37]

please, take a look

[lushnikov, 2017-05-19 18:38:04]

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

[commit-bot: I haz the power, 2017-05-19 18:39:06]

Dry run: CQ is trying da patch.

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

[dgozman, 2017-05-19 18:40:51]

Description was changed from

==========
DevTools: ResourceTreeModel should not report resources twice

Today, in one loads page 'http://example.com', the devtools will
lack the main resource and thus will create one out of frame pyaload.

However, in case of loading 'http://example.com/index.html', this is
not necessary and results in reporting main resource twice.

This patch fixes the issue, creating a fake resource only if it
is not reported among frame resources.

R=dgozman
BUG=670180
==========

to

==========
DevTools: ResourceTreeModel should not report resources twice

Today, if one loads the page 'http://example.com', devtools will
lack the main resource and thus will create one out of frame pyaload.

However, in case of loading 'http://example.com/index.html', this is
not necessary and results in reporting main resource twice.

This patch fixes the issue, creating a fake resource only if it
is not reported among frame resources.

R=dgozman
BUG=670180
==========

[dgozman, 2017-05-19 18:46:01]

lgtm

[lushnikov, 2017-05-19 18:56:25]

The CQ bit was unchecked by lushnikov@chromium.org

[lushnikov, 2017-05-19 18:56:26]

The CQ bit was checked by lushnikov@chromium.org

[commit-bot: I haz the power, 2017-05-19 18:56:50]

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 20:54:28]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1495220186137830, "parent_rev":
"7b707c0865f329eb498a7634aaf88756f5a3449a", "commit_rev":
"ad816ae2af03119667c954d52cfcac88d084cf1a"}

[commit-bot: I haz the power, 2017-05-19 20:54:37]

Description was changed from

==========
DevTools: ResourceTreeModel should not report resources twice

Today, if one loads the page 'http://example.com', devtools will
lack the main resource and thus will create one out of frame pyaload.

However, in case of loading 'http://example.com/index.html', this is
not necessary and results in reporting main resource twice.

This patch fixes the issue, creating a fake resource only if it
is not reported among frame resources.

R=dgozman
BUG=670180
==========

to

==========
DevTools: ResourceTreeModel should not report resources twice

Today, if one loads the page 'http://example.com', devtools will
lack the main resource and thus will create one out of frame pyaload.

However, in case of loading 'http://example.com/index.html', this is
not necessary and results in reporting main resource twice.

This patch fixes the issue, creating a fake resource only if it
is not reported among frame resources.

R=dgozman
BUG=670180

Review-Url: https://codereview.chromium.org/2897713002
Cr-Commit-Position: refs/heads/master@{#473307}
Committed:
https://chromium.googlesource.com/chromium/src/+/ad816ae2af03119667c954d52cfc...
==========

[commit-bot: I haz the power, 2017-05-19 20:54:38]

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