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

Side by Side Diff: chrome/browser/extensions/extension_settings_apitest.cc

Issue 7189029: Implement an initial extension settings API. (Closed) Base URL: http://git.chromium.org/git/chromium.git@trunk
Patch Set: Change where extension settings are saved, update TODO, api test Created 9 years, 6 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
(Empty)
1 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file.
4
5 #include "base/command_line.h"
6 #include "chrome/browser/extensions/extension_apitest.h"
7 #include "chrome/common/chrome_switches.h"
8 #include "net/base/mock_host_resolver.h"
9
10 class ExtensionSettingsApiTest : public ExtensionApiTest {
11 public:
12 virtual void SetUpInProcessBrowserTestFixture() {
13 ExtensionApiTest::SetUpInProcessBrowserTestFixture();
14 // TODO(kalman): necessary?
15 host_resolver()->AddRule("*", "127.0.0.1");
Mihai Parparita -not on Chrome 2011/06/21 23:40:11 I don't think this is necessary.
not at google - send to devlin 2011/06/22 09:40:38 Done.
16 ASSERT_TRUE(StartTestServer());
Mihai Parparita -not on Chrome 2011/06/21 23:40:11 We generally put this in the test case, though I d
not at google - send to devlin 2011/06/22 09:40:38 Done. Yeah this just came from one of the tests I
17 }
18 };
19
20 IN_PROC_BROWSER_TEST_F(ExtensionSettingsApiTest, Settings) {
21 CommandLine::ForCurrentProcess()->AppendSwitch(
22 switches::kEnableExperimentalExtensionApis);
23 ASSERT_TRUE(RunExtensionTest("settings")) << message_;
24 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698