LayoutTests/screen_orientation/resources/sandboxed-iframe-locking.html - Issue 283423005: Use Promises for screen.lockOrientation().

Keyboard Shortcuts

	File
u :	up to issue
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line

	Issue
u :	up to list of issues
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue

Unified Diff: LayoutTests/screen_orientation/resources/sandboxed-iframe-locking.html

Issue 283423005: Use Promises for screen.lockOrientation(). (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master

Patch Set: review commenst Created 6 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 side-by-side diff with in-line comments

Download patch

Index: LayoutTests/screen_orientation/resources/sandboxed-iframe-locking.html

diff --git a/LayoutTests/screen_orientation/resources/sandboxed-iframe-locking.html b/LayoutTests/screen_orientation/resources/sandboxed-iframe-locking.html

index cc931a49fb8f982d2799fc4e92b20161486be216..8b81f64b91202f68bf917c5b1cf46f2d7a697154 100644

--- a/LayoutTests/screen_orientation/resources/sandboxed-iframe-locking.html

+++ b/LayoutTests/screen_orientation/resources/sandboxed-iframe-locking.html

@@ -1,9 +1,17 @@

+<!DOCTYPE html>

+<html>

- var exceptionName = "";

- try {

- screen.lockOrientation("portrait-primary");

- } catch (ex) {

- exceptionName = ex.name;

- }

- parent.window.postMessage(exceptionName, "*");

+ var msg = "";

+ screen.lockOrientation("portrait-primary").then(function() {

+ }, function(error) {

+ msg = error.name;

+ });

+ // FIXME: for the moment, tests gets notified when there is a failure but

+ // not a success, this hack should allow us to still have the test running.

+ // That should be removed as soon as we get a resolved promise in tests.

+ setTimeout(function() {

+ parent.window.postMessage(msg, "*");

+ })

</script>

+</html>

« no previous file with comments | « LayoutTests/screen_orientation/lockOrientation-basic-expected.txt ('k') | Source/modules/modules.gypi » ('j') | no next file with comments »