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

Side by Side Diff: extensions/browser/api/system_cpu/system_cpu_apitest.cc

Issue 2802593005: extensions: Add CPU temperatures field to system.cpu.getInfo() function (Closed)
Patch Set: Responded to tbarzic's feedback 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 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 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 "extensions/browser/api/system_cpu/cpu_info_provider.h" 5 #include "extensions/browser/api/system_cpu/cpu_info_provider.h"
6 #include "extensions/shell/test/shell_apitest.h" 6 #include "extensions/shell/test/shell_apitest.h"
7 7
8 namespace extensions { 8 namespace extensions {
9 9
10 using api::system_cpu::CpuInfo; 10 using api::system_cpu::CpuInfo;
(...skipping 10 matching lines...) Expand all
21 info_.features.clear(); 21 info_.features.clear();
22 info_.features.push_back("mmx"); 22 info_.features.push_back("mmx");
23 info_.features.push_back("avx"); 23 info_.features.push_back("avx");
24 24
25 info_.processors.clear(); 25 info_.processors.clear();
26 info_.processors.push_back(api::system_cpu::ProcessorInfo()); 26 info_.processors.push_back(api::system_cpu::ProcessorInfo());
27 info_.processors[0].usage.kernel = 1; 27 info_.processors[0].usage.kernel = 1;
28 info_.processors[0].usage.user = 2; 28 info_.processors[0].usage.user = 2;
29 info_.processors[0].usage.idle = 3; 29 info_.processors[0].usage.idle = 3;
30 info_.processors[0].usage.total = 6; 30 info_.processors[0].usage.total = 6;
31
32 // The fractional part of these values should be exactly represented as
33 // floating points to avoid rounding errors.
34 info_.temperatures = {30.125, 40.0625};
31 return true; 35 return true;
32 } 36 }
33 37
34 private: 38 private:
35 ~MockCpuInfoProviderImpl() override {} 39 ~MockCpuInfoProviderImpl() override {}
36 }; 40 };
37 41
38 class SystemCpuApiTest : public ShellApiTest { 42 class SystemCpuApiTest : public ShellApiTest {
39 public: 43 public:
40 SystemCpuApiTest() {} 44 SystemCpuApiTest() {}
41 ~SystemCpuApiTest() override {} 45 ~SystemCpuApiTest() override {}
42 46
43 void SetUpInProcessBrowserTestFixture() override { 47 void SetUpInProcessBrowserTestFixture() override {
44 ShellApiTest::SetUpInProcessBrowserTestFixture(); 48 ShellApiTest::SetUpInProcessBrowserTestFixture();
45 } 49 }
46 }; 50 };
47 51
48 IN_PROC_BROWSER_TEST_F(SystemCpuApiTest, Cpu) { 52 IN_PROC_BROWSER_TEST_F(SystemCpuApiTest, Cpu) {
49 CpuInfoProvider* provider = new MockCpuInfoProviderImpl(); 53 CpuInfoProvider* provider = new MockCpuInfoProviderImpl();
50 // The provider is owned by the single CpuInfoProvider instance. 54 // The provider is owned by the single CpuInfoProvider instance.
51 CpuInfoProvider::InitializeForTesting(provider); 55 CpuInfoProvider::InitializeForTesting(provider);
52 ASSERT_TRUE(RunAppTest("system/cpu")) << message_; 56 ASSERT_TRUE(RunAppTest("system/cpu")) << message_;
53 } 57 }
54 58
55 } // namespace extensions 59 } // namespace extensions
OLDNEW
« no previous file with comments | « extensions/browser/api/system_cpu/cpu_info_provider.cc ('k') | extensions/common/api/system_cpu.idl » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698