Closed
Bug 1659589
Opened 2 years ago
Closed 2 years ago
Get rid of a dependant of stylesheet actor from StyleEditor (to resource based)
Categories
(DevTools :: Style Editor, task)
DevTools
Style Editor
Tracking
(Fission Milestone:M6c, firefox82 fixed)
Tracking | Status | |
---|---|---|
firefox82 | --- | fixed |
People
(Reporter: daisuke, Assigned: daisuke)
References
(Blocks 1 open bug, Regressed 1 open bug)
Details
(Whiteboard: dt-fission-m2-mvp)
Attachments
(8 files, 1 obsolete file)
47 bytes,
text/x-phabricator-request
|
Details | Review | |
47 bytes,
text/x-phabricator-request
|
Details | Review | |
47 bytes,
text/x-phabricator-request
|
Details | Review | |
47 bytes,
text/x-phabricator-request
|
Details | Review | |
47 bytes,
text/x-phabricator-request
|
Details | Review | |
47 bytes,
text/x-phabricator-request
|
Details | Review | |
47 bytes,
text/x-phabricator-request
|
Details | Review | |
47 bytes,
text/x-phabricator-request
|
Details | Review |
Now, we watch/call stylesheet
actor directly in StyleEditor.
However, in order to implement stylesheet watcher in the server-side, to get rid of a dependant of stylesheet actor is the desired way.
Instead, we change StyleEditor to resource-based.
Assignee | ||
Comment 1•2 years ago
|
||
Assignee | ||
Comment 2•2 years ago
|
||
Depends on D87040
Assignee | ||
Comment 3•2 years ago
|
||
Depends on D87041
Assignee | ||
Comment 4•2 years ago
|
||
Depends on D87042
Assignee | ||
Comment 5•2 years ago
|
||
Depends on D87043
Assignee | ||
Comment 6•2 years ago
|
||
Depends on D87044
Assignee | ||
Comment 7•2 years ago
|
||
Depends on D87045
Assignee | ||
Comment 8•2 years ago
|
||
Depends on D87046
Comment 9•2 years ago
|
||
Hello Daisuke. Should I add this bug as an MVP for Fission M2?
Flags: needinfo?(daisuke)
Updated•2 years ago
|
Whiteboard: dt-fission-m2-mvp
Updated•2 years ago
|
Fission Milestone: --- → M6c
Updated•2 years ago
|
Attachment #9170522 -
Attachment is obsolete: true
Assignee | ||
Updated•2 years ago
|
Keywords: leave-open
Comment 11•2 years ago
|
||
Pushed by dakatsuka.birchill@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/9a55dbee660a Handle importing stylesheet from file using resource watcher mechanism to keep consistency. r=ochameau,devtools-backward-compat-reviewers,jdescottes https://hg.mozilla.org/integration/autoland/rev/5d2a954c65cc Get style-applied event via resource watcher. r=ochameau,bomsy https://hg.mozilla.org/integration/autoland/rev/b98fe620038d Get property-change event via resource watcher. r=ochameau https://hg.mozilla.org/integration/autoland/rev/5ce60ce82c52 Get media-rules-changed event via resource watcher. r=ochameau
Comment 12•2 years ago
|
||
bugherder |
Assignee | ||
Comment 13•2 years ago
|
||
Depends on D87047
Assignee | ||
Updated•2 years ago
|
Keywords: leave-open
Comment 14•2 years ago
|
||
Pushed by dakatsuka.birchill@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/bc075afd9fdf Implement toggleDisabled in stylesheets actor. r=ochameau,devtools-backward-compat-reviewers https://hg.mozilla.org/integration/autoland/rev/e8258610951f Implement getText in stylesheets actor. r=ochameau,devtools-backward-compat-reviewers https://hg.mozilla.org/integration/autoland/rev/ad6708adb4cc Implement update method in stylesheets actor. r=ochameau,devtools-backward-compat-reviewers https://hg.mozilla.org/integration/autoland/rev/e1242e04253e Remove error event listener that set to stylesheet actor. r=ochameau
Comment 15•2 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/bc075afd9fdf
https://hg.mozilla.org/mozilla-central/rev/e8258610951f
https://hg.mozilla.org/mozilla-central/rev/ad6708adb4cc
https://hg.mozilla.org/mozilla-central/rev/e1242e04253e
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
status-firefox82:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 82 Branch
You need to log in
before you can comment on or make changes to this bug.
Description
•