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

Side by Side Diff: third_party/WebKit/LayoutTests/editing/execCommand/crash-inserting-span.html

Issue 2135993003: Add null check to fix crash in blink::ComputedStyle::display (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: add a test case Created 4 years, 5 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
« no previous file with comments | « no previous file | third_party/WebKit/Source/core/editing/EditingUtilities.cpp » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
(Empty)
1 <script src="../../resources/testharness.js"></script>
2 <script src="../../resources/testharnessreport.js"></script>
3 <script>
4 // This test case was created by ClusterFuzz.
5 // See http://crbug.com/626991
6 function runTest() {
7 // The body of description() is defined in third_party/webgl/src/sdk/tests/j s/js-test-pre.js
8 msg = document.title;
9 // For MSIE 6 compatibility
10 var span = document.createElement("span");
11 span.innerHTML = '<p>' + msg + '</p><p>On success, you will see a series of "<span class="pass">PASS</span>" messages, followed by "<span class="pass">TEST COMPLETE</span>".</p>';
12 var description = document.getElementById("description");
13 if (description.firstChild)
14 description.replaceChild(span, description.firstChild);
15 else
16 description.appendChild(span);
17 }
18 </script>
19 <body onload="runTest()">
20 <script>
21 function fuzz() {
22 document.designMode = 'on';
23 document.execCommand("selectAll");
24 document.execCommand("CreateLink", 0, "foo");
25 document.execCommand("inserthtml", false, "<span id='green' style='color:gre en'>green</span>");
yosin_UTC9 2016/07/13 02:19:43 Could you try to simplify test case? We need to ha
joone 2016/07/13 04:49:25 Okay, I will try to add HTML code instead of runni
26 }
27 test(function() {
28 setTimeout(fuzz);
29 }, 'run setTimeout');
30 </script>
31 <div id="description"></div>
32 </body>
OLDNEW
« no previous file with comments | « no previous file | third_party/WebKit/Source/core/editing/EditingUtilities.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698