NEW 208154
ImageBuffer should not use ImageData since this is a layering violation
https://bugs.webkit.org/show_bug.cgi?id=208154
Summary ImageBuffer should not use ImageData since this is a layering violation
Said Abou-Hallawa
Reported 2020-02-24 13:25:02 PST
ImageBuffer::getImageData() and ImageBuffer::puttImageData() deal with ImageData. Since ImageBuffer is in platform/graphics and ImageData is in html, this is considered a layering violation. The fix could be the following: 1. Move ImageData to platform/graphics except the DOM ImageData creation methods. 2. Add DOMImageData to html and make it inherit from ImageData. 3. Add the DOM ImageData creation methods to DOMImageData. 4. Rename ImageData.idl to DOMImageData.idl but make InterfaceName=ImageData. 5. Change all the DOM related reference to ImageData to be DOMImageData.
Attachments
Patch (72.46 KB, patch)
2020-02-25 18:32 PST, Said Abou-Hallawa
no flags
Patch (70.84 KB, patch)
2020-02-25 18:34 PST, Said Abou-Hallawa
no flags
Said Abou-Hallawa
Comment 1 2020-02-25 18:32:08 PST
Said Abou-Hallawa
Comment 2 2020-02-25 18:34:27 PST
Note You need to log in before you can comment on or make changes to this bug.