aboutsummaryrefslogtreecommitdiffstats
path: root/content_scripts
diff options
context:
space:
mode:
authorStephen Blott2015-06-17 05:08:03 +0100
committerStephen Blott2015-06-17 05:53:01 +0100
commit8ff1aef751a533c17e683207dae1eb165b210f92 (patch)
treef1614099efdb8b297656576dcf4855b4a0d1c86d /content_scripts
parentaa00e29dc2533b6701c65935223599671c5833b1 (diff)
downloadvimium-8ff1aef751a533c17e683207dae1eb165b210f92.tar.bz2
Fix non-default front-end settings.
(@mrmr1993: This is yet another approach to the Settings problem.) With the new Settings implemetation, settings which have a non-default value and which are not in synced storage (that is, they have not been changed since synced storage was introduced) are not currently accessible to content scripts. This commit makes such settings accessible via chrome.storage.local. Important: - There's a change to the established settings data model here. Previously, settings with default values were not stored; here, they are. This eliminates a considerable amount logic from Settings, but means that migrations will be required if default values are changed in future. (Other than type changes, have we ever changed a default value?) - There's also a change (bug fix?) to the behaviour when an affected setting is reset to its default value. Previously, the change would *not* have been synced (whereas all other changes are). Here, such changes *are* synced. The previous behaviour was inconsistent with the syncing behaviour of all other options changes. Note: - This isn't particularly well tested. It's being committed mainly just for consideration of the approach, initially.
Diffstat (limited to 'content_scripts')
0 files changed, 0 insertions, 0 deletions