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

Side by Side Diff: chrome/browser/extensions/api/page_capture/page_capture_apitest.cc

Issue 12212047: Linux/ChromeOS Chromium style checker cleanup, chrome/browser/extensions edition. (Closed) Base URL: http://src.chromium.org/svn/trunk/src/
Patch Set: Created 7 years, 10 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
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "base/base_switches.h" 5 #include "base/base_switches.h"
6 #include "base/command_line.h" 6 #include "base/command_line.h"
7 #include "chrome/browser/extensions/extension_apitest.h" 7 #include "chrome/browser/extensions/extension_apitest.h"
8 #include "chrome/browser/extensions/api/page_capture/page_capture_api.h" 8 #include "chrome/browser/extensions/api/page_capture/page_capture_api.h"
9 #include "chrome/common/chrome_switches.h" 9 #include "chrome/common/chrome_switches.h"
10 #include "chrome/test/base/ui_test_utils.h" 10 #include "chrome/test/base/ui_test_utils.h"
11 #include "net/base/mock_host_resolver.h" 11 #include "net/base/mock_host_resolver.h"
12 12
13 using extensions::PageCaptureSaveAsMHTMLFunction; 13 using extensions::PageCaptureSaveAsMHTMLFunction;
14 14
15 class ExtensionPageCaptureApiTest : public ExtensionApiTest { 15 class ExtensionPageCaptureApiTest : public ExtensionApiTest {
16 public: 16 public:
17 virtual void SetUpCommandLine(CommandLine* command_line) { 17 virtual void SetUpCommandLine(CommandLine* command_line) OVERRIDE {
18 ExtensionApiTest::SetUpCommandLine(command_line); 18 ExtensionApiTest::SetUpCommandLine(command_line);
19 command_line->AppendSwitchASCII(switches::kJavaScriptFlags, "--expose-gc"); 19 command_line->AppendSwitchASCII(switches::kJavaScriptFlags, "--expose-gc");
20 } 20 }
21 21
22 virtual void SetUpInProcessBrowserTestFixture() { 22 virtual void SetUpInProcessBrowserTestFixture() OVERRIDE {
23 ExtensionApiTest::SetUpInProcessBrowserTestFixture(); 23 ExtensionApiTest::SetUpInProcessBrowserTestFixture();
24 24
25 host_resolver()->AddRule("www.a.com", "127.0.0.1"); 25 host_resolver()->AddRule("www.a.com", "127.0.0.1");
26 26
27 ASSERT_TRUE(StartTestServer()); 27 ASSERT_TRUE(StartTestServer());
28 } 28 }
29 }; 29 };
30 30
31 // Disabled on Linux http://crbug.com/98194 31 // Disabled on Linux http://crbug.com/98194
32 #if defined(OS_LINUX) 32 #if defined(OS_LINUX)
(...skipping 22 matching lines...) Expand all
55 55
56 IN_PROC_BROWSER_TEST_F(ExtensionPageCaptureApiTest, MAYBE_SaveAsMHTML) { 56 IN_PROC_BROWSER_TEST_F(ExtensionPageCaptureApiTest, MAYBE_SaveAsMHTML) {
57 PageCaptureSaveAsMHTMLDelegate delegate; 57 PageCaptureSaveAsMHTMLDelegate delegate;
58 ASSERT_TRUE(RunExtensionTest("page_capture")) << message_; 58 ASSERT_TRUE(RunExtensionTest("page_capture")) << message_;
59 ASSERT_FALSE(delegate.temp_file_.empty()); 59 ASSERT_FALSE(delegate.temp_file_.empty());
60 // Flush the message loops to make sure the delete happens. 60 // Flush the message loops to make sure the delete happens.
61 content::RunAllPendingInMessageLoop(content::BrowserThread::FILE); 61 content::RunAllPendingInMessageLoop(content::BrowserThread::FILE);
62 content::RunAllPendingInMessageLoop(content::BrowserThread::IO); 62 content::RunAllPendingInMessageLoop(content::BrowserThread::IO);
63 ASSERT_FALSE(file_util::PathExists(delegate.temp_file_)); 63 ASSERT_FALSE(file_util::PathExists(delegate.temp_file_));
64 } 64 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698