<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugs.webkit.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.4.1"
          urlbase="https://bugs.webkit.org/"
          
          maintainer="admin@webkit.org"
>

    <bug>
          <bug_id>213234</bug_id>
          
          <creation_ts>2020-06-15 22:00:17 -0700</creation_ts>
          <short_desc>Web Inspector: Sources: objects in Scope Chain panel flicker when resizing details sidebar</short_desc>
          <delta_ts>2020-06-16 14:11:30 -0700</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>Web Inspector</component>
          <version>WebKit Nightly Build</version>
          <rep_platform>All</rep_platform>
          <op_sys>All</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords>InRadar</keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Devin Rousso">hi</reporter>
          <assigned_to name="Devin Rousso">hi</assigned_to>
          <cc>ddkilzer</cc>
    
    <cc>hi</cc>
    
    <cc>inspector-bugzilla-changes</cc>
    
    <cc>mjs</cc>
    
    <cc>webkit-bug-importer</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1662981</commentid>
    <comment_count>0</comment_count>
    <who name="Devin Rousso">hi</who>
    <bug_when>2020-06-15 22:00:17 -0700</bug_when>
    <thetext># STEPS TO REPRODUCE
1. inspect any page
2. set a JavaScript breakpoint
3. trigger the breakpoint from step #2
4. select the Scope Chain panel in the details sidebar
5. resize the details sidebar
 =&gt; objects in the Scope Chain panel flicker</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1662983</commentid>
    <comment_count>1</comment_count>
    <who name="Devin Rousso">hi</who>
    <bug_when>2020-06-15 22:00:34 -0700</bug_when>
    <thetext>&lt;rdar://problem/64391213&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1662984</commentid>
    <comment_count>2</comment_count>
      <attachid>401981</attachid>
    <who name="Devin Rousso">hi</who>
    <bug_when>2020-06-15 22:02:31 -0700</bug_when>
    <thetext>Created attachment 401981
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1663191</commentid>
    <comment_count>3</comment_count>
      <attachid>401981</attachid>
    <who name="David Kilzer (:ddkilzer)">ddkilzer</who>
    <bug_when>2020-06-16 12:08:34 -0700</bug_when>
    <thetext>Comment on attachment 401981
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=401981&amp;action=review

&gt; Source/WebInspectorUI/ChangeLog:12
&gt; +        Don&apos;t regenerate the DOM if the `layout` unless the reason is `WI.View.LayoutReason.Dirty`,
&gt; +        such as if the current call frame changes or a watch expression is added/updated/removed.

&quot;Don&apos;t regenerate the DOM if the `layout`...&quot; does what?  (Sentence doesn&apos;t read properly.)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1663215</commentid>
    <comment_count>4</comment_count>
      <attachid>401981</attachid>
    <who name="Devin Rousso">hi</who>
    <bug_when>2020-06-16 13:38:35 -0700</bug_when>
    <thetext>Comment on attachment 401981
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=401981&amp;action=review

&gt;&gt; Source/WebInspectorUI/ChangeLog:12
&gt;&gt; +        such as if the current call frame changes or a watch expression is added/updated/removed.
&gt; 
&gt; &quot;Don&apos;t regenerate the DOM if the `layout`...&quot; does what?  (Sentence doesn&apos;t read properly.)

Oh oops I meant &quot;Don&apos;t regenerate the DOM in `layout` unless ...&quot; 😅</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1663218</commentid>
    <comment_count>5</comment_count>
      <attachid>402035</attachid>
    <who name="Devin Rousso">hi</who>
    <bug_when>2020-06-16 13:46:36 -0700</bug_when>
    <thetext>Created attachment 402035
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1663234</commentid>
    <comment_count>6</comment_count>
    <who name="EWS">ews-feeder</who>
    <bug_when>2020-06-16 14:11:29 -0700</bug_when>
    <thetext>Committed r263115: &lt;https://trac.webkit.org/changeset/263115&gt;

