Chromium Code Reviews| Index: chrome/common/extensions/docs/examples/api/windows/merge_windows/background.html |
| diff --git a/chrome/common/extensions/docs/examples/api/windows/merge_windows/background.html b/chrome/common/extensions/docs/examples/api/windows/merge_windows/background.html |
| index 6575af892dcf15435ecd9fad18639cbeae36b086..078ea4558a08ced4dc1b490a6dc54709bc5c32b5 100644 |
| --- a/chrome/common/extensions/docs/examples/api/windows/merge_windows/background.html |
| +++ b/chrome/common/extensions/docs/examples/api/windows/merge_windows/background.html |
| @@ -1,49 +1,9 @@ |
| +<!doctype html> |
| <html> |
| <head> |
| - <script> |
| - var targetWindow = null; |
| - var tabCount = 0; |
| - |
| - function start(tab) { |
| - chrome.windows.getCurrent(getWindows); |
| - } |
| - |
| - function getWindows(win) { |
| - targetWindow = win; |
| - chrome.tabs.getAllInWindow(targetWindow.id, getTabs); |
| - } |
| - |
| - function getTabs(tabs) { |
| - tabCount = tabs.length; |
| - // We require all the tab information to be populated. |
| - chrome.windows.getAll({"populate" : true}, moveTabs); |
| - } |
| - |
| - function moveTabs(windows) { |
| - var numWindows = windows.length; |
| - var tabPosition = tabCount; |
| - |
| - for (var i = 0; i < numWindows; i++) { |
| - var win = windows[i]; |
| - |
| - if (targetWindow.id != win.id) { |
| - var numTabs = win.tabs.length; |
| - |
| - for (var j = 0; j < numTabs; j++) { |
| - var tab = win.tabs[j]; |
| - |
| - // Move the tab into the window that triggered the browser action. |
| - chrome.tabs.move(tab.id, |
| - {"windowId": targetWindow.id, "index": tabPosition}); |
| - |
| - tabPosition++; |
| - } |
| - } |
| - } |
| - } |
| - |
| - // Set up a click handler so that we can merge all the windows. |
| - chrome.browserAction.onClicked.addListener(start); |
| - </script> |
| + <title>Background Page</title> |
| + <script src="background.js"></script> |
| </head> |
| -</html> |
| + <body> |
| + </body> |
|
Boris Smus
2011/10/18 21:42:06
According to http://html5.validator.nu/, body elem
Mike West
2011/10/19 08:39:12
Ok... there's a body element. I'm not sure I under
|
| +</html> |