Attachment 371783 Details for Bug 198505 – Patch

[patch] Patch

patch.txt (text/plain), 2.88 KB, created by Nikita Vasilyev on 2019-06-10 15:19:18 PDT

(hide)

Description:

Filename:

MIME Type:

Creator: Nikita Vasilyev

Created: 2019-06-10 15:19:18 PDT

Size: 2.88 KB

patch

obsolete

>diff --git a/Source/WebInspectorUI/ChangeLog b/Source/WebInspectorUI/ChangeLog
>index 788d30a070f..25dedfc0b59 100644
>--- a/Source/WebInspectorUI/ChangeLog
>+++ b/Source/WebInspectorUI/ChangeLog
>@@ -1,3 +1,22 @@
>+2019-06-08  Nikita Vasilyev  <nvasilyev@apple.com>
>+
>+        REGRESSION(r240946): Web Inspector: Styles: Pasting multiple properties has issues
>+        https://bugs.webkit.org/show_bug.cgi?id=198505
>+        <rdar://problem/51374780>
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        Since r240946, setting WI.CSSStyleDeclaration.prototype.text updates the text immediately.
>+        When WI.CSSStyleDeclaration.prototype.update gets called after setting text, it exits early
>+        without firing WI.CSSStyleDeclaration.Event.PropertiesChanged.
>+
>+        This patch adds _pendingPropertiesChanged flag to fire WI.CSSStyleDeclaration.Event.PropertiesChanged
>+        after changing text.
>+
>+        * UserInterface/Models/CSSStyleDeclaration.js:
>+        (WI.CSSStyleDeclaration):
>+        (WI.CSSStyleDeclaration.prototype.set text):
>+
> 2019-05-24  Devin Rousso  <drousso@apple.com>
> 
>         Web Inspector: Overlay: don't show setting for showing rulers/guides during element selection if it's not supported
>diff --git a/Source/WebInspectorUI/UserInterface/Models/CSSStyleDeclaration.js b/Source/WebInspectorUI/UserInterface/Models/CSSStyleDeclaration.js
>index 74b694f2212..ffbd924ce3d 100644
>--- a/Source/WebInspectorUI/UserInterface/Models/CSSStyleDeclaration.js
>+++ b/Source/WebInspectorUI/UserInterface/Models/CSSStyleDeclaration.js
>@@ -42,6 +42,7 @@ WI.CSSStyleDeclaration = class CSSStyleDeclaration extends WI.Object
> 
>         this._initialState = null;
>         this._updatesInProgressCount = 0;
>+        this._pendingPropertiesChanged = false;
>         this._locked = false;
>         this._pendingProperties = [];
>         this._propertyNameMap = {};
>@@ -183,9 +184,11 @@ WI.CSSStyleDeclaration = class CSSStyleDeclaration extends WI.Object
> 
>         // Don't fire the event if text hasn't changed. However, it should still fire for Computed style declarations
>         // because it never has text.
>-        if (oldText === this._text && this._type !== WI.CSSStyleDeclaration.Type.Computed)
>+        if (oldText === this._text && !this._pendingPropertiesChanged && this._type !== WI.CSSStyleDeclaration.Type.Computed)
>             return;
> 
>+        this._pendingPropertiesChanged = false;
>+
>         function delayed()
>         {
>             this.dispatchEventToListeners(WI.CSSStyleDeclaration.Event.PropertiesChanged);
>@@ -237,6 +240,7 @@ WI.CSSStyleDeclaration = class CSSStyleDeclaration extends WI.Object
>             clearTimeout(timeoutId);
>             timeoutId = null;
>             this._updatesInProgressCount = Math.max(0, this._updatesInProgressCount - 1);
>+            this._pendingPropertiesChanged = true;
>         };
> 
>         this._nodeStyles.changeStyleText(this, text, styleTextDidChange);

Flags:

ews-watchlist: commit-queue-

Actions: View | Formatted Diff | Diff

Attachments on bug 198505: 371683 | 371688 | 371690 | 371694 | 371783 | 371814 | 372003 | 372011 | 372203 | 372208 | 372308 | 372318 | 372355 | 372362 | 372394