Move the getter / setter for compatibility browser list prefs to CompatibilityActor
Categories
(DevTools :: Inspector: Compatibility, enhancement, P3)
Tracking
(firefox82 fixed)
Tracking | Status | |
---|---|---|
firefox82 | --- | fixed |
People
(Reporter: lelouch.cpp, Assigned: lelouch.cpp)
References
(Blocks 2 open bugs)
Details
Attachments
(1 file, 1 obsolete file)
(deleted),
text/x-phabricator-request
|
Details |
User Agent: Mozilla/5.0 (X11; Linux x86_64; rv:80.0) Gecko/20100101 Firefox/80.0
Expected results:
We have code duplication for getting and setting browser list pref in compatibility/UserSetting.js
and the CompatibilityActor.
We should move the functions from UserSetting.js
to the CompatibilityActor
There is a lot of code common between the Compatibility Actor and
the UserSettings and moving this to shared folder will cause
inconsistencies between values read by the client and server.
This patch moves the functions of UserSetting to Compatibility
Actor.
Instead of replicationg the browser list fetch using the
preference on the server side, we send the browser list from the
client side to the CompatibilityActor to get the declaration block
compatibility issues.
This way the pref setting is retained completely on the client
side and server doesn't have code duplication.
Depends on D86523
Updated•4 years ago
|
Updated•4 years ago
|
Comment 4•4 years ago
|
||
bugherder |
Updated•4 years ago
|
Description
•