All reviewed patches have been landed. Closing bug and clearing flags on attachment 402035.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>401981</attachid>
            <date>2020-06-15 22:02:31 -0700</date>
            <delta_ts>2020-06-16 13:46:36 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-213234-20200615220231.patch</filename>
            <type>text/plain</type>
            <size>1818</size>
            <attacher name="Devin Rousso">hi</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJJbnNwZWN0b3JVSS9DaGFuZ2VMb2cgYi9Tb3VyY2UvV2Vi
SW5zcGVjdG9yVUkvQ2hhbmdlTG9nCmluZGV4IDdlNzJlNzM5ODE2ZTQ3YTQ3OGVmYTdmYzEzMGZk
NzMzZjc2YmZmM2MuLjM3NzkxOTMyMDcyZjJiNTE5NDdjYjk3ZjVmNDdkM2E5MWQ0YjU3NGIgMTAw
NjQ0Ci0tLSBhL1NvdXJjZS9XZWJJbnNwZWN0b3JVSS9DaGFuZ2VMb2cKKysrIGIvU291cmNlL1dl
Ykluc3BlY3RvclVJL0NoYW5nZUxvZwpAQCAtMSwzICsxLDE2IEBACisyMDIwLTA2LTE1ICBEZXZp
biBSb3Vzc28gIDxkcm91c3NvQGFwcGxlLmNvbT4KKworICAgICAgICBXZWIgSW5zcGVjdG9yOiBT
b3VyY2VzOiBvYmplY3RzIGluIFNjb3BlIENoYWluIHBhbmVsIGZsaWNrZXIgd2hlbiByZXNpemlu
ZyBkZXRhaWxzIHNpZGViYXIKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19i
dWcuY2dpP2lkPTIxMzIzNAorICAgICAgICA8cmRhcjovL3Byb2JsZW0vNjQzOTEyMTM+CisKKyAg
ICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgKiBVc2VySW50ZXJm
YWNlL1ZpZXdzL1Njb3BlQ2hhaW5EZXRhaWxzU2lkZWJhclBhbmVsLmpzOgorICAgICAgICAoV0ku
U2NvcGVDaGFpbkRldGFpbHNTaWRlYmFyUGFuZWwucHJvdG90eXBlLmxheW91dCk6CisgICAgICAg
IERvbid0IHJlZ2VuZXJhdGUgdGhlIERPTSBpZiB0aGUgYGxheW91dGAgdW5sZXNzIHRoZSByZWFz
b24gaXMgYFdJLlZpZXcuTGF5b3V0UmVhc29uLkRpcnR5YCwKKyAgICAgICAgc3VjaCBhcyBpZiB0
aGUgY3VycmVudCBjYWxsIGZyYW1lIGNoYW5nZXMgb3IgYSB3YXRjaCBleHByZXNzaW9uIGlzIGFk
ZGVkL3VwZGF0ZWQvcmVtb3ZlZC4KKwogMjAyMC0wNi0xNSAgUGF2ZWwgRmVsZG1hbiA8cGF2ZWwu
ZmVsZG1hbkBnbWFpbC5jb20+CiAKICAgICAgICAgV2ViIEluc3BlY3RvcjogaW50cm9kdWNlIHJl
cXVlc3QgaW50ZXJjZXB0aW9uCmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViSW5zcGVjdG9yVUkvVXNl
ckludGVyZmFjZS9WaWV3cy9TY29wZUNoYWluRGV0YWlsc1NpZGViYXJQYW5lbC5qcyBiL1NvdXJj
ZS9XZWJJbnNwZWN0b3JVSS9Vc2VySW50ZXJmYWNlL1ZpZXdzL1Njb3BlQ2hhaW5EZXRhaWxzU2lk
ZWJhclBhbmVsLmpzCmluZGV4IDRjOTQ4OWEwMTAzNDc3MmQxN2JmZDNkZTg5NTk4ZjQ4ZjFlMTFm
ZDkuLjI5OGYxNDY5MzA1Njc2OWIyZTlkMGMzNmFmODQ5YTM4NjAyNjZkN2EgMTAwNjQ0Ci0tLSBh
L1NvdXJjZS9XZWJJbnNwZWN0b3JVSS9Vc2VySW50ZXJmYWNlL1ZpZXdzL1Njb3BlQ2hhaW5EZXRh
aWxzU2lkZWJhclBhbmVsLmpzCisrKyBiL1NvdXJjZS9XZWJJbnNwZWN0b3JVSS9Vc2VySW50ZXJm
YWNlL1ZpZXdzL1Njb3BlQ2hhaW5EZXRhaWxzU2lkZWJhclBhbmVsLmpzCkBAIC0xMjQsNiArMTI0
LDExIEBAIFdJLlNjb3BlQ2hhaW5EZXRhaWxzU2lkZWJhclBhbmVsID0gY2xhc3MgU2NvcGVDaGFp
bkRldGFpbHNTaWRlYmFyUGFuZWwgZXh0ZW5kcyBXCiAKICAgICBsYXlvdXQoKQogICAgIHsKKyAg
ICAgICAgc3VwZXIubGF5b3V0KCk7CisKKyAgICAgICAgaWYgKHRoaXMubGF5b3V0UmVhc29uICE9
PSBXSS5WaWV3LkxheW91dFJlYXNvbi5EaXJ0eSkKKyAgICAgICAgICAgIHJldHVybjsKKwogICAg
ICAgICBsZXQgY2FsbEZyYW1lID0gdGhpcy5fY2FsbEZyYW1lOwogCiAgICAgICAgIFByb21pc2Uu
YWxsKFt0aGlzLl9nZW5lcmF0ZVdhdGNoRXhwcmVzc2lvbnNTZWN0aW9uKCksIHRoaXMuX2dlbmVy
YXRlQ2FsbEZyYW1lc1NlY3Rpb24oKV0pLnRoZW4oZnVuY3Rpb24oc2VjdGlvbnMpIHsK
</data>
<flag name="review"
          id="417415"
          type_id="1"
          status="+"
          setter="ddkilzer"
    />
          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>402035</attachid>
            <date>2020-06-16 13:46:36 -0700</date>
            <delta_ts>2020-06-16 14:11:30 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-213234.patch</filename>
            <type>text/plain</type>
            <size>1812</size>
            <attacher name="Devin Rousso">hi</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJJbnNwZWN0b3JVSS9DaGFuZ2VMb2cgYi9Tb3VyY2UvV2Vi
