Bug 51127 – Web Inspector: timeline records should only operate WebCore time.

• Status: RESOLVED
• Resolution: FIXED
• Component: Web Inspector (Deprecated)

[Pavel Feldman]

Network timing and such should belong to the data / aux properties. Primary timing info in timeline should be taken by means of currentTime in WebCore.

[Pavel Feldman]

Created attachment 76679 [details]
[PATCH] Proposed change

[Yury Semikhatsky]

Comment on attachment 76679 [details]
[PATCH] Proposed change

>     // Sometimes network stack can provide for us exact finish loading time. In the other case we will use currentTime.
The comment should be removed too. But at the first place what's wrong with using the time provided by the network stack?

(In reply to comment #2)
> (From update of attachment 76679 [details])
> >     // Sometimes network stack can provide for us exact finish loading time. In the other case we will use currentTime.
> The comment should be removed too. But at the first place what's wrong with using the time provided by the network stack?

You can (and actually do in practice) end up with the time reported in the finish being well before the time reported by the response received event.

In general, the timeline agent has a contract that it records times in WebCore on the UI thread. It has strict ordering guarantees, and the timestamps should follow this.

Also, the network stack and resource loader will be out of process in chrome, so the webcore callbacks get invoked after some Async IPC action. So the common case will be this time wont make sense when combined with the responsetime taken in webcore on the UI thread.

Basically negative time deltas (end  - responseReceived) is incorrect for the timeline agent.

[WebKit Commit Bot]

Comment on attachment 76679 [details]
[PATCH] Proposed change

Rejecting attachment 76679 [details] from commit-queue.

Failed to run "['./WebKitTools/Scripts/webkit-patch', '--status-host=queues.webkit.org', '--bot-id=cr-jail-3', 'apply-attachment', '--non-interactive', 76679]" exit_code: 2
Last 500 characters of output:
ailed to merge in the changes.
Patch failed at 0001 2010-12-16  Yury Semikhatsky  <[e-mail]>

When you have resolved this problem run "git rebase --continue".
If you would prefer to skip this patch, instead run "git rebase --skip".
To restore the original branch and stop rebasing run "git rebase --abort".

rebase refs/remotes/origin/master: command returned error: 1

Died at WebKitTools/Scripts/update-webkit line 132.

Failed to run "['WebKitTools/Scripts/update-webkit']" exit_code: 2

Full output: http://queues.webkit.org/results/7131071

[Pavel Feldman]

Comment on attachment 76679 [details]
[PATCH] Proposed change

Clearing flags on attachment: 76679

Committed r74198: <http://trac.webkit.org/changeset/74198>

[Pavel Feldman]

All reviewed patches have been landed.  Closing bug.

[WebKit Review Bot]

http://trac.webkit.org/changeset/74198 might have broken Leopard Intel Debug (Tests)

[WebKit Review Bot]

http://trac.webkit.org/changeset/74199 might have broken Leopard Intel Debug (Tests)