Closed Bug 1364369 Opened 7 years ago Closed 7 years ago

update more files to use srcdoc attr for iframes instead of data: in mochitests under dom/

Categories

(Core :: DOM: Security, enhancement, P2)

Product:
Core
Component:
DOM: Security
Type:
enhancement

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla55
Tracking Flags:
Tracking Status
firefox55 --- fixed

People

(Reporter: allstars.chh, Assigned: allstars.chh)

References

Details

(Whiteboard: [domsecurity-active])

Attachments

(1 file)

Patch.
16.61 KB, patch
smaug
: review+
Details | Diff | Splinter Review 
+++ This bug was initially created as a clone of Bug #1361641 +++

found more tests can be updated by simply using srcdoc.
Hi smaug
Sorry ther failures are too many, I missed some of them when I fixed Bug 1361641, would you mind reviewing this again?

Hopefully I've catched them all in dom/. finger crossed.

Thanks
Flags: needinfo?(bugs)
Comment on attachment 8867107 [details] [diff] [review]
Patch.

Need to think if we need some more tests for data: urls.
Attachment #8867107 - Flags: review+
(In reply to Olli Pettay [:smaug] from comment #3)
> Comment on attachment 8867107 [details] [diff] [review]
> Patch.
> 
> Need to think if we need some more tests for data: urls.

Bug 1365157.
Flags: needinfo?(bugs)
Pushed by yhuang@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/f7e2f109410e
update more files to use srcdoc attr. r=smaug
https://hg.mozilla.org/mozilla-central/rev/f7e2f109410e
Status: NEW → RESOLVED
Closed: 7 years ago
status-firefox55: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: