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

Unified Diff: third_party/WebKit/LayoutTests/media/mediasession/mediasession-constructor-hidden.html

Issue 2401013002: Make MediaSession per frame as an attribute of Navigator (Closed)
Patch Set: fixed tests Created 4 years, 2 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: third_party/WebKit/LayoutTests/media/mediasession/mediasession-constructor-hidden.html
diff --git a/third_party/WebKit/LayoutTests/fast/dom/canvas-fallback-focus-crash.html b/third_party/WebKit/LayoutTests/media/mediasession/mediasession-constructor-hidden.html
similarity index 52%
copy from third_party/WebKit/LayoutTests/fast/dom/canvas-fallback-focus-crash.html
copy to third_party/WebKit/LayoutTests/media/mediasession/mediasession-constructor-hidden.html
index a02efb0a2a890a292cc98a5ab9b0a426a05518f7..1bb4a95ead380c56a4e414d81efb6d18c5227f7e 100644
--- a/third_party/WebKit/LayoutTests/fast/dom/canvas-fallback-focus-crash.html
+++ b/third_party/WebKit/LayoutTests/media/mediasession/mediasession-constructor-hidden.html
@@ -1,11 +1,11 @@
<!DOCTYPE html>
+<title>MediaSession() constructor</title>
whywhat 2016/10/10 20:45:09 hm, do we need a test for that? I haven't found an
Zhiqiang Zhang (Slow) 2016/10/10 21:21:18 OK, I'll remove this test. These is a test for the
<script src="../../resources/testharness.js"></script>
<script src="../../resources/testharnessreport.js"></script>
-<canvas>
- <object id="f"></object>
-</canvas>
<script>
+
test(function() {
- f.focus();
-}, "crbug.com/587263 focusing plugin object in canvas fallback may dereference nullptr");
+ assert_throws(new TypeError(), () => { var mediaSession = new MediaSession(); }, "Illegal constructor");
+});
+
</script>

Powered by Google App Engine
This is Rietveld 408576698