Closed
Bug 1659589
Opened 5 years ago
Closed 5 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•5 years ago
|
||
Assignee | ||
Comment 2•5 years ago
|
||
Depends on D87040
Assignee | ||
Comment 3•5 years ago
|
||
Depends on D87041
Assignee | ||
Comment 4•5 years ago
|
||
Depends on D87042
Assignee | ||
Comment 5•5 years ago
|
||
Depends on D87043
Assignee | ||
Comment 6•5 years ago
|
||
Depends on D87044
Assignee | ||
Comment 7•5 years ago
|
||
Depends on D87045
Assignee | ||
Comment 8•5 years ago
|
||
Depends on D87046
Comment 9•5 years ago
|
||
Hello Daisuke. Should I add this bug as an MVP for Fission M2?
Flags: needinfo?(daisuke)
Updated•5 years ago
|
Whiteboard: dt-fission-m2-mvp
Updated•5 years ago
|
Fission Milestone: --- → M6c
Updated•5 years ago
|
Attachment #9170522 -
Attachment is obsolete: true
Assignee | ||
Updated•5 years ago
|
Keywords: leave-open
Comment 11•5 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•5 years ago
|
||
bugherder |
Assignee | ||
Comment 13•5 years ago
|
||
Depends on D87047
Assignee | ||
Updated•5 years ago
|
Keywords: leave-open
Comment 14•5 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•5 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: 5 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
•