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

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

Issue 203053002: Add null check for frame when creating WebCore::Performance. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Add rebaseline expectations Created 6 years, 9 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: LayoutTests/fast/dom/Window/resources/window-property-collector.js
diff --git a/LayoutTests/fast/dom/Window/resources/window-property-collector.js b/LayoutTests/fast/dom/Window/resources/window-property-collector.js
index efb5b21ef03de39a831412204c6bdeee3bcbe9f2..1bdbce2de7a24a485c44327a1474a627593cecb3 100644
--- a/LayoutTests/fast/dom/Window/resources/window-property-collector.js
+++ b/LayoutTests/fast/dom/Window/resources/window-property-collector.js
@@ -35,7 +35,7 @@ function emitExpectedResult(path, expected)
if (path[path.length - 1].toUpperCase() == path[path.length - 1])
return;
- // Various special cases...
+ // Various special cases for legacy reasons. Please do not add entries to this list.
var propertyPath = path.join('.');
switch (propertyPath) {
case "location.href":
@@ -60,18 +60,6 @@ function emitExpectedResult(path, expected)
case "navigator.vendor":
expected = "window." + propertyPath;
break;
- case "navigator.battery.charging":
- expected = "true";
- break;
- case "navigator.battery.chargingTime":
- expected = "Infinity";
- break;
- case "navigator.battery.dischargingTime":
- expected = "Infinity";
- break;
- case "navigator.battery.level":
- expected = "1";
- break;
}
insertExpectedResult(path, expected);

Powered by Google App Engine
This is Rietveld 408576698