Tabs Extension: Implementation of tabs.duplicate api.

chrome/browser/extensions/api/tabs/tabs.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/extensions/api/tabs/tabs.h"
 
 #include <algorithm>
 #include <limits>
 #include <vector>
 
@@ skipping 1071 matching lines @@
   // Return data about the newly created tab.
   if (has_callback()) {
     SetResult(ExtensionTabUtil::CreateTabValue(
         params.target_contents->web_contents(),
         tab_strip, new_index));
   }
 
   return true;
 }
 
+bool DuplicateTabFunction::RunImpl() {
+  int tab_id = -1;
+  EXTENSION_FUNCTION_VALIDATE(args_->GetInteger(0, &tab_id));
+
+  Browser* browser = NULL;
+  TabStripModel* tab_strip = NULL;
+  TabContents* contents = NULL;
+  int tab_index = -1;
+  if (!GetTabById(tab_id, profile(), include_incognito(),
+                  &browser, &tab_strip, &contents, &tab_index, &error_)) {
+    return false;
+  }
+
+  TabContents* new_contents = chrome::DuplicateTabAt(browser, tab_index);
+  int new_index = tab_strip->GetIndexOfTabContents(new_contents);
+
+  // Return data about the newly created tab.
+  if (has_callback()) {

[Aaron Boodman, 2012/08/15 04:14:05]

You can invert this check and return early to reduce indentation.

[SanjoyPal, 2012/09/05 08:31:15]

Done.

+    bool has_permission = GetExtension()->HasAPIPermissionForTab(tab_id,

[Aaron Boodman, 2012/08/15 04:14:05]

Actually, I think it would be better to just pass Extension* to CreateTabValue
and let it decide whether to include the sensitive fields or not. This way, it
will be harder for callers to mess up.

[SanjoyPal, 2012/09/05 08:31:15]

Done. We need cleanup other callers of CreateTabValue to pass Extension*. Can we
do it as a separate activity/CL?

[Aaron Boodman, 2012/09/05 21:56:21]

Since I made the original comment, we already modified extension_tab_util to
take Extension* for all CreateTabValue overloads.

If you sync, you'll see it and you can just call that.

+        extensions::APIPermission::kTab);
+    if (has_permission)

[Aaron Boodman, 2012/08/15 04:14:05]

Curly braces are required here...

[SanjoyPal, 2012/09/05 08:31:15]

Done.

+      SetResult(ExtensionTabUtil::CreateTabValue(
+          new_contents->web_contents(),
+          tab_strip, new_index,
+          ExtensionTabUtil::INCLUDE_PRIVACY_SENSITIVE_FIELDS));
+    else

[Aaron Boodman, 2012/08/15 04:14:05]

... and here ...

[SanjoyPal, 2012/09/05 08:31:15]

Done.

+      SetResult(ExtensionTabUtil::CreateTabValue(
+          new_contents->web_contents(),
+          tab_strip, new_index,
+          ExtensionTabUtil::OMIT_PRIVACY_SENSITIVE_FIELDS));
+  }
+
+  return true;
+}
+
 bool GetTabFunction::RunImpl() {
   int tab_id = -1;
   EXTENSION_FUNCTION_VALIDATE(args_->GetInteger(0, &tab_id));
 
   TabStripModel* tab_strip = NULL;
   TabContents* contents = NULL;
   int tab_index = -1;
   if (!GetTabById(tab_id, profile(), include_incognito(),
                   NULL, &tab_strip, &contents, &tab_index, &error_))
     return false;
@@ skipping 718 matching lines @@
   // called for every API call the extension made.
   GotLanguage(language);
 }
 
 void DetectTabLanguageFunction::GotLanguage(const std::string& language) {
   SetResult(Value::CreateStringValue(language.c_str()));
   SendResponse(true);
 
   Release();  // Balanced in Run()
 }