third_party/WebKit/LayoutTests/fast/css/background-image-url-setproperty.html - Issue 2622993003: Compare relative url if absolute urls are empty for CSSImageValue

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: third_party/WebKit/LayoutTests/fast/css/background-image-url-setproperty.html

Issue 2622993003: Compare relative url if absolute urls are empty for CSSImageValue (Closed)

Patch Set: add a test case Created 3 years, 11 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

OLD	NEW
(Empty)
	1 <p id="result">
	rune 2017/01/11 14:13:13 Test should be strict unless specifically testing Test should be strict unless specifically testing quirks. Use <!DOCTYPE html>
	2 </p>

	3 <script>

	4 if (window.testRunner)

	5 testRunner.dumpAsText();

	6

	7 var rulesForCssTextFromBrowser = function (styleContent) {

	8 var doc = document.implementation.createHTMLDocument(""),

	9 styleElement = document.createElement("style"),

	10 rules;

	11

	12 styleElement.textContent = styleContent;

	13 // the style will only be parsed once it is added to a document

	14 doc.body.appendChild(styleElement);

	15 rules = styleElement.sheet.cssRules;

	16

	17 return Array.prototype.slice.call(rules);

	18 };

	19

	20 var rules = rulesForCssTextFromBrowser("span { background: url('aaaaaaa'); } ");

	21 var rule = rules[0];

	22 rule.style.setProperty('background', 'url("bbbbbbb")', '');

	23 var url = rule.style.getPropertyValue('background');

	24 document.getElementById("result").innerText = url == "url(\"bbbbbbb\")" ? "P ASS" : "FAIL";

	25 </script>
	rune 2017/01/11 14:13:14 We prefer using testharness.js. Then you should dr We prefer using testharness.js. Then you should drop the expectation file as well: <!DOCTYPE html> <script src="../../resources/testharness.js"></script> <script src="../../resources/testharnessreport.js"></script> <script> var doc = document.implementation.createHTMLDocument(""); doc.head.innerHTML = "<style>span { background: url(fail) }</style>"; var rule = doc.head.firstChild.sheet.cssRules[0]; test(() => { assert_equals(rule.style.background, 'url("fail")'); }, "Check initial background url."); test(() => { rule.style.setProperty("background", "url(pass)"); assert_equals(rule.style.background, 'url("pass")'); }, "Setting background to a relative url should change its value."); </script>
OLD	NEW

« no previous file with comments | « no previous file | third_party/WebKit/LayoutTests/fast/css/background-image-url-setproperty-expected.txt » ('j') | no next file with comments »