[Devtools] Reverse dependency of ResourceTreeModel and NetworkLog

[Devtools] Reverse dependency of ResourceTreeModel and NetworkLog

This patch reverses the dependency of NetworkLog depending on
ResourceTreeModel to be reversed. This is because ResourceTreeModel does
not exist if there is no DOM, but NetworkLog should exist, but with
current setup this causes problems with HAR output with service workers
because service workers end up without a NetworkLog, but since there's a
request they should have a NetworkLog object.

R=dgozman_slow,caseq,luoe
BUG=668032

[allada, 2017-01-06 01:28:10]

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

[allada, 2017-01-06 01:28:16]

PTL

[commit-bot: I haz the power, 2017-01-06 01:29:38]

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

[caseq, 2017-01-06 01:40:48]

https://codereview.chromium.org/2616023003/diff/1/third_party/WebKit/Source/d...
File third_party/WebKit/Source/devtools/front_end/sdk/NetworkLog.js (right):

https://codereview.chromium.org/2616023003/diff/1/third_party/WebKit/Source/d...
third_party/WebKit/Source/devtools/front_end/sdk/NetworkLog.js:112:
mainFrameNavigated(mainFrame) {
Let's take this one step further and make sure NetworkLog does not logically
depend on any page stuff, i.e. perhaps move this logic elsewhere?

https://codereview.chromium.org/2616023003/diff/1/third_party/WebKit/Source/d...
third_party/WebKit/Source/devtools/front_end/sdk/NetworkLog.js:142:
domContentLoaded(time) {
ditto.

https://codereview.chromium.org/2616023003/diff/1/third_party/WebKit/Source/d...
third_party/WebKit/Source/devtools/front_end/sdk/NetworkLog.js:150:
pageLoaded(time) {
ditto.

[commit-bot: I haz the power, 2017-01-06 02:57:51]

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

[commit-bot: I haz the power, 2017-01-06 02:57:52]

Dry run: This issue passed the CQ dry run.

[luoe, 2017-01-06 20:17:28]

Nice patch!  After this change, there will be NetworkLog(NL)s when they were
null before.  ConsoleModel uses NL to attach the request to a ConsoleMessage. 
Let's check that console logs/errors from service workers look fine as well.

[allada, 2017-05-15 23:21:43]

Description was changed from

==========
[Devtools] Reverse dependency of ResourceTreeModel and NetworkLog

This patch reverses the dependency of NetworkLog depending on
ResourceTreeModel to be reversed. This is because ResourceTreeModel does
not exist if there is no DOM, but NetworkLog should exist, but with
current setup this causes problems with HAR output with service workers
because service workers end up without a NetworkLog, but since there's a
request they should have a NetworkLog object.

R=dgozman_slow,caseq,luoe
BUG=668032
==========

to

==========
[Devtools] Reverse dependency of ResourceTreeModel and NetworkLog

This patch reverses the dependency of NetworkLog depending on
ResourceTreeModel to be reversed. This is because ResourceTreeModel does
not exist if there is no DOM, but NetworkLog should exist, but with
current setup this causes problems with HAR output with service workers
because service workers end up without a NetworkLog, but since there's a
request they should have a NetworkLog object.

R=dgozman_slow,caseq,luoe
BUG=668032
==========