Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(186)

Side by Side Diff: chrome/test/data/extensions/samples/screenshot/screenshot.js

Issue 179027: Make a bunch of backlogged breaking changes to the extensions (Closed)
Patch Set: Put chrome.self back Created 11 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 function takeScreenshot() { 1 function takeScreenshot() {
2 chrome.tabs.captureVisibleTab(null, function(img) { 2 chrome.tabs.captureVisibleTab(null, function(img) {
3 var screenshotUrl = img; 3 var screenshotUrl = img;
4 var viewTabUrl = chrome.extension.getURL('screenshot.html'); 4 var viewTabUrl = chrome.extension.getURL('screenshot.html');
5 5
6 chrome.tabs.create({url: viewTabUrl}, function(tab) { 6 chrome.tabs.create({url: viewTabUrl}, function(tab) {
7 var targetId = tab.id; 7 var targetId = tab.id;
8 8
9 chrome.tabs.onUpdated.addListener(function(tabId, changedProps) { 9 chrome.tabs.onUpdated.addListener(function(tabId, changedProps) {
10 if (tabId == targetId && 10 if (tabId == targetId &&
11 changedProps.status == "complete") { 11 changedProps.status == "complete") {
12 var views = chrome.self.getViews(); 12 var views = chrome.extension.getViews();
13 for (var i = 0; i < views.length; i++) { 13 for (var i = 0; i < views.length; i++) {
14 var view = views[i]; 14 var view = views[i];
15 if (view.location.href == viewTabUrl) { 15 if (view.location.href == viewTabUrl) {
16 view.setScreenshotUrl(screenshotUrl); 16 view.setScreenshotUrl(screenshotUrl);
17 } 17 }
18 } 18 }
19 } 19 }
20 }); 20 });
21 }) 21 })
22 }); 22 });
23 }; 23 };
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698