Attachment 361972 Details for Bug 194619 – Patch

[patch] Patch

patch.txt (text/plain), 7.15 KB, created by Nikita Vasilyev on 2019-02-13 17:23:28 PST

(hide)

Description:

Filename:

MIME Type:

Creator: Nikita Vasilyev

Created: 2019-02-13 17:23:28 PST

Size: 7.15 KB

patch

obsolete

>diff --git a/LayoutTests/ChangeLog b/LayoutTests/ChangeLog
>index cb919c2bd06..c8b6c17702b 100644
>--- a/LayoutTests/ChangeLog
>+++ b/LayoutTests/ChangeLog
>@@ -1,3 +1,14 @@
>+2019-02-13  Nikita Vasilyev  <nvasilyev@apple.com>
>+
>+        Web Inspector: Styles: valid values in style attributes are reported as unsupported property values
>+        https://bugs.webkit.org/show_bug.cgi?id=194619
>+        <rdar://problem/47917373>
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        * inspector/css/modify-inline-style-expected.txt: Added.
>+        * inspector/css/modify-inline-style.html: Added.
>+
> 2019-02-09  Nikita Vasilyev  <nvasilyev@apple.com>
> 
>         Web Inspector: fix typos in tests
>diff --git a/LayoutTests/inspector/css/modify-inline-style-expected.txt b/LayoutTests/inspector/css/modify-inline-style-expected.txt
>new file mode 100644
>index 00000000000..a9063a909c9
>--- /dev/null
>+++ b/LayoutTests/inspector/css/modify-inline-style-expected.txt
>@@ -0,0 +1,18 @@
>+Testing that adding and editing CSS properties of inline styles works.
>+
>+
>+== Running test suite: ModifyInlineStyle
>+-- Running test case: AddPropertyAndEdit
>+CSSProperty changed to "font: 12px normal sans-serif!important;".
>+font: 12px normal sans-serif!important;
>+
>+CSSProperty changed to "color: red;".
>+font: 12px normal sans-serif!important;color: red;
>+
>+CSSProperty changed to "font: 12px sans-serif;".
>+font: 12px sans-serif;color: red;
>+
>+CSSProperty changed to "color: invalid_c010r;".
>+font: 12px sans-serif;color: invalid_c010r;
>+
>+
>diff --git a/LayoutTests/inspector/css/modify-inline-style.html b/LayoutTests/inspector/css/modify-inline-style.html
>new file mode 100644
>index 00000000000..305f9a58be0
>--- /dev/null
>+++ b/LayoutTests/inspector/css/modify-inline-style.html
>@@ -0,0 +1,85 @@
>+<!DOCTYPE html>
>+<html>
>+<head>
>+<script src="../../http/tests/inspector/resources/inspector-test.js"></script>
>+<script>
>+function test() {
>+    let nodeStyles = null;
>+    let suite = InspectorTest.createAsyncSuite("ModifyInlineStyle");
>+
>+    suite.addTestCase({
>+        name: "AddPropertyAndEdit",
>+        test(resolve, reject) {
>+            let getInlineStyleDeclaration = () => {
>+                for (let styleDeclaration of nodeStyles.orderedStyles) {
>+                    if (styleDeclaration.type === styleDeclaration.constructor.Type.Inline)
>+                        return styleDeclaration;
>+                }
>+                InspectorTest.fail("No declaration found.");
>+                resolve();
>+            };
>+
>+            let style = getInlineStyleDeclaration();
>+
>+            const fontPropertyIndex = 0;
>+            let fontProperty = style.newBlankProperty(fontPropertyIndex);
>+            fontProperty.name = "font";
>+            fontProperty.rawValue = "12px normal sans-serif!important";
>+            let colorProperty = null;
>+
>+            WI.CSSProperty.addEventListener(WI.CSSProperty.Event.Changed, function(event) {
>+                if (event.target.ownerStyle && event.target.ownerStyle.type === WI.CSSStyleDeclaration.Type.Computed)
>+                    return;
>+
>+                InspectorTest.log(`CSSProperty changed to "${event.target.text}".`);
>+                InspectorTest.log(style.text + "\n");
>+            });
>+
>+            fontProperty.awaitEvent(WI.CSSProperty.Event.Changed).then((event) => {
>+                const colorPropertyIndex = 1;
>+                colorProperty = style.newBlankProperty(colorPropertyIndex);
>+                let promise = colorProperty.awaitEvent(WI.CSSProperty.Event.Changed);
>+                colorProperty.name = "color";
>+                colorProperty.rawValue = "red";
>+                return promise;
>+            }).then(() => {
>+                let promise = fontProperty.awaitEvent(WI.CSSProperty.Event.Changed);
>+                fontProperty.rawValue = "12px sans-serif";
>+                return promise;
>+            }).then(() => {
>+                let promise = colorProperty.awaitEvent(WI.CSSProperty.Event.Changed);
>+                colorProperty.rawValue = "invalid_c010r";
>+                return promise;
>+            }).then(() => {
>+                resolve();
>+                return true;
>+            });
>+        }
>+    });
>+
>+    WI.domManager.requestDocument((documentNode) => {
>+        WI.domManager.querySelector(documentNode.id, "#x", (contentNodeId) => {
>+            if (contentNodeId) {
>+                let domNode = WI.domManager.nodeForId(contentNodeId);
>+                nodeStyles = WI.cssManager.stylesForNode(domNode);
>+
>+                if (nodeStyles.needsRefresh) {
>+                    nodeStyles.singleFireEventListener(WI.DOMNodeStyles.Event.Refreshed, (event) => {
>+                        suite.runTestCasesAndFinish()
>+                    });
>+                } else
>+                    suite.runTestCasesAndFinish();
>+            } else {
>+                InspectorTest.fail("DOM node not found.");
>+                InspectorTest.completeTest();
>+            }
>+        });
>+    });
>+}
>+</script>
>+</head>
>+<body onload="runTest()">
>+    <p>Testing that adding and editing CSS properties of inline styles works.</p>
>+    <div id="x"></div>
>+</body>
>+</html>
>diff --git a/Source/WebInspectorUI/ChangeLog b/Source/WebInspectorUI/ChangeLog
>index 35f37ac904e..ab4100f6d55 100644
>--- a/Source/WebInspectorUI/ChangeLog
>+++ b/Source/WebInspectorUI/ChangeLog
>@@ -1,3 +1,18 @@
>+2019-02-13  Nikita Vasilyev  <nvasilyev@apple.com>
>+
>+        Web Inspector: Styles: valid values in style attributes are reported as unsupported property values
>+        https://bugs.webkit.org/show_bug.cgi?id=194619
>+        <rdar://problem/47917373>
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        Payload of inline styles may contain `range` that doesn't match
>+        the actual text of the payload â it has an extra empty line at the end.
>+        Mismatching ranges caused data corruption.
>+
>+        * UserInterface/Models/DOMNodeStyles.js:
>+        (WI.DOMNodeStyles.prototype._parseStylePropertyPayload):
>+
> 2019-02-08  Devin Rousso  <drousso@apple.com>
> 
>         Web Inspector: Audit: show keyboard shortcut in export tooltip
>diff --git a/Source/WebInspectorUI/UserInterface/Models/DOMNodeStyles.js b/Source/WebInspectorUI/UserInterface/Models/DOMNodeStyles.js
>index c7845078fbc..c3c153c568f 100644
>--- a/Source/WebInspectorUI/UserInterface/Models/DOMNodeStyles.js
>+++ b/Source/WebInspectorUI/UserInterface/Models/DOMNodeStyles.js
>@@ -536,6 +536,19 @@ WI.DOMNodeStyles = class DOMNodeStyles extends WI.Object
>             break;
>         }
> 
>+        if (payload.range) {
>+            // Last property of inline style has mismatching range.
>+            // The actual text has one line, but the range spans two lines.
>+            let rangeLineCount = 1 + payload.range.endLine - payload.range.startLine;
>+            if (rangeLineCount > 1) {
>+                let textLineCount = payload.text.lineCount;
>+                if (textLineCount === rangeLineCount - 1) {
>+                    payload.range.endLine = payload.range.startLine + (textLineCount - 1);
>+                    payload.range.endColumn = payload.range.startColumn + payload.text.lastLine.length;
>+                }
>+            }
>+        }
>+
>         var styleSheetTextRange = this._parseSourceRangePayload(payload.range);
> 
>         if (styleDeclaration) {

Flags:

hi: review+
hi: commit-queue-

Actions: View | Formatted Diff | Diff

Attachments on bug 194619: 361965 | 361971 | 361972 | 361984 | 362101 | 362157