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

Unified Diff: chrome/common/extensions/api/tabs.json

Issue 9699065: "index" property should be optional with "chrome.tabs.move" (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Created 8 years, 9 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 side-by-side diff with in-line comments
Download patch
Index: chrome/common/extensions/api/tabs.json
diff --git a/chrome/common/extensions/api/tabs.json b/chrome/common/extensions/api/tabs.json
index de8ed6724f6afa847d699e8f4a8ee23fe349a0ca..39e4969b34175274cd318fbde54310fd2c4019f8 100644
--- a/chrome/common/extensions/api/tabs.json
+++ b/chrome/common/extensions/api/tabs.json
@@ -429,6 +429,7 @@
{
"type": "object",
"name": "moveProperties",
+ "optional": true,
"properties": {
"windowId": {
"type": "integer",
@@ -439,7 +440,8 @@
"index": {
"type": "integer",
"minimum": 0,
- "description": "The position to move the window to. The provided value will be clamped to between zero and the number of tabs in the window."
+ "optional": true,
+ "description": "The position to move the window to. The provided value will be clamped to between zero and the number of tabs in the window. Defaults to the index of the last tab in the window."
}
}
},

Powered by Google App Engine
This is Rietveld 408576698