Updated description. Track when the browser action closes in the back…

…ground page to clean up any lagging previews on the current page

stylebot/background/background.js

@@ -32,6 +32,7 @@ function init() {
  });
  });
 
+ BrowserAction.init();
  attachListeners();
 }
 

stylebot/background/browseraction.js

@@ -1,5 +1,28 @@
 // Update the extension browser action
 var BrowserAction = {
+ init: function() {
+ // Track when the browser action closes to do cleanup on the current page
+ chrome.runtime.onConnect.addListener(function(port) {
+ if (port.name === "browserAction") {
+ var activeTab;
+
+ port.onMessage.addListener(function(message) {
+ if (message.name === "activeTab") {
+ activeTab = message.tab;
+ }
+ });
+
+ port.onDisconnect.addListener(function() {
+ if (activeTab) {
+ chrome.tabs.sendRequest(activeTab.id, {
+ name: "resetPreview"
+ }, function(response){});
+ }
+ });
+ }
+ });
+ },
+
  /**
  * Update browser action for the specified tab to indicate:
  * - stylebot is not visible

stylebot/browseraction/browseraction.js

@@ -9,18 +9,25 @@ var BrowserAction = {
  styles: {},
 
  init: function() {
+ var port = chrome.runtime.connect({name: "browserAction"});
+
  chrome.windows.getCurrent({populate: true}, function(aWindow) {
  var tabs = aWindow.tabs;
  var len = tabs.length;
+
  for (var i = 0; i < len; i++) {
  var tab = tabs[i];
  if (tab.active) {
+ port.postMessage({name: "activeTab", tab: tab});
+
  $(".share").click(function(e) {
  BrowserAction.share(e, tab);
  });
+
  $(".open").click(function(e) {
  BrowserAction.open(e, tab);
  });
+
  $(".reset").click(function(e) {
  BrowserAction.reset(e, tab);
  }).mouseenter(function(e) {
@@ -32,6 +39,7 @@ var BrowserAction = {
  name: "resetPreview"
  }, function(response){});
  })
+
  $(".options").click(function(e) {
  BrowserAction.options(e, tab);
  });
@@ -130,6 +138,7 @@ var BrowserAction = {
  });
  });
  });
+
  return;
  }
  }

stylebot/editor/style.js

@@ -595,7 +595,7 @@ stylebot.style = {
  * Reset the preview of any style and reset to the specifed CSS.
  * @param {String} css The CSS to apply to the page.
  */
- resetPreview: function(css) {
+ resetPreview: function() {
  CSSUtils.crunchCSS(this.rules, true, true, $.proxy(function(css) {
  $(this.CSS_SELECTOR).html(css);
  }, this));

stylebot/manifest.json

@@ -2,7 +2,8 @@
  "manifest_version": 2,
  "name" : "Stylebot",
  "version" : "2",
- "description" : "Adapt the web's appearance",
+ "description" : "Change the appearance of websites instantly.
+ Preview and install styles created by users for your favorite websites",
 
  "background" : {
  "scripts": [
@@ -16,7 +17,7 @@
  ]
  },
 
- "options_page" : "options/index.html",
+ "options_page": "options/index.html",
 
  "content_scripts": [{
  "matches": ["<all_urls>"],