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

Issues Search
    My Issues | Starred     Open | Closed | All

Side by Side Diff: LayoutTests/fast/dom/Window/resources/window-property-collector.js

Issue 262723002: Implement navigator.hardwareConcurrency (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Address reviewer comments Created 6 years, 6 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 | Annotate | Revision Log
« no previous file with comments | « LayoutTests/fast/dom/Window/property-access-on-cached-window-after-frame-removed-expected.txt ('k') | LayoutTests/fast/workers/resources/worker-navigator-hardware-concurrency.js » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 function collectProperties() 1 function collectProperties()
2 { 2 {
3 // Collect properties of the top-level window, since touching the properties 3 // Collect properties of the top-level window, since touching the properties
4 // of a DOMWindow affects its internal C++ state. 4 // of a DOMWindow affects its internal C++ state.
5 collectPropertiesHelper(window, []); 5 collectPropertiesHelper(window, []);
6 6
7 propertiesToVerify.sort(function (a, b) 7 propertiesToVerify.sort(function (a, b)
8 { 8 {
9 if (a.property < b.property) 9 if (a.property < b.property)
10 return -1 10 return -1
(...skipping 40 matching lines...) Expand 10 before | Expand all | Expand 10 after
51 expected = "'null'"; 51 expected = "'null'";
52 break; 52 break;
53 case "location.pathname": 53 case "location.pathname":
54 expected = "'blank'"; 54 expected = "'blank'";
55 break; 55 break;
56 case "location.protocol": 56 case "location.protocol":
57 expected = "'about:'"; 57 expected = "'about:'";
58 break; 58 break;
59 case "navigator.appCodeName": 59 case "navigator.appCodeName":
60 case "navigator.appName": 60 case "navigator.appName":
61 case "navigator.hardwareConcurrency":
61 case "navigator.language": 62 case "navigator.language":
62 case "navigator.onLine": 63 case "navigator.onLine":
63 case "navigator.platform": 64 case "navigator.platform":
64 case "navigator.product": 65 case "navigator.product":
65 case "navigator.productSub": 66 case "navigator.productSub":
66 case "navigator.vendor": 67 case "navigator.vendor":
67 case "navigator.connection.type": 68 case "navigator.connection.type":
68 expected = "window." + propertyPath; 69 expected = "window." + propertyPath;
69 break; 70 break;
70 case "screen.orientation": 71 case "screen.orientation":
(...skipping 25 matching lines...) Expand all
96 } else if (type == "string") { 97 } else if (type == "string") {
97 emitExpectedResult(path, "''"); 98 emitExpectedResult(path, "''");
98 } else if (type == "number") { 99 } else if (type == "number") {
99 emitExpectedResult(path, "0"); 100 emitExpectedResult(path, "0");
100 } else if (type == "boolean") { 101 } else if (type == "boolean") {
101 emitExpectedResult(path, "false"); 102 emitExpectedResult(path, "false");
102 } 103 }
103 path.pop(); 104 path.pop();
104 } 105 }
105 } 106 }
OLDNEW
« no previous file with comments | « LayoutTests/fast/dom/Window/property-access-on-cached-window-after-frame-removed-expected.txt ('k') | LayoutTests/fast/workers/resources/worker-navigator-hardware-concurrency.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698