Closed
Bug 1006575
Opened 11 years ago
Closed 10 years ago
[ResourceTiming] Redirected iframe inserts 2 entries into buffer
Categories
(Core :: DOM: Core & HTML, defect)
Tracking
()
RESOLVED
FIXED
mozilla32
People
(Reporter: valentin, Assigned: valentin)
References
Details
Attachments
(1 file)
(deleted),
patch
|
mcmanus
:
review+
|
Details | Diff | Splinter Review |
On a page with
<iframe sandbox="allow-scripts" src="http://test2.com/index2.html"></iframe>
where the page is redirected with a 301 to another domain, there are 2 resource timing entries with the same name.
This does not happen with a redirected img tag.
Assignee | ||
Comment 1•10 years ago
|
||
Assignee | ||
Updated•10 years ago
|
Assignee: nobody → valentin.gosu
Status: NEW → ASSIGNED
Assignee | ||
Comment 2•10 years ago
|
||
Comment on attachment 8421454 [details] [diff] [review]
Redirected iframe inserts 2 entries into resource timing buffer
Review of attachment 8421454 [details] [diff] [review]:
-----------------------------------------------------------------
This patch sets the mTimingEnabled flag to false for the old channel, or else there will be 2 resources with the same name.
Also passing the initiatorType from the old channel to the new one.
Attachment #8421454 -
Flags: review?(honzab.moz)
Assignee | ||
Comment 3•10 years ago
|
||
Comment on attachment 8421454 [details] [diff] [review]
Redirected iframe inserts 2 entries into resource timing buffer
Review of attachment 8421454 [details] [diff] [review]:
-----------------------------------------------------------------
Note: setting the flag is only necessary because the iframe ends up adding an entry to the buffer, even if it was redirected.
This only happens for the first load of the page because of bug 1009360
Attachment #8421454 -
Flags: review?(honzab.moz) → review?(mcmanus)
Updated•10 years ago
|
Attachment #8421454 -
Flags: review?(mcmanus) → review+
Assignee | ||
Comment 4•10 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla32
Updated•6 years ago
|
Component: DOM → DOM: Core & HTML
You need to log in
before you can comment on or make changes to this bug.
Description
•