Browse Source

more spurious warnings in console foiled

pull/2/head
Raymond Hill 10 years ago
committed by gorhill
parent
commit
ff6c553fb2
  1. 16
      meta/crx/vapi-background.js

16
meta/crx/vapi-background.js

@ -69,11 +69,22 @@ vAPI.tabs.registerListeners = function() {
/******************************************************************************/ /******************************************************************************/
vAPI.tabs.get = function(tabId, callback) { vAPI.tabs.get = function(tabId, callback) {
var onTabReady = function(tab) {
// https://code.google.com/p/chromium/issues/detail?id=410868#c8
if ( chrome.runtime.lastError ) {
return;
}
callback(tab);
}
if ( tabId !== null ) { if ( tabId !== null ) {
chrome.tabs.get(tabId, callback);
chrome.tabs.get(tabId, onTabReady);
return; return;
} }
var onTabReceived = function(tabs) { var onTabReceived = function(tabs) {
// https://code.google.com/p/chromium/issues/detail?id=410868#c8
if ( chrome.runtime.lastError ) {
return;
}
callback(tabs[0]); callback(tabs[0]);
}; };
chrome.tabs.query({ active: true, currentWindow: true }, onTabReceived); chrome.tabs.query({ active: true, currentWindow: true }, onTabReceived);
@ -181,8 +192,7 @@ vAPI.tabs.remove = function(tabId) {
vAPI.tabs.injectScript = function(tabId, details, callback) { vAPI.tabs.injectScript = function(tabId, details, callback) {
var onScriptExecuted = function() { var onScriptExecuted = function() {
// Must check `lastError` or else this may happen in the console:
// Unchecked runtime.lastError while running tabs.executeScript: The tab was closed.
// https://code.google.com/p/chromium/issues/detail?id=410868#c8
if ( chrome.runtime.lastError ) { if ( chrome.runtime.lastError ) {
} }
if ( typeof callback === 'function' ) { if ( typeof callback === 'function' ) {

Loading…
Cancel
Save