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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: third_party/WebKit/LayoutTests/editing/spelling/spellcheck-remove-markers.html

Issue 2650183002: Remove spelling markers when element is not editable (Closed)
Patch Set: Apply suggested changes 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | third_party/WebKit/Source/core/editing/spellcheck/SpellChecker.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: third_party/WebKit/LayoutTests/editing/spelling/spellcheck-remove-markers.html
diff --git a/third_party/WebKit/LayoutTests/editing/spelling/spellcheck-remove-markers.html b/third_party/WebKit/LayoutTests/editing/spelling/spellcheck-remove-markers.html
new file mode 100644
index 0000000000000000000000000000000000000000..8696e381922164ad4a427043aca5ffdcf77fefa4
--- /dev/null
+++ b/third_party/WebKit/LayoutTests/editing/spelling/spellcheck-remove-markers.html
@@ -0,0 +1,99 @@
+<!DOCTYPE html>
+<script src="../../resources/testharness.js"></script>
+<script src="../../resources/testharnessreport.js"></script>
+<script src="../assert_selection.js"></script>
+<script src="spellcheck_test.js"></script>
+
+<script>
+ spellcheck_test(
+ '<div contenteditable>|zz. I have a issue.</div>',
+ '',
+ '<div contenteditable>#zz#. I have ~a~ issue.</div>',
+ {
+ title: '1) Setup for the initial markers.',
+ callback: sample => spellcheck_test(
+ sample,
+ document => document.querySelector('div').removeAttribute('contenteditable'),
+ '<div>zz. I have a issue.</div>',
+ 'Removing "contenteditable" attribute removes spelling markers.')
+ });
+
+ spellcheck_test(
+ '<div contenteditable>|zz. I have a issue.</div>',
+ '',
+ '<div contenteditable>#zz#. I have ~a~ issue.</div>',
+ {
+ title: '2) Setup for the initial markers.',
+ callback: sample => spellcheck_test(
+ sample,
+ document => document.querySelector('div').setAttribute('contenteditable', 'false'),
+ '<div contenteditable="false">zz. I have a issue.</div>',
+ 'Setting "contenteditable" attribute to "false" removes spelling markers.')
+ });
+
+ spellcheck_test(
+ '<div contenteditable id="parent">zz.<div>|zz.</div>zz.</div>',
+ '',
+ '<div contenteditable id="parent">#zz#.<div>#zz#.</div>#zz#.</div>',
+ {
+ title: '3) Setup for the initial markers.',
+ callback: sample => spellcheck_test(
+ sample,
+ document => document.getElementById('parent').removeAttribute('contenteditable'),
+ '<div id="parent">zz.<div>zz.</div>zz.</div>',
+ 'Removing "contenteditable" attribute from parent removes spelling markers.')
+ });
+
+ spellcheck_test(
+ '<div contenteditable id="parent">zz.<div>|zz.</div>zz.</div>',
+ '',
+ '<div contenteditable id="parent">#zz#.<div>#zz#.</div>#zz#.</div>',
+ {
+ title: '4) Setup for the initial markers.',
+ callback: sample => spellcheck_test(
+ sample,
+ document => document.getElementById('parent').setAttribute('contenteditable', 'false'),
+ '<div contenteditable="false" id="parent">zz.<div>zz.</div>zz.</div>',
+ 'Setting "contenteditable" attribute to "false" on parent removes spelling markers.')
+ });
+
+ spellcheck_test(
+ '<div contenteditable>zz.<div id="child">|zz.</div>zz.</div>',
+ '',
+ '<div contenteditable>#zz#.<div id="child">#zz#.</div>#zz#.</div>',
+ {
+ title: '5) Setup for the initial markers.',
+ callback: sample => spellcheck_test(
+ sample,
+ document => document.getElementById('child').setAttribute('contenteditable', 'false'),
+ '<div contenteditable>#zz#.<div contenteditable="false" id="child">zz.</div>#zz#.</div>',
+ 'Setting "contenteditable" attribute to "false" on child removes spelling markers.')
+ });
+
+ spellcheck_test(
+ '<div contenteditable id="parent">zz.<div contenteditable>|zz.</div>zz.</div>',
+ '',
+ '<div contenteditable id="parent">#zz#.<div contenteditable>#zz#.</div>#zz#.</div>',
+ {
+ title: '6) Setup for the initial markers.',
+ callback: sample => spellcheck_test(
+ sample,
+ document => document.getElementById('parent').removeAttribute('contenteditable'),
+ '<div id="parent">zz.<div contenteditable>#zz#.</div>zz.</div>',
+ 'Removing "contenteditable" attribute on parent still keeps the spelling markers on child.')
+ });
+
+ spellcheck_test(
+ '<div contenteditable>zz.<div contenteditable id="child">|zz.</div>zz.</div>',
+ '',
+ '<div contenteditable>#zz#.<div contenteditable id="child">#zz#.</div>#zz#.</div>',
+ {
+ title: '7) Setup for the initial markers.',
+ callback: sample => spellcheck_test(
+ sample,
+ document => document.getElementById('child').removeAttribute('contenteditable'),
+ '<div contenteditable>#zz#.<div id="child">#zz#.</div>#zz#.</div>',
+ 'Removing "contenteditable" attribute on child still keeps the spelling markers.')
+ });
+
+</script>
« no previous file with comments | « no previous file | third_party/WebKit/Source/core/editing/spellcheck/SpellChecker.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698