diff options
| -rw-r--r-- | chrome/browser/extensions/extension_tabs_module.cc | 10 | ||||
| -rwxr-xr-x | chrome/chrome_tests.gypi | 2 | ||||
| -rwxr-xr-x | chrome/common/extensions/api/extension_api.json | 3 | ||||
| -rw-r--r-- | chrome/test/data/extensions/api_test/tabs/basics/test.js | 9 |
4 files changed, 4 insertions, 20 deletions
diff --git a/chrome/browser/extensions/extension_tabs_module.cc b/chrome/browser/extensions/extension_tabs_module.cc index 6f40ad6..19d2687 100644 --- a/chrome/browser/extensions/extension_tabs_module.cc +++ b/chrome/browser/extensions/extension_tabs_module.cc @@ -386,15 +386,7 @@ bool UpdateWindowFunction::RunImpl() { } browser->window()->SetBounds(bounds); - - if (update_props->HasKey(keys::kFocusedKey)) { - bool focused = false; - EXTENSION_FUNCTION_VALIDATE(update_props->GetBoolean(keys::kFocusedKey, - &focused)); - if (focused) - browser->window()->Activate(); - } - + // TODO(rafaelw): Support |focused|. result_.reset(ExtensionTabUtil::CreateWindowValue(browser, false)); return true; diff --git a/chrome/chrome_tests.gypi b/chrome/chrome_tests.gypi index 026584f..e82d3c9 100755 --- a/chrome/chrome_tests.gypi +++ b/chrome/chrome_tests.gypi @@ -15,6 +15,7 @@ 'browser/extensions/extension_shelf_model_unittest.cc', 'browser/extensions/extension_startup_unittest.cc', 'browser/extensions/extension_storage_apitest.cc', + 'browser/extensions/extension_tabs_apitest.cc', 'browser/extensions/extension_popup_apitest.cc', # TODO(jam): http://crbug.com/15101 These tests fail on Linux and Mac. 'browser/child_process_security_policy_browser_test.cc', @@ -1114,7 +1115,6 @@ 'browser/extensions/extension_javascript_url_apitest.cc', 'browser/extensions/extension_messages_apitest.cc', 'browser/extensions/extension_override_apitest.cc', - 'browser/extensions/extension_tabs_apitest.cc', 'browser/extensions/extension_toolstrip_apitest.cc', 'browser/extensions/incognito_noscript_apitest.cc', 'browser/extensions/isolated_world_apitest.cc', diff --git a/chrome/common/extensions/api/extension_api.json b/chrome/common/extensions/api/extension_api.json index a02895f..1daeb09 100755 --- a/chrome/common/extensions/api/extension_api.json +++ b/chrome/common/extensions/api/extension_api.json @@ -357,8 +357,7 @@ "left": {"type": "integer", "optional": true, "description": "The offset from the left edge of the screen to move the window to in pixels."}, "top": {"type": "integer", "optional": true, "description": "The offset from the top edge of the screen to move the window to in pixels."}, "width": {"type": "integer", "minimum": 0, "optional": true, "description": "The width to resize the window to in pixels."}, - "height": {"type": "integer", "minimum": 0, "optional": true, "description": "The height to resize the window to in pixels."}, - "focused": {"type": "boolean", "optional": true, "description": "Set to true to focus the window. Setting to false has no effect on the window (it doesn't blur it)."} + "height": {"type": "integer", "minimum": 0, "optional": true, "description": "The height to resize the window to in pixels."} } }, { diff --git a/chrome/test/data/extensions/api_test/tabs/basics/test.js b/chrome/test/data/extensions/api_test/tabs/basics/test.js index 00f20dd..a516ddc 100644 --- a/chrome/test/data/extensions/api_test/tabs/basics/test.js +++ b/chrome/test/data/extensions/api_test/tabs/basics/test.js @@ -35,6 +35,7 @@ chrome.test.runTests([ assertTrue(tab.index > firstTabIndex); assertEq(firstWindowId, tab.windowId); assertEq(false, tab.selected); + assertEq("chrome://newtab/", tab.url); })); }, @@ -240,14 +241,6 @@ chrome.test.runTests([ })); }, - function focus() { - chrome.test.listenOnce(chrome.windows.onFocusChanged, function(winId) { - assertEq(firstWindowId, winId); - }); - - chrome.windows.update(firstWindowId, {focused: true}, pass()); - }, - /* // TODO(jcampan): http://crbug.com/30662 the detection language library // crashes on some sites and has been temporarily disabled. |