Make chrome.tabs.update's tabId parameter optional.

chrome/browser/extensions/extension_tabs_module.cc

Index: chrome/browser/extensions/extension_tabs_module.cc
diff --git a/chrome/browser/extensions/extension_tabs_module.cc b/chrome/browser/extensions/extension_tabs_module.cc
index 219e36674dbe60ebe021bfd37ba6dfdbe67fd801..07145b5a0ddadf0b099b78bc06d48479abef53c7 100644
--- a/chrome/browser/extensions/extension_tabs_module.cc
+++ b/chrome/browser/extensions/extension_tabs_module.cc
@@ -892,18 +892,38 @@ UpdateTabFunction::UpdateTabFunction() {
 }
 
 bool UpdateTabFunction::RunImpl() {
-  int tab_id;
-  EXTENSION_FUNCTION_VALIDATE(args_->GetInteger(0, &tab_id));
   DictionaryValue* update_props;
   EXTENSION_FUNCTION_VALIDATE(args_->GetDictionary(1, &update_props));
 
-  TabStripModel* tab_strip = NULL;
+  Value* tab_value = NULL;

[Mihai Parparita -not on Chrome, 2011/09/21 23:24:05]

ExecuteCodeInTabFunction::RunImpl, ReloadTabFunction::RunImpl, and
DetectTabLanguageFunction::RunImpl all have very similar code. It seems like
it's time to move it to a helper function (perhaps by expanding what
ExtensionTabUtil::GetDefaultTab does).

[miket_OOO, 2011/09/22 00:06:44]

Yes, I agree. If I promise not to avoid doing it, would you be OK with my doing
it in a separate CL? I would indeed enjoy refactoring this code.

+  if (HasOptionalArgument(0)) {
+    EXTENSION_FUNCTION_VALIDATE(args_->Get(0, &tab_value));
+  }
+
+  int tab_id = -1;
   TabContentsWrapper* contents = NULL;
+  if (tab_value == NULL || tab_value->IsType(Value::TYPE_NULL)) {
+    Browser* browser = GetCurrentBrowser();
+    if (!browser) {
+      error_ = keys::kNoCurrentWindowError;
+      return false;
+    }
+    contents = browser->tabstrip_model()->GetActiveTabContents();
+    if (!contents) {
+      error_ = keys::kNoSelectedTabError;
+      return false;
+    }
+    tab_id = ExtensionTabUtil::GetTabId(contents->tab_contents());
+  } else {
+    EXTENSION_FUNCTION_VALIDATE(tab_value->GetAsInteger(&tab_id));
+  }
+
   int tab_index = -1;
+  TabStripModel* tab_strip = NULL;
   if (!GetTabById(tab_id, profile(), include_incognito(),
-                  NULL, &tab_strip, &contents, &tab_index, &error_))
+                  NULL, &tab_strip, &contents, &tab_index, &error_)) {
     return false;
-
+  }
   NavigationController& controller = contents->controller();
 
   // TODO(rafaelw): handle setting remaining tab properties: