Closed Bug 1844178 Opened 1 year ago Closed 1 year ago

Update ShoppingProduct constructor to take a nsIURI

Categories

(Firefox :: Shopping, task)

task
Points:
1

Tracking

()

RESOLVED FIXED
117 Branch
Tracking Status
firefox117 --- fixed

People

(Reporter: fchasen, Assigned: fchasen)

References

(Blocks 1 open bug)

Details

(Whiteboard: [fidefe-shopping] )

Attachments

(1 file)

Android team wants to pass an nsIURI directly into new ShoppingProduct(uri) this, same as they do isProductURL.

Adding the URL.fromURI to the constructor should allow that.

Updates the ShoppingProduct constructor to be consistent with isProductURL by checking if a nsIURI is passed and converting it to a URL if needed.

Pushed by fchasen@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/163c85e5903a Update ShoppingProduct constructor to take a nsIURI r=calu,shopping-reviewers,Gijs
Status: ASSIGNED → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → 117 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: