Closed Bug 1733559 Opened 2 years ago Closed 1 year ago

Support resizeWidth and resizeHeight options in createImageBitmap()

Categories

(Core :: Graphics: Canvas2D, enhancement)

Product:
Core
Component:
Graphics: Canvas2D
Version:
Firefox 93
Type:
enhancement

Tracking

()

Status:
RESOLVED FIXED
Milestone:
98 Branch
Tracking Flags:
Tracking Status
firefox98 --- fixed

People

(Reporter: angus.sawyer, Assigned: angus.sawyer)

References

(Blocks 1 open bug)

Details

(Keywords: dev-doc-complete)

Attachments

(1 file)

Bug 1733559 - createImageBitmap() resizeWidth and resizeHeight support, r=aosmond
48 bytes, text/x-phabricator-request
Details | Review

User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/94.0.4606.61 Safari/537.36

Steps to reproduce:

wpt test failures in (https://wpt.live/html/canvas/element/manual/imagebitmap/createImageBitmap-drawImage.html)

Actual results:

no resizing takes place

Expected results:

images should be resized

Also fixed negative cropping width/height for blob images.
Removes two unused class variables.

Assignee: nobody → angus.sawyer
Blocks: 1732681
Status: UNCONFIRMED → NEW
Type: defect → enhancement
Component: Untriaged → Canvas: 2D
Ever confirmed: true
Product: Firefox → Core
Summary: createImageBitmap() does not support the resizeWidth and resizeHeight options. → Support resizeWidth and resizeHeight options in createImageBitmap()

There's a r+ patch which didn't land and no activity in this bug for 2 weeks.
:angus.sawyer, could you have a look please?
For more information, please visit auto_nag documentation.

Flags: needinfo?(aosmond)
Flags: needinfo?(angus.sawyer)
Pushed by aosmond@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/563424398bd2
createImageBitmap() resizeWidth and resizeHeight support, r=aosmond,emilio

https://hg.mozilla.org/mozilla-central/rev/563424398bd2

Status: NEW → RESOLVED
Closed: 1 year ago
status-firefox98: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 98 Branch

(In reply to Release mgmt bot [:marco/ :calixte] from comment #2)

There's a r+ patch which didn't land and no activity in this bug for 2 weeks.
:angus.sawyer, could you have a look please?
For more information, please visit auto_nag documentation.

Flags: needinfo?(aosmond)
Flags: needinfo?(angus.sawyer)
Keywords: dev-doc-needed
Blocks: 1363861

See last comments for Bug 1748868 re update of documentation required.

You need to log in before you can comment on or make changes to this bug.