Fixed Screenshot extension example

chrome/common/extensions/docs/examples/api/tabs/screenshot/background.js

Index: chrome/common/extensions/docs/examples/api/tabs/screenshot/background.js
diff --git a/chrome/common/extensions/docs/examples/api/tabs/screenshot/background.js b/chrome/common/extensions/docs/examples/api/tabs/screenshot/background.js
index 4124857248261f40e234d7c8f49e94bfc92cd0b6..ecba5f40078bb7465b7004426e5520fbb986e354 100644
--- a/chrome/common/extensions/docs/examples/api/tabs/screenshot/background.js
+++ b/chrome/common/extensions/docs/examples/api/tabs/screenshot/background.js
@@ -9,49 +9,26 @@
 // open.
 var id = 100;
 
-function takeScreenshot() {
-  chrome.tabs.captureVisibleTab(null, function(img) {
+// Listen for a click on the camera icon. On that click, take a screenshot.
+chrome.browserAction.onClicked.addListener(function() {
+
+  chrome.tabs.captureVisibleTab(function(img) {
     var screenshotUrl = img;
     var viewTabUrl = chrome.extension.getURL('screenshot.html?id=' + id++)
 
-    chrome.tabs.create({url: viewTabUrl}, function(tab) {
-      var targetId = tab.id;
-
-      var addSnapshotImageToTab = function(tabId, changedProps) {
-        // We are waiting for the tab we opened to finish loading.
-        // Check that the the tab's id matches the tab we opened,
-        // and that the tab is done loading.
-        if (tabId != targetId || changedProps.status != "complete")
-          return;
-
-        // Passing the above test means this is the event we were waiting for.
-        // There is nothing we need to do for future onUpdated events, so we
-        // use removeListner to stop geting called when onUpdated events fire.
-        chrome.tabs.onUpdated.removeListener(addSnapshotImageToTab);
-
-        // Look through all views to find the window which will display
-        // the screenshot.  The url of the tab which will display the
-        // screenshot includes a query parameter with a unique id, which
-        // ensures that exactly one view will have the matching URL.
-        var views = chrome.extension.getViews();
-        for (var i = 0; i < views.length; i++) {
-          var view = views[i];
-          if (view.location.href == viewTabUrl) {
-            view.setScreenshotUrl(screenshotUrl);
-            break;
-          }
+    chrome.tabs.create({url: viewTabUrl}, function() {

[not at google - send to devlin, 2014/08/21 15:00:50]

You still do need to do the onUpdated dance here. Creating a tab doesn't
necessarily mean it's loaded, it just means the UI frame has been created.

+      // Look through all views to find the window which will display
+      // the screenshot.  The url of the tab which will display the
+      // screenshot includes a query parameter with a unique id, which
+      // ensures that exactly one view will have the matching URL.
+      var views = chrome.extension.getViews();
+      for (var i = 0; i < views.length; i++) {
+        var view = views[i];
+        if (view.location.href == viewTabUrl) {
+          view.setScreenshotUrl(screenshotUrl);
+          break;
         }
-      };
-      chrome.tabs.onUpdated.addListener(addSnapshotImageToTab);
+      }
     });
   });
-}
-
-// Listen for a click on the camera icon.  On that click, take a screenshot.
-chrome.browserAction.onClicked.addListener(function(tab) {
-  if (tab.url.match("^https?://code.google.com")) {
-    takeScreenshot();
-  } else {
-    alert('This sample can only take screenshots of code.google.com pages');
-  }
 });