SW5zcGVjdG9yVUkvQ2hhbmdlTG9nCmluZGV4IDdlNzJlNzM5ODE2ZTQ3YTQ3OGVmYTdmYzEzMGZk
NzMzZjc2YmZmM2MuLjM3NzkxOTMyMDcyZjJiNTE5NDdjYjk3ZjVmNDdkM2E5MWQ0YjU3NGIgMTAw
NjQ0Ci0tLSBhL1NvdXJjZS9XZWJJbnNwZWN0b3JVSS9DaGFuZ2VMb2cKKysrIGIvU291cmNlL1dl
Ykluc3BlY3RvclVJL0NoYW5nZUxvZwpAQCAtMSwzICsxLDE2IEBACisyMDIwLTA2LTE1ICBEZXZp
biBSb3Vzc28gIDxkcm91c3NvQGFwcGxlLmNvbT4KKworICAgICAgICBXZWIgSW5zcGVjdG9yOiBT
b3VyY2VzOiBvYmplY3RzIGluIFNjb3BlIENoYWluIHBhbmVsIGZsaWNrZXIgd2hlbiByZXNpemlu
ZyBkZXRhaWxzIHNpZGViYXIKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19i
dWcuY2dpP2lkPTIxMzIzNAorICAgICAgICA8cmRhcjovL3Byb2JsZW0vNjQzOTEyMTM+CisKKyAg
ICAgICAgUmV2aWV3ZWQgYnkgRGF2aWQgS2lsemVyLgorCisgICAgICAgICogVXNlckludGVyZmFj
ZS9WaWV3cy9TY29wZUNoYWluRGV0YWlsc1NpZGViYXJQYW5lbC5qczoKKyAgICAgICAgKFdJLlNj
b3BlQ2hhaW5EZXRhaWxzU2lkZWJhclBhbmVsLnByb3RvdHlwZS5sYXlvdXQpOgorICAgICAgICBE
b24ndCByZWdlbmVyYXRlIHRoZSBET00gaW4gYGxheW91dGAgdW5sZXNzIHRoZSByZWFzb24gaXMg
YFdJLlZpZXcuTGF5b3V0UmVhc29uLkRpcnR5YCwgc3VjaAorICAgICAgICBhcyBpZiB0aGUgY3Vy
cmVudCBjYWxsIGZyYW1lIGNoYW5nZXMgb3IgYSB3YXRjaCBleHByZXNzaW9uIGlzIGFkZGVkL3Vw
ZGF0ZWQvcmVtb3ZlZC4KKwogMjAyMC0wNi0xNSAgUGF2ZWwgRmVsZG1hbiA8cGF2ZWwuZmVsZG1h
bkBnbWFpbC5jb20+CiAKICAgICAgICAgV2ViIEluc3BlY3RvcjogaW50cm9kdWNlIHJlcXVlc3Qg
aW50ZXJjZXB0aW9uCmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViSW5zcGVjdG9yVUkvVXNlckludGVy
ZmFjZS9WaWV3cy9TY29wZUNoYWluRGV0YWlsc1NpZGViYXJQYW5lbC5qcyBiL1NvdXJjZS9XZWJJ
bnNwZWN0b3JVSS9Vc2VySW50ZXJmYWNlL1ZpZXdzL1Njb3BlQ2hhaW5EZXRhaWxzU2lkZWJhclBh
bmVsLmpzCmluZGV4IDRjOTQ4OWEwMTAzNDc3MmQxN2JmZDNkZTg5NTk4ZjQ4ZjFlMTFmZDkuLjI5
OGYxNDY5MzA1Njc2OWIyZTlkMGMzNmFmODQ5YTM4NjAyNjZkN2EgMTAwNjQ0Ci0tLSBhL1NvdXJj
ZS9XZWJJbnNwZWN0b3JVSS9Vc2VySW50ZXJmYWNlL1ZpZXdzL1Njb3BlQ2hhaW5EZXRhaWxzU2lk
ZWJhclBhbmVsLmpzCisrKyBiL1NvdXJjZS9XZWJJbnNwZWN0b3JVSS9Vc2VySW50ZXJmYWNlL1Zp
ZXdzL1Njb3BlQ2hhaW5EZXRhaWxzU2lkZWJhclBhbmVsLmpzCkBAIC0xMjQsNiArMTI0LDExIEBA
IFdJLlNjb3BlQ2hhaW5EZXRhaWxzU2lkZWJhclBhbmVsID0gY2xhc3MgU2NvcGVDaGFpbkRldGFp
bHNTaWRlYmFyUGFuZWwgZXh0ZW5kcyBXCiAKICAgICBsYXlvdXQoKQogICAgIHsKKyAgICAgICAg
c3VwZXIubGF5b3V0KCk7CisKKyAgICAgICAgaWYgKHRoaXMubGF5b3V0UmVhc29uICE9PSBXSS5W
aWV3LkxheW91dFJlYXNvbi5EaXJ0eSkKKyAgICAgICAgICAgIHJldHVybjsKKwogICAgICAgICBs
ZXQgY2FsbEZyYW1lID0gdGhpcy5fY2FsbEZyYW1lOwogCiAgICAgICAgIFByb21pc2UuYWxsKFt0
aGlzLl9nZW5lcmF0ZVdhdGNoRXhwcmVzc2lvbnNTZWN0aW9uKCksIHRoaXMuX2dlbmVyYXRlQ2Fs
bEZyYW1lc1NlY3Rpb24oKV0pLnRoZW4oZnVuY3Rpb24oc2VjdGlvbnMpIHsK
</data>

          </attachment>
      

    </bug>

</bugzilla>