Move chrome.alarms API from chrome/ to extensions/.

extensions/browser/api_unittest.h

 // Copyright 2014 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.
 
 #ifndef EXTENSIONS_BROWSER_API_UNITTEST_H_
 #define EXTENSIONS_BROWSER_API_UNITTEST_H_
 
 #include <string>
 
 #include "base/memory/ref_counted.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/prefs/testing_pref_service.h"
 #include "extensions/browser/extensions_test.h"
 #include "extensions/browser/mock_extension_system.h"
 
 namespace base {
 class Value;
 class DictionaryValue;
 class ListValue;
 }
 
 namespace content {
 class NotificationService;
 class TestBrowserThreadBundle;
+class WebContents;
 }
 
 class UIThreadExtensionFunction;
 
 namespace extensions {
 
 // Use this class to enable calling API functions in a unittest.
 // By default, this class will create and load an empty unpacked |extension_|,
 // which will be used in all API function calls. This extension can be
 // overridden using set_extension().
 // When calling RunFunction[AndReturn*], |args| should be in JSON format,
 // wrapped in a list. See also RunFunction* in api_test_utils.h.
 class ApiUnitTest : public ExtensionsTest {
  public:
   ApiUnitTest();
   ~ApiUnitTest() override;
 
+  content::WebContents* contents() { return contents_.get(); }
   const Extension* extension() const { return extension_.get(); }
   scoped_refptr<Extension> extension_ref() { return extension_; }
   void set_extension(scoped_refptr<Extension> extension) {
     extension_ = extension;
   }
 
  protected:
   // SetUp creates and loads an empty, unpacked Extension.
   void SetUp() override;
 
+  // Creates WebContents for |extension_| that can be used in API calls.
+  // If |contents_| is already set, this does nothing.
+  void CreateWebContents();

[James Cook, 2015/01/17 00:51:29]

Is there any particular reason not to call this CreateBackgroundPage()?  Keeping
the name might make it easier to move other tests in the future, and helps new
authors realize this is the same method.

[babu, 2015/01/17 11:53:56]

Done. 

Changed to the original name and removed the function from
extension_api_unittest.h since no other tests are using it.

+
   // Various ways of running an API function. These methods take ownership of
   // |function|. |args| should be in JSON format, wrapped in a list.
   // See also the RunFunction* methods in extension_function_test_utils.h.
 
   // Return the function result as a base::Value.
   scoped_ptr<base::Value> RunFunctionAndReturnValue(
       UIThreadExtensionFunction* function,
       const std::string& args);
 
   // Return the function result as a base::DictionaryValue, or NULL.
@@ skipping 18 matching lines @@
                    const std::string& args);
 
  private:
   scoped_ptr<content::NotificationService> notification_service_;
 
   scoped_ptr<content::TestBrowserThreadBundle> thread_bundle_;
   TestingPrefServiceSimple testing_pref_service_;
 
   MockExtensionSystemFactory<MockExtensionSystem> extension_system_factory_;
 
+  // The WebContents used to associate a RenderViewHost with API function calls,
+  // or NULL.

[James Cook, 2015/01/17 00:51:29]

nit: "null" in comments, since we don't use the NULL macro any more.

[babu, 2015/01/17 11:53:56]

Done.

+  scoped_ptr<content::WebContents> contents_;
+
   // The Extension used when running API function calls.
   scoped_refptr<Extension> extension_;
 };
 
 }  // namespace extensions
 
 #endif  // EXTENSIONS_BROWSER_API_UNITTEST_H_