Bug 48924

Summary: [Web Timing] Remove requestEnd
Product: WebKit Reporter: Tony Gentilcore <tonyg>
Component: WebCore Misc.Assignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue, dglazkov, simonjam
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
URL: http://test.w3.org/webperf/specs/NavigationTiming/
Bug Depends on:    
Bug Blocks: 30685    
Attachments:
Description Flags
Patch none

Tony Gentilcore
Reported 2010-11-03 10:01:03 PDT
requestEnd has been removed from the spec since it is now equal to responseStart.
Attachments
Patch (11.77 KB, patch)
2010-11-08 17:05 PST, James Simonsen 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
James Simonsen
Comment 1 2010-11-08 17:05:50 PST
Created attachment 73310 [details] Patch
Tony Gentilcore
Comment 2 2010-11-08 17:17:04 PST
LGTM!
Dimitri Glazkov (Google)
Comment 3 2010-11-08 18:58:29 PST
Comment on attachment 73310 [details] Patch ok.
WebKit Commit Bot
Comment 4 2010-11-08 21:46:48 PST
Comment on attachment 73310 [details] Patch Rejecting patch 73310 from commit-queue. Failed to run "['./WebKitTools/Scripts/webkit-patch', '--status-host=queues.webkit.org', '--bot-id=abarth-cq-sl', 'apply-attachment', '--force-clean', '--non-interactive', 73310]" exit_code: 1 Traceback (most recent call last): File "./WebKitTools/Scripts/webkit-patch", line 70, in <module> main() File "./WebKitTools/Scripts/webkit-patch", line 63, in main from webkitpy.tool.main import WebKitPatch File "/Users/abarth/git/webkit-queue/WebKitTools/Scripts/webkitpy/tool/main.py", line 42, in <module> from webkitpy.common.net.rietveld import Rietveld ImportError: No module named rietveld Full output: http://queues.webkit.org/results/5581007
WebKit Commit Bot
Comment 5 2010-11-09 00:22:50 PST
Comment on attachment 73310 [details] Patch Clearing flags on attachment: 73310 Committed r71612: <http://trac.webkit.org/changeset/71612>
WebKit Commit Bot
Comment 6 2010-11-09 00:22:55 PST
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.