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

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

Issue 624153002: replace OVERRIDE and FINAL with override and final in chrome/browser/extensions/ (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 2 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
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/api/page_capture/page_capture_api.h" 7 #include "chrome/browser/extensions/api/page_capture/page_capture_api.h"
8 #include "chrome/browser/extensions/extension_apitest.h" 8 #include "chrome/browser/extensions/extension_apitest.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/dns/mock_host_resolver.h" 11 #include "net/dns/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) OVERRIDE { 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 22
23 class PageCaptureSaveAsMHTMLDelegate 23 class PageCaptureSaveAsMHTMLDelegate
24 : public PageCaptureSaveAsMHTMLFunction::TestDelegate { 24 : public PageCaptureSaveAsMHTMLFunction::TestDelegate {
25 public: 25 public:
26 PageCaptureSaveAsMHTMLDelegate() { 26 PageCaptureSaveAsMHTMLDelegate() {
27 PageCaptureSaveAsMHTMLFunction::SetTestDelegate(this); 27 PageCaptureSaveAsMHTMLFunction::SetTestDelegate(this);
28 } 28 }
29 29
30 virtual ~PageCaptureSaveAsMHTMLDelegate() { 30 virtual ~PageCaptureSaveAsMHTMLDelegate() {
31 PageCaptureSaveAsMHTMLFunction::SetTestDelegate(NULL); 31 PageCaptureSaveAsMHTMLFunction::SetTestDelegate(NULL);
32 } 32 }
33 33
34 virtual void OnTemporaryFileCreated( 34 virtual void OnTemporaryFileCreated(
35 const base::FilePath& temp_file) OVERRIDE { 35 const base::FilePath& temp_file) override {
36 temp_file_ = temp_file; 36 temp_file_ = temp_file;
37 } 37 }
38 38
39 base::FilePath temp_file_; 39 base::FilePath temp_file_;
40 }; 40 };
41 41
42 IN_PROC_BROWSER_TEST_F(ExtensionPageCaptureApiTest, SaveAsMHTML) { 42 IN_PROC_BROWSER_TEST_F(ExtensionPageCaptureApiTest, SaveAsMHTML) {
43 host_resolver()->AddRule("www.a.com", "127.0.0.1"); 43 host_resolver()->AddRule("www.a.com", "127.0.0.1");
44 ASSERT_TRUE(StartEmbeddedTestServer()); 44 ASSERT_TRUE(StartEmbeddedTestServer());
45 PageCaptureSaveAsMHTMLDelegate delegate; 45 PageCaptureSaveAsMHTMLDelegate delegate;
46 ASSERT_TRUE(RunExtensionTest("page_capture")) << message_; 46 ASSERT_TRUE(RunExtensionTest("page_capture")) << message_;
47 ASSERT_FALSE(delegate.temp_file_.empty()); 47 ASSERT_FALSE(delegate.temp_file_.empty());
48 // Flush the message loops to make sure the delete happens. 48 // Flush the message loops to make sure the delete happens.
49 content::RunAllPendingInMessageLoop(content::BrowserThread::FILE); 49 content::RunAllPendingInMessageLoop(content::BrowserThread::FILE);
50 content::RunAllPendingInMessageLoop(content::BrowserThread::IO); 50 content::RunAllPendingInMessageLoop(content::BrowserThread::IO);
51 ASSERT_FALSE(base::PathExists(delegate.temp_file_)); 51 ASSERT_FALSE(base::PathExists(delegate.temp_file_));
52 } 52 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698