Closed
Bug 1209416
Opened 9 years ago
Closed 9 years ago
Sanitizer.js should be able to unwrap multiple "safe html" objects
Categories
(Firefox OS Graveyard :: Gaia::Shared, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: freddy, Unassigned)
References
Details
Attachments
(1 file)
> // currently allowed:
> foo.innerHTML = unwrapSafeHTML(foo);
> // now also allowed
> foo.innerHTML = unwrapSafeHTML(foo, bar);
|
||
Comment 1•9 years ago
|
||
|
Reporter | |
Comment 2•9 years ago
|
||
Comment on attachment 8667134 [details] [review] [gaia] mozfreddyb:sanitizer-unwrap-multiple > mozilla-b2g:master Kevin, can you take a look? This is a very small change.
Attachment #8667134 -
Flags: review?(kevingrandon)
|
||
Comment 3•9 years ago
|
||
Comment on attachment 8667134 [details] [review] [gaia] mozfreddyb:sanitizer-unwrap-multiple > mozilla-b2g:master Looks fine, thanks!
Attachment #8667134 -
Flags: review?(kevingrandon) → review+
|
|
Reporter | |
Updated•9 years ago
|
Keywords: checkin-needed
|
||
Comment 4•9 years ago
|
||
https://github.com/mozilla-b2g/gaia/commit/8eeeeaa6073b6948be4dd1ced812f4f2c502f757
You need to log in
before you can comment on or make changes to this bug.
Description
•