LayoutTests/webexposed/internal-properties-should-not-be-exposed.html - Issue 1274643002: CSSStyleDeclaraction returns empty string

Side by Side Diff

Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.

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

Side by Side Diff: LayoutTests/webexposed/internal-properties-should-not-be-exposed.html

Issue 1274643002: CSSStyleDeclaraction returns empty string (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master

Patch Set: Removed changes from imported/csswg-test Created 5 years, 4 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

« LayoutTests/imported/csswg-test/css-shapes-1/shape-outside/values/shape-image-threshold-002-expected.txt ('K') | « LayoutTests/virtual/stable/webexposed/nonstable-css-properties-expected.txt ('k') | LayoutTests/webexposed/internal-properties-should-not-be-exposed-expected.txt » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

OLD	NEW
1 <!DOCTYPE html>	1 <!DOCTYPE html>

2 <body>	2 <body>

3 <script src="../resources/js-test.js"></script>	3 <script src="../resources/js-test.js"></script>

4 <script>	4 <script>

5 debug("This test checks that -internal-* properties are not exposed to JS.")	5 debug("This test checks that -internal-* properties are not exposed to JS.")

6 debug("");	6 debug("");

7 debug("");	7 debug("");

8	8

9 // This contains the properties along with an accepted value to ensure that we c an't	9 // This contains the properties along with an accepted value to ensure that we c an't

10 // set it from JavaScript.	10 // set it from JavaScript.

11 var internalProperties = { "-internal-marquee-direction": "alternate" };	11 var internalProperties = { "-internal-marquee-direction": "alternate" };

12 for (var property in internalProperties) {	12 for (var property in internalProperties) {

13 debug("Testing " + property);	13 debug("Testing " + property);

14 element = document.createElement("foo");	14 element = document.createElement("foo");

15 document.body.appendChild(element);	15 document.body.appendChild(element);

16 shouldBeNull("window.getComputedStyle(element).getPropertyValue(property)")	16 shouldBeEqualToString("window.getComputedStyle(element).getPropertyValue(pro perty)", "");

17 shouldBeUndefined("element.style[property]")	17 shouldBeUndefined("element.style[property]");

18	18

19 debug("Value after setting " + property + " to " + internalProperties[proper ty]);	19 debug("Value after setting " + property + " to " + internalProperties[proper ty]);

20 element.style.setProperty(property, internalProperties[property]);	20 element.style.setProperty(property, internalProperties[property]);

21 shouldBeUndefined("element.style[property]")	21 shouldBeUndefined("element.style[property]")

22	22

23 document.body.removeChild(element);	23 document.body.removeChild(element);

24 debug("");	24 debug("");

25 }	25 }

26 </script>	26 </script>

27 </body>	27 </body>

OLD	NEW