Bug 31082 - Web Inspector - Adds markTimeline() API to the console for annotating the timeline tree
Summary: Web Inspector - Adds markTimeline() API to the console for annotating the tim...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Web Inspector (Deprecated) (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC OS X 10.5
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-11-03 13:58 PST by jaimeyap
Modified: 2009-11-04 00:34 PST (History)
4 users (show)

See Also:


Attachments
Patch to add console.markTimeline() (7.84 KB, patch)
2009-11-03 14:02 PST, jaimeyap
pfeldman: review-
Details | Formatted Diff | Diff
Fixes indenting in previous patch (7.84 KB, patch)
2009-11-03 14:24 PST, jaimeyap
pfeldman: review+
commit-queue: commit-queue-
Details | Formatted Diff | Diff
updates layout test for window properties (8.50 KB, patch)
2009-11-03 15:14 PST, jaimeyap
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description jaimeyap 2009-11-03 13:58:57 PST
This patch adds console.markTimeline() for annotating the timeline tree. This makes it easy for developers to mark application level checkpoints and have them overlaid on the timeline trace tree.

With regards to dealing with calls to this in a loop, I am leaning towards potentially doing message aggregation in the inspector UI.

(patch to be appended to this bug shortly)
Comment 1 jaimeyap 2009-11-03 14:02:37 PST
Created attachment 42415 [details]
Patch to add console.markTimeline()
Comment 2 Pavel Feldman 2009-11-03 14:18:20 PST
Comment on attachment 42415 [details]
Patch to add console.markTimeline()

> +{
> +    if (timelineAgent())
> +      timelineAgent()->didMarkTimeline(message);

wrong indent

> +        MarkTimelineRecordType = 11,

will be a conflict with resources events, who gets r+ first? :)

> +    page->inspectorController()->markTimeline(message);

we could pass the stack and try doing .log-alike the formatting, but that is too hard.

r- for indent. otherwise good.
Comment 3 jaimeyap 2009-11-03 14:24:30 PST
Created attachment 42419 [details]
Fixes indenting in previous patch
Comment 4 WebKit Commit Bot 2009-11-03 14:58:42 PST
Comment on attachment 42419 [details]
Fixes indenting in previous patch

Rejecting patch 42419 from commit-queue.

Failed to run "['WebKitTools/Scripts/run-webkit-tests', '--no-launch-safari', '--quiet', '--exit-after-n-failures=1']" exit_code: 1
Running build-dumprendertree
Running tests from /Users/eseidel/Projects/CommitQueue/LayoutTests
Testing 11563 test cases.
fast/dom/Window/window-properties.html -> failed

Exiting early after 1 failures. 5840 tests run.
107.16s total testing time

5839 test cases (99%) succeeded
1 test case (<1%) had incorrect layout
1 test case (<1%) had stderr output
Comment 5 jaimeyap 2009-11-03 15:14:21 PST
Created attachment 42427 [details]
updates layout test for window properties

Forgot to update a test case. Updated now with this patch.
Comment 6 WebKit Commit Bot 2009-11-04 00:33:57 PST
Comment on attachment 42427 [details]
updates layout test for window properties

Clearing flags on attachment: 42427

Committed r50501: <http://trac.webkit.org/changeset/50501>
Comment 7 WebKit Commit Bot 2009-11-04 00:34:01 PST
All reviewed patches have been landed.  Closing bug.