Bug 106409

Summary: [Resource Timing] XMLHttpRequests should have initiator type 'xmlhttprequest'
Product: WebKit Reporter: James Simonsen <simonjam>
Component: New BugsAssignee: James Simonsen <simonjam>
Status: RESOLVED FIXED    
Severity: Normal CC: japhet, tonyg, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Patch
none
Patch for landing none

James Simonsen
Reported 2013-01-08 19:46:44 PST
[Resource Timing] XMLHttpRequests should have initiator type 'xmlhttprequest'
Attachments
Patch (13.15 KB, patch)
2013-01-08 19:50 PST, James Simonsen 		no flags
DetailsFormatted DiffDiff
Patch (13.28 KB, patch)
2013-01-09 13:41 PST, James Simonsen 		no flags
DetailsFormatted DiffDiff
Patch for landing (13.04 KB, patch)
2013-01-11 15:07 PST, James Simonsen 		no flags
DetailsFormatted DiffDiff
Show Obsolete (2) View All Add attachment proposed patch, testcase, etc.
James Simonsen
Comment 1 2013-01-08 19:50:11 PST
Created attachment 181825 [details] Patch
Tony Gentilcore
Comment 2 2013-01-09 10:00:58 PST
lgtm, but japhet should really review.
Alexey Proskuryakov
Comment 3 2013-01-09 10:02:23 PST
Comment on attachment 181825 [details] Patch Shouldn't all this code be behind an ifdef?
James Simonsen
Comment 4 2013-01-09 13:41:56 PST
Created attachment 181978 [details] Patch
James Simonsen
Comment 5 2013-01-09 13:42:11 PST
(In reply to comment #3) > (From update of attachment 181825 [details]) > Shouldn't all this code be behind an ifdef? Sure. Done.
James Simonsen
Comment 6 2013-01-11 14:24:00 PST
Is there anything more that needs to be done here?
James Simonsen
Comment 7 2013-01-11 15:07:28 PST
Created attachment 182429 [details] Patch for landing
WebKit Review Bot
Comment 8 2013-01-11 15:56:20 PST
Comment on attachment 182429 [details] Patch for landing Clearing flags on attachment: 182429 Committed r139513: <http://trac.webkit.org/changeset/139513>
WebKit Review Bot
Comment 9 2013-01-11 15:56:24 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.