Browse Source

Adjust use of chrome.storage.managed for Opera

It turns out that chrome.storage.managed is defined for Opera, so the
previous feature detection method did not work.
Rob Wu 11 years ago
parent
commit
c9b3ea2f84
  1. 6
      web/preferences.js

6
web/preferences.js

@ -204,11 +204,15 @@ var Preferences = {
// // These preferences can be overridden by the user. // // These preferences can be overridden by the user.
// chrome.storage.managed.get(DEFAULT_PREFERENCES, getPreferences); // chrome.storage.managed.get(DEFAULT_PREFERENCES, getPreferences);
// } else { // } else {
// // Managed storage not supported, e.g. in Opera. // // Managed storage not supported, e.g. in old Chromium versions.
// getPreferences(DEFAULT_PREFERENCES); // getPreferences(DEFAULT_PREFERENCES);
// } // }
// //
// function getPreferences(defaultPrefs) { // function getPreferences(defaultPrefs) {
// if (chrome.runtime.lastError) {
// // Managed storage not supported, e.g. in Opera.
// defaultPrefs = DEFAULT_PREFERENCES;
// }
// chrome.storage.local.get(defaultPrefs, function(readPrefs) { // chrome.storage.local.get(defaultPrefs, function(readPrefs) {
// resolve(readPrefs); // resolve(readPrefs);
// }); // });

Loading…
Cancel
Save