- Lazily initialize file IO-specific stuff in ExtensionStorage.jsm,
and limit this work to the main process.
- Add local versions of the `storage.local.get` and `storage.local.set`
implementations that perform sanitization (without the change, values
are improperly serialized over IPC).
- Copied the `backgroundScript` test from xpcshell/test_ext_storage.js
to mochitest/test_ext_storage_content.html because they should behave
identical. Before this patch the test failed due to IPC serialization
issues, now the test passes.
Note that the old test also passes with the changes.
MozReview-Commit-ID: 8J8CCdwMACN
--HG--
extra : rebase_source : 7b8a490b79c09f69a10d4dc13ad23f8382b641d1
The base .eslintrc is essentially a merge of the root Toolkit .eslintrc and
the devtools .eslintrc, with some minor changes to match our prevalent style.
For the most enforces the coding styles that we've been using most
consistently. There are a couple of significant differences, though:
* The rule for opening brace alignment can only be applied globally, and
doesn't make exceptions for top-level functions. I chose to turn it on, and
change the brace style of existing top-level functions that violated it,
since the rule seemed worth using, and that's the direction most Toolkit JS
code has been headed anyway.
* The rule for switch/case statements requires an added indentation level for
case statements. Most of our switch statements did not use an extra level
of indentation, and I initially wrote the rule to enforce that style, until
I came across case statements that used blocks, and required the extra
indentation level for sanity.
--HG--
rename : devtools/.eslintrc => toolkit/components/extensions/.eslintrc
extra : commitid : 4aDZOi5MNPc
extra : rebase_source : f34ba43aeb8046a0c971cb76b76410497bd12b42
extra : source : 2a7eb4ef7b375e5fd52d843ca206a071367a6475
Florian Queze
Sebastian Hengst
Luca Greco
Rob Wu
Martin Giger
Andrew Swan
Kris Maglione
Bill McCloskey
Johann Hofmann
Phil Ringnalda
Shu-yu Guo
Fabrice Desré
Carsten "Tomcat" Book