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

Unified Diff: chrome/test/data/extensions/api_test/systeminfo/cpu/test_cpuinfo_api.js

Issue 10914060: Refine systemInfo.cpu API defintions and provide systemInfo.cpu.get impl for Windows and Linux (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 years, 3 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 side-by-side diff with in-line comments
Download patch
Index: chrome/test/data/extensions/api_test/systeminfo/cpu/test_cpuinfo_api.js
diff --git a/chrome/test/data/extensions/api_test/systeminfo/cpu/test_cpuinfo_api.js b/chrome/test/data/extensions/api_test/systeminfo/cpu/test_cpuinfo_api.js
index 5bf45546269da4cb26ce5cbeeb9ae53168c63b16..7f466a827fb2a80c31d59194d28e3b78521946e4 100644
--- a/chrome/test/data/extensions/api_test/systeminfo/cpu/test_cpuinfo_api.js
+++ b/chrome/test/data/extensions/api_test/systeminfo/cpu/test_cpuinfo_api.js
@@ -11,12 +11,9 @@ chrome.test.runTests([
function testGet() {
for(var i = 0; i < 20; ++i) {
chrome.systemInfo.cpu.get(chrome.test.callbackPass(function(result) {
- chrome.test.assertTrue(result.cores.length == 4);
- var core;
- for (var i = 0; i < result.cores.length; ++i) {
- core = result.cores[i];
- chrome.test.assertTrue(core.load == i*10);
- }
+ chrome.test.assertTrue(result.numOfProcessors == 4);
Mihai Parparita -not on Chrome 2012/09/05 23:58:39 Still not seeing assertEq here.
Hongbo Min 2012/09/06 01:43:53 Done.
+ chrome.test.assertTrue(result.archName == "x86");
+ chrome.test.assertTrue(result.modelName == "unknown");
}));
}
},

Powered by Google App Engine
This is Rietveld 408576698