Как забить функции Chrome Extension API в переменные?

Question

[Petr Flaks]

Здравствуйте.

У меня есть расширение для Google Chrome. Есть задача перегнать это расширение на браузеры, поддерживающие Chrome Extension API (типа Яндекс.Браузера, Opera и т.п.). Только как известно, не все браузеры в полной мере поддерживают все возможности API. По меньшей мере Opera не поддерживает chrome.storage.sync и некоторые параметры в chrome.notifications.create. Чтобы не делать для каждого браузера индивидуальное расширение, было решено обернуть функции со слабой поддержкой в собственную переменную. Например, чтобы вместо chrome.storage.sync.get(keys, callback) вызывать cbf.storage.sync.get(keys, callback). Реализуется это примерно так:

var cbf = {};
cbf.storage = {};
cbf.storage.sync = {};
// browser_name задается чуть раньше
cbf.storage.sync.get = browser_name == "Chrome" ? chrome.storage.sync.get : chrome.storage.local.get;

Только вот проблема в том, что это почему-то не работает. При вызове функции в консоль отбрасывается ошибка:

cbf.storage.sync.get(function(sync_storage) { /* ... */ });

Текст ошибки:

extensions::StorageArea:26 Uncaught TypeError: Cannot read property 'get' of undefined

Похоже, что я что-то не понимаю. Подскажите, пожалуйста, как правильно.

Answer 1

[Dmitry Pavlov]

А то, что вы туда складываете - chrome.storage.sync.get или chrome.storage.local.get - оно определено? Что там, если вывести в консоль?

console.log(chrome.storage.local.get);

Comments

[Petr Flaks]

Да, chrome.storage.sync.get и chrome.storage.local.get являются встроенными методами, используемыми в расширениях Chrome.

Вот наглядная демонстрация проблемы:


Если ввести console.log(chrome.storage.local.get);, то выведет следующее:


Изображения кликабельны.

[Dmitry Pavlov]

terron: что если попробовать обернуть в свою функцию?

Что-то типа:

cbf.storage.sync = {
get: function (/*string or array of string or object*/ keys, /*function*/ callback) {
// тут - проброс в то или иное хранилище в зависимости от браузера - https://developer.chrome.com/extensions/storage#ty...
}
};

[Petr Flaks]

Dmitry Pavlov: в принципе, так и сделал:

var cbf = {};
cbf.storage = {};
cbf.storage.sync = {};
cbf.notifications = {};

cbf.storage.sync.get = function() {	
	if (browser_info['name'] == "Chrome") {
		if      (arguments.length === 1) { chrome.storage.sync.get(arguments[0]); }
		else if (arguments.length === 2) { chrome.storage.sync.get(arguments[0], arguments[1]); }
	}
	else {
		if      (arguments.length === 1) { chrome.storage.local.get(arguments[0]); }
		else if (arguments.length === 2) { chrome.storage.local.get(arguments[0], arguments[1]); }
	}
}

Хотя решение мне кажется не очень элегантным, Да и вообще, было бы классно узнать, почему приведенный выше код не дал ожидаемых результатов. Для саморазвития, так сказать.

[Dmitry Pavlov]

terron: может контекста не хватает? stackoverflow.com/questions/8773579/if-javascript-...