Tabs Extension: Implementation of tabs.duplicate api.

chrome/browser/extensions/api/tabs/tabs_test.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 <string>
 
 #include "base/memory/ref_counted.h"
 #include "base/memory/scoped_ptr.h"
@@ skipping 529 matching lines @@
 
   static const char kArgsMaximizedWithBounds[] =
       "[%u, {\"state\": \"maximized\", \"width\": 500}]";
   EXPECT_TRUE(MatchPattern(
       utils::RunFunctionAndReturnError(
           new UpdateWindowFunction(),
           base::StringPrintf(kArgsMaximizedWithBounds, window_id),
           browser()),
       keys::kInvalidWindowStateError));
 }
+
+IN_PROC_BROWSER_TEST_F(ExtensionTabsTest, DuplicateTabNoPermission) {

[Aaron Boodman, 2012/07/16 21:37:59]

What about a test for when the extension has the permission?

[SanjoyPal, 2012/07/17 10:39:56]

Done.

+  scoped_refptr<CreateTabFunction> create_tab_function(new CreateTabFunction());
+  // Without a callback the function will not generate a result.
+  create_tab_function->set_has_callback(true);
+
+  static const char kNewBlankTabArgs[] =
+      "[{\"url\": \"about:blank\"}]";
+
+  scoped_ptr<base::DictionaryValue> result(utils::ToDictionary(
+      utils::RunFunctionAndReturnResult(
+          create_tab_function.get(),
+          kNewBlankTabArgs,
+          browser())));
+
+  int tab_id = utils::GetInteger(result.get(), "id");
+  int tab_index = utils::GetInteger(result.get(), "index");
+
+  scoped_refptr<DuplicateTabFunction> duplicate_tab_function(
+      new DuplicateTabFunction());
+  scoped_refptr<extensions::Extension> empty_extension(
+      utils::CreateEmptyExtension());
+  duplicate_tab_function->set_extension(empty_extension.get());
+  // Without a callback the function will not generate a result.
+  duplicate_tab_function->set_has_callback(true);
+
+  scoped_ptr<base::DictionaryValue> duplicate_result(utils::ToDictionary(
+      utils::RunFunctionAndReturnResult(
+          duplicate_tab_function.get(), base::StringPrintf("[%u]", tab_id),
+          browser())));
+
+  int duplicate_tab_id = utils::GetInteger(duplicate_result.get(), "id");
+  int duplicate_tab_index = utils::GetInteger(duplicate_result.get(), "index");
+  // The test empty extension has no permissions, therefore it should get
+  // tab id of TYPE_INTEGER in the function result.

[Aaron Boodman, 2012/07/16 21:37:59]

Comment is out of date.

[SanjoyPal, 2012/07/17 10:39:56]

Done.

+  EXPECT_EQ(base::Value::TYPE_DICTIONARY, duplicate_result->GetType());
+  // Duplicate tab id should be different from the original tab id.
+  EXPECT_NE(tab_id, duplicate_tab_id);
+  EXPECT_EQ(tab_index + 1, duplicate_tab_index);

[Aaron Boodman, 2012/07/16 21:37:59]

Can you test that URL, faviconURL, and title are *not* present?

Also can you factor that bit out into a utility function because it will be
commonly needed: crbug.com/137404.

[SanjoyPal, 2012/07/17 10:39:56]

Done.

+}
+

[Aaron Boodman, 2012/07/16 21:37:59]

extraneous blank line

[SanjoyPal, 2012/07/17 10:39:56]

Done.