LayoutTests/fast/dom/StyleSheet/stylesheet-move-between-documents-crash.html - Issue 721103002: Fix lfiespan of ScopedStyleResolver

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/fast/dom/StyleSheet/stylesheet-move-between-documents-crash.html

Issue 721103002: Fix lfiespan of ScopedStyleResolver (Closed) Base URL: svn://svn.chromium.org/blink/trunk

Patch Set: add comment in layout test. Created 6 years, 1 month 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 | Annotate | Revision Log

« no previous file with comments | « LayoutTests/fast/dom/StyleSheet/resources/stylesheet-move-iframe2.html ('k') | LayoutTests/fast/dom/StyleSheet/stylesheet-move-between-documents-crash-expected.txt » ('j') | Source/core/css/resolver/StyleResolver.h » ('J')
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

OLD	NEW
(Empty)
	1 <script>

	2 if (window.testRunner)

	3 testRunner.waitUntilDone();

	4

	5 var iframe;

	6 var div;

	7

	8 function start() {

	9 iframe = document.createElement('iframe');

	10 iframe.src='resources/stylesheet-move-iframe1.xml';

	11 document.getElementById('store_div').appendChild(iframe);

	12 window.setTimeout('startrly()', 100);

	13 }

	14

	15 function startrly() {

	16 if (location.hash == '#done') {

	17 document.write('PASS');

	18 if (window.testRunner) {

	19 testRunner.dumpAsText();

	20 testRunner.notifyDone();

	21 }

	22 return;

	23 }

	24 var count = 0;

	25 if (location.hash !== "")

	26 count = parseInt(location.hash.substr(1));

	27 if (count < 10) {

	28 ++count;

	29 location.hash = '#' + count;

	30 } else {

	31 location.hash = 'done';

	32 }

	33 div = document.createElement('div');

	34 // <marquee> is special as it is implemented using Blink-in-JS (JS code runs for

	35 // creating a custom element and a shadow root) and invokes its attachedCall back

	36 // from C++ code, which can cause a special race condition with reloading wh en

	37 // it is moved between different documents.

	38 marquee = document.createElement('marquee');
	hayato 2014/11/19 10:52:15 Looks 'var' is missing for marquee. Looks 'var' is missing for marquee. kochi 2014/11/19 11:41:52 Done. Show quoted text On 2014/11/19 10:52:15, hayato wrote: > Looks 'var' is missing for marquee. Done.
	39 document.body.appendChild(marquee);

	40 document.body.scrollIntoView(false);

	41

	42 var iframe2 = document.createElement('iframe');

	43 iframe2.src='resources/stylesheet-move-iframe2.html';
	hayato 2014/11/19 10:52:15 spaces around '='. spaces around '='. kochi 2014/11/19 11:41:52 Done. Show quoted text On 2014/11/19 10:52:15, hayato wrote: > spaces around '='. Done.
	44 document.body.appendChild(iframe2);

	45

	46 document.body.appendChild(iframe.contentDocument.firstChild);

	47 div.appendChild(marquee);

	48 window.setTimeout('location.reload();',20);

	49 }

	50 </script>

	51 <body onload="start()">

	52 <div id="store_div"></div>

	53 </script>

OLD	NEW