Bug 46276

Summary: Web Inspector: export raw data from Timeline panel.
Product: WebKit Reporter: Ilya Tikhonovsky <loislo>
Component: Web Inspector (Deprecated)Assignee: Ilya Tikhonovsky <loislo>
Status: RESOLVED DUPLICATE    
Severity: Normal CC: bweinstein, hallettj, jmalonzo, joepeck, keishi, loislo, pfeldman, pmuellr, rik, yurys
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Attachments:
Description Flags
screenshot of the button. Need a UI guy for it.
none
[patch] initial version. pfeldman: review-

Description Ilya Tikhonovsky 2010-09-22 10:15:42 PDT
http://www.chromeboard.com/showthread.php?p=114349
Comment 1 Ilya Tikhonovsky 2010-09-22 10:19:29 PDT
Created attachment 68388 [details]
screenshot of the button. Need a UI guy for it.
Comment 2 Ilya Tikhonovsky 2010-09-22 10:21:05 PDT
Created attachment 68390 [details]
[patch] initial version.

has binary data.
Need a design for 'export raw data' button.
Comment 3 Yury Semikhatsky 2010-09-23 00:24:12 PDT
Comment on attachment 68390 [details]
[patch] initial version.

View in context: https://bugs.webkit.org/attachment.cgi?id=68390&action=review

> WebCore/inspector/front-end/TimelinePanel.js:88
> +    this._exportToFileTitleText = WebInspector.UIString("Export to file");

Why not just use it inplace?

> WebCore/inspector/front-end/TimelinePanel.js:297
> +        objectForExport.version = /AppleWebKit\/([^ ]+)/.exec(window.navigator.userAgent)[0];

What is the version format? Is it consistent between browsers?
Comment 4 Pavel Feldman 2010-09-23 01:26:36 PDT
Comment on attachment 68390 [details]
[patch] initial version.

View in context: https://bugs.webkit.org/attachment.cgi?id=68390&action=review

I am against adding this button until the format being exported is documented. At least on the Web Inspector wiki. If you'd like to push this functionality faster, make Ctrl+C export the data into the clipboard.

>> WebCore/inspector/front-end/TimelinePanel.js:297
>> +        objectForExport.version = /AppleWebKit\/([^ ]+)/.exec(window.navigator.userAgent)[0];
> 
> What is the version format? Is it consistent between browsers?

We need a format version here, not the webkit version.

> WebCore/inspector/front-end/inspector.css:3762
> +    -webkit-mask-image: url(Images/exportButtonGlyph.png);

Need a better icon prior to landing.
Comment 5 Pavel Feldman 2010-09-23 06:37:40 PDT
*** Bug 46246 has been marked as a duplicate of this bug. ***
Comment 6 Ilya Tikhonovsky 2011-08-03 08:28:14 PDT

*** This bug has been marked as a duplicate of bug 64601 ***