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

Side by Side Diff: chrome/browser/ui/webui/help/version_updater_chromeos_unittest.cc

Issue 2873193002: Make update over cellular an option for user (Closed)
Patch Set: Put code in CHROME_OS wrapper to fix trybot error Created 3 years, 7 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 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 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/ui/webui/help/version_updater_chromeos.h" 5 #include "chrome/browser/ui/webui/help/version_updater_chromeos.h"
6 6
7 #include <memory> 7 #include <memory>
8 8
9 #include "base/bind.h" 9 #include "base/bind.h"
10 #include "base/compiler_specific.h" 10 #include "base/compiler_specific.h"
(...skipping 14 matching lines...) Expand all
25 25
26 using ::testing::AtLeast; 26 using ::testing::AtLeast;
27 using ::testing::Return; 27 using ::testing::Return;
28 28
29 namespace chromeos { 29 namespace chromeos {
30 30
31 namespace { 31 namespace {
32 32
33 void CheckNotification(VersionUpdater::Status /* status */, 33 void CheckNotification(VersionUpdater::Status /* status */,
34 int /* progress */, 34 int /* progress */,
35 const base::string16& /* message */) { 35 const std::string& /* version */,
36 } 36 int64_t /* size */,
37 const base::string16& /* message */) {}
37 38
38 } // namespace 39 } // namespace
39 40
40 class VersionUpdaterCrosTest : public ::testing::Test { 41 class VersionUpdaterCrosTest : public ::testing::Test {
41 protected: 42 protected:
42 VersionUpdaterCrosTest() 43 VersionUpdaterCrosTest()
43 : version_updater_(VersionUpdater::Create(nullptr)), 44 : version_updater_(VersionUpdater::Create(nullptr)),
44 fake_update_engine_client_(NULL), 45 fake_update_engine_client_(NULL),
45 mock_user_manager_(new MockUserManager()), 46 mock_user_manager_(new MockUserManager()),
46 user_manager_enabler_(mock_user_manager_) {} 47 user_manager_enabler_(mock_user_manager_) {}
(...skipping 98 matching lines...) Expand 10 before | Expand all | Expand 10 after
145 UpdateEngineClient::Status status; 146 UpdateEngineClient::Status status;
146 status.status = UpdateEngineClient::UPDATE_STATUS_IDLE; 147 status.status = UpdateEngineClient::UPDATE_STATUS_IDLE;
147 fake_update_engine_client_->set_default_status(status); 148 fake_update_engine_client_->set_default_status(status);
148 fake_update_engine_client_->NotifyObserversThatStatusChanged(status); 149 fake_update_engine_client_->NotifyObserversThatStatusChanged(status);
149 } 150 }
150 151
151 EXPECT_EQ(2, fake_update_engine_client_->request_update_check_call_count()); 152 EXPECT_EQ(2, fake_update_engine_client_->request_update_check_call_count());
152 } 153 }
153 154
154 } // namespace chromeos 155 } // namespace chromeos
OLDNEW
« no previous file with comments | « chrome/browser/ui/webui/help/version_updater_chromeos.cc ('k') | chrome/browser/ui/webui/help/version_updater_mac.mm » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698