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

Side by Side Diff: third_party/WebKit/LayoutTests/custom-elements/spec/resources/custom-elements-helpers.js

Issue 2456773002: Clear the custom element's reaction queue if upgrade fails. (Closed)
Patch Set: Created 4 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
OLDNEW
1 function create_window_in_test(t, srcdoc) { 1 function create_window_in_test(t, srcdoc) {
2 let p = new Promise((resolve) => { 2 let p = new Promise((resolve) => {
3 let f = document.createElement('iframe'); 3 let f = document.createElement('iframe');
4 f.srcdoc = srcdoc ? srcdoc : ''; 4 f.srcdoc = srcdoc ? srcdoc : '';
5 f.onload = (event) => { 5 f.onload = (event) => {
6 let w = f.contentWindow; 6 let w = f.contentWindow;
7 t.add_cleanup(() => f.parentNode && f.remove()); 7 t.add_cleanup(() => f.parentNode && f.remove());
8 resolve(w); 8 resolve(w);
9 }; 9 };
10 document.body.appendChild(f); 10 document.body.appendChild(f);
(...skipping 47 matching lines...) Expand 10 before | Expand all | Expand 10 after
58 errors.push(error); 58 errors.push(error);
59 return true; // the error is handled 59 return true; // the error is handled
60 }; 60 };
61 try { 61 try {
62 func(); 62 func();
63 } catch (e) { 63 } catch (e) {
64 assert_unreached(`should report, not throw, an exception: ${e}`); 64 assert_unreached(`should report, not throw, an exception: ${e}`);
65 } finally { 65 } finally {
66 w.onerror = old_onerror; 66 w.onerror = old_onerror;
67 } 67 }
68 assert_equals(1, errors.length); 68 assert_equals(errors.length, 1, 'only one error should have been reported');
69 // assert_throws has an error expectation matcher that can only be 69 // assert_throws has an error expectation matcher that can only be
70 // accessed by throwing the error 70 // accessed by throwing the error
71 assert_throws(expected_error, () => { throw errors[0]; }); 71 assert_throws(expected_error, () => { throw errors[0]; });
72 } 72 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698