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

Side by Side Diff: chrome/browser/plugins/plugin_installer_unittest.cc

Issue 836393007: Update {virtual,override,final} for chrome/ to follow C++11 style. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Cleanups Created 5 years, 11 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 "chrome/browser/plugins/plugin_installer.h" 5 #include "chrome/browser/plugins/plugin_installer.h"
6 6
7 #include "base/memory/scoped_ptr.h" 7 #include "base/memory/scoped_ptr.h"
8 #include "base/run_loop.h" 8 #include "base/run_loop.h"
9 #include "chrome/browser/plugins/plugin_installer_observer.h" 9 #include "chrome/browser/plugins/plugin_installer_observer.h"
10 #include "chrome/test/base/chrome_render_view_host_test_harness.h" 10 #include "chrome/test/base/chrome_render_view_host_test_harness.h"
11 #include "content/public/browser/download_url_parameters.h" 11 #include "content/public/browser/download_url_parameters.h"
12 #include "content/public/test/mock_download_item.h" 12 #include "content/public/test/mock_download_item.h"
13 #include "content/public/test/mock_download_manager.h" 13 #include "content/public/test/mock_download_manager.h"
14 #include "testing/gmock/include/gmock/gmock.h" 14 #include "testing/gmock/include/gmock/gmock.h"
15 #include "testing/gtest/include/gtest/gtest.h" 15 #include "testing/gtest/include/gtest/gtest.h"
16 16
17 using ::testing::_; 17 using ::testing::_;
18 18
19 namespace { 19 namespace {
20 20
21 class PluginInstallerTest : public ChromeRenderViewHostTestHarness { 21 class PluginInstallerTest : public ChromeRenderViewHostTestHarness {
22 public: 22 public:
23 PluginInstallerTest(); 23 PluginInstallerTest();
24 virtual void SetUp() override; 24 void SetUp() override;
25 virtual void TearDown() override; 25 void TearDown() override;
26 26
27 PluginInstaller* installer() { return installer_.get(); } 27 PluginInstaller* installer() { return installer_.get(); }
28 28
29 scoped_ptr<content::MockDownloadItem> CreateMockDownloadItem(); 29 scoped_ptr<content::MockDownloadItem> CreateMockDownloadItem();
30 30
31 private: 31 private:
32 scoped_ptr<PluginInstaller> installer_; 32 scoped_ptr<PluginInstaller> installer_;
33 }; 33 };
34 34
35 PluginInstallerTest::PluginInstallerTest() { 35 PluginInstallerTest::PluginInstallerTest() {
(...skipping 144 matching lines...) Expand 10 before | Expand all | Expand 10 after
180 EXPECT_CALL(*download_item, GetState()) 180 EXPECT_CALL(*download_item, GetState())
181 .WillOnce(testing::Return(content::DownloadItem::INTERRUPTED)); 181 .WillOnce(testing::Return(content::DownloadItem::INTERRUPTED));
182 EXPECT_CALL(*download_item, GetLastReason()).WillOnce( 182 EXPECT_CALL(*download_item, GetLastReason()).WillOnce(
183 testing::Return(content::DOWNLOAD_INTERRUPT_REASON_NETWORK_FAILED)); 183 testing::Return(content::DOWNLOAD_INTERRUPT_REASON_NETWORK_FAILED));
184 download_item->NotifyObserversDownloadUpdated(); 184 download_item->NotifyObserversDownloadUpdated();
185 185
186 EXPECT_TRUE(installer_observer.download_started()); 186 EXPECT_TRUE(installer_observer.download_started());
187 EXPECT_FALSE(installer_observer.download_finished()); 187 EXPECT_FALSE(installer_observer.download_finished());
188 EXPECT_EQ("NETWORK_FAILED", installer_observer.download_error()); 188 EXPECT_EQ("NETWORK_FAILED", installer_observer.download_error());
189 } 189 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698