Closed
Bug 1840847
Opened 1 year ago
Closed 1 year ago
Set up JSWindowActor messaging
Categories
(Firefox :: Shopping, task, P1)
Firefox
Shopping
Tracking
()
RESOLVED
FIXED
117 Branch
Tracking | Status | |
---|---|---|
firefox117 | --- | fixed |
People
(Reporter: jhirsch, Assigned: jhirsch)
References
(Blocks 2 open bugs)
Details
(Whiteboard: [fidefe-shopping])
Attachments
(1 file)
(deleted),
text/x-phabricator-request
|
Details |
In bug 1838438, we hacked together a sidebar for shopping, where the parent context communicated the product URL down to the sidebar by settings the src
on its browser
element.
Let's update the code that loads the shopping page to pass down the product URL using Actor messaging instead, as usual for internal components. We will file a separate bug to change the init flow from within the page.
Assignee | ||
Updated•1 year ago
|
Assignee | ||
Updated•1 year ago
|
Whiteboard: [fidefe-shopping]
Updated•1 year ago
|
Assignee | ||
Comment 1•1 year ago
|
||
Picking this up now, as we will need this to relay the pref flips (from clicking the settings UI) up to the parent window. (I think the OMC team might also be hoping to use this to send signals up from content to parent.)
Assignee: nobody → jhirsch
Status: NEW → ASSIGNED
Priority: P2 → P1
Assignee | ||
Updated•1 year ago
|
Summary: Use Actor messaging to send product URL down to sidebar → Set up JSWindowActor messaging
Assignee | ||
Comment 2•1 year ago
|
||
Also closes bug 1844101 and bug 1844097.
Pushed by jhirsch@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/328b5b27b469
Switch shopping sidebar to use Actors for product URL messaging. r=Gijs
Comment 4•1 year ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 1 year ago
status-firefox117:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 117 Branch
You need to log in
before you can comment on or make changes to this bug.
Description
•