<?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>167866</bug_id>
          
          <creation_ts>2017-02-05 21:27:00 -0800</creation_ts>
          <short_desc>Re-land r210095 (avoid a tile revalidation on scale change)</short_desc>
          <delta_ts>2017-02-10 10:30:18 -0800</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>New Bugs</component>
          <version>WebKit Nightly Build</version>
          <rep_platform>Unspecified</rep_platform>
          <op_sys>Unspecified</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="Simon Fraser (smfr)">simon.fraser</reporter>
          <assigned_to name="Simon Fraser (smfr)">simon.fraser</assigned_to>
          <cc>simon.fraser</cc>
    
    <cc>thorton</cc>
    
    <cc>webkit-bug-importer</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1273457</commentid>
    <comment_count>0</comment_count>
    <who name="Simon Fraser (smfr)">simon.fraser</who>
    <bug_when>2017-02-05 21:27:00 -0800</bug_when>
    <thetext>Re-land r210095 (avoid a tile revalidation on scale change)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1273459</commentid>
    <comment_count>1</comment_count>
      <attachid>300683</attachid>
    <who name="Simon Fraser (smfr)">simon.fraser</who>
    <bug_when>2017-02-05 21:28:23 -0800</bug_when>
    <thetext>Created attachment 300683
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1273463</commentid>
    <comment_count>2</comment_count>
      <attachid>300683</attachid>
    <who name="Tim Horton">thorton</who>
    <bug_when>2017-02-05 22:01:16 -0800</bug_when>
    <thetext>Comment on attachment 300683
Patch

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

Yay!

&gt; Source/WebCore/platform/graphics/ca/TileGrid.cpp:57
&gt; +        if (validationPolicy &amp; TileGrid::PruneSecondaryTiles)
&gt; +            builder.appendLiteral(&quot;, &quot;);

For future&apos;s sake, check the builder&apos;s length instead?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1273811</commentid>
    <comment_count>3</comment_count>
    <who name="Simon Fraser (smfr)">simon.fraser</who>
    <bug_when>2017-02-06 18:22:00 -0800</bug_when>
    <thetext>https://trac.webkit.org/r211766</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1275376</commentid>
    <comment_count>4</comment_count>
    <who name="Radar WebKit Bug Importer">webkit-bug-importer</who>
    <bug_when>2017-02-10 10:30:18 -0800</bug_when>
    <thetext>&lt;rdar://problem/30467124&gt;</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>300683</attachid>
            <date>2017-02-05 21:28:23 -0800</date>
            <delta_ts>2017-02-05 22:01:16 -0800</delta_ts>
            <desc>Patch</desc>
            <filename>bug-167866-20170205212603.patch</filename>
            <type>text/plain</type>
            <size>3917</size>
            <attacher name="Simon Fraser (smfr)">simon.fraser</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMjExNjg4CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9D
aGFuZ2VMb2cgYi9Tb3VyY2UvV2ViQ29yZS9DaGFuZ2VMb2cKaW5kZXggNWNmYzFhMDBjNTliODRi
ZmQ3OWM2ZGU1ZTY3N2IwNWZiODI2OGQwNS4uZDA2MmZhY2MwYTJlN2Q5Nzg3NzUwYTJjNTZiMzk0
OTU2MzdlMTVmZiAxMDA2NDQKLS0tIGEvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCisrKyBiL1Nv
dXJjZS9XZWJDb3JlL0NoYW5nZUxvZwpAQCAtMSw1ICsxLDI0IEBACiAyMDE3LTAyLTA1ICBTaW1v
biBGcmFzZXIgIDxzaW1vbi5mcmFzZXJAYXBwbGUuY29tPgogCisgICAgICAgIFJlLWxhbmQgcjIx
MDA5NSAoYXZvaWQgYSB0aWxlIHJldmFsaWRhdGlvbiBvbiBzY2FsZSBjaGFuZ2UpCisgICAgICAg
IGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0xNjc4NjYKKworICAgICAg
ICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKyAgICAgICAgCisgICAgICAgIHIyMTAwOTUg
d2FzIHJvbGxlZCBvdXQgaW4gcjIxMTIzMCBidXQgbm93IHRoYXQgYWxsIFRpbGVDb250cm9sbGVy
cyB1bnBhcmVudAorICAgICAgICBvZmZzY3JlZW4gdGlsZXMsIHdlIGNhbiByb2xsIGl0IGJhY2sg
aXQuCisgICAgICAgIAorICAgICAgICBBbHNvIGFkZCBtb3JlIFRpbGluZyBsb2dnaW5nLgorCisg
ICAgICAgICogcGxhdGZvcm0vZ3JhcGhpY3MvY2EvVGlsZUdyaWQuY3BwOgorICAgICAgICAoV2Vi
Q29yZTo6dmFsaWRhdGlvblBvbGljeUFzU3RyaW5nKToKKyAgICAgICAgKFdlYkNvcmU6OlRpbGVH
cmlkOjpzZXRTY2FsZSk6CisgICAgICAgIChXZWJDb3JlOjpUaWxlR3JpZDo6cHJlcG9wdWxhdGVS
ZWN0KToKKyAgICAgICAgKFdlYkNvcmU6OlRpbGVHcmlkOjpyZXZhbGlkYXRlVGlsZXMpOgorICAg
ICAgICAoV2ViQ29yZTo6VGlsZUdyaWQ6OmVuc3VyZVRpbGVzRm9yUmVjdCk6CisKKzIwMTctMDIt
MDUgIFNpbW9uIEZyYXNlciAgPHNpbW9uLmZyYXNlckBhcHBsZS5jb20+CisKICAgICAgICAgUmVt
b3ZlIHVucGFyZW50c09mZnNjcmVlblRpbGVzIGxvZ2ljIGluIFRpbGVDb250cm9sbGVyCiAgICAg
ICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0xNjc4MjMKIApkaWZm
IC0tZ2l0IGEvU291cmNlL1dlYkNvcmUvcGxhdGZvcm0vZ3JhcGhpY3MvY2EvVGlsZUdyaWQuY3Bw
IGIvU291cmNlL1dlYkNvcmUvcGxhdGZvcm0vZ3JhcGhpY3MvY2EvVGlsZUdyaWQuY3BwCmluZGV4
IGI4OTQ3MGVhY2NhZDU5M2FlZTIzZWQ1NDRiZTVhYmJhZmExOGMyOTAuLjBmMTcxMWMxYTdhZjI2
YTRlNDM5NWM1Y2VlNWFjYzg0OTg5NzY2MDAgMTAwNjQ0Ci0tLSBhL1NvdXJjZS9XZWJDb3JlL3Bs
YXRmb3JtL2dyYXBoaWNzL2NhL1RpbGVHcmlkLmNwcAorKysgYi9Tb3VyY2UvV2ViQ29yZS9wbGF0
Zm9ybS9ncmFwaGljcy9jYS9UaWxlR3JpZC5jcHAKQEAgLTQ0LDYgKzQ0LDI1IEBACiAKIG5hbWVz
cGFjZSBXZWJDb3JlIHsKIAorI2lmICFMT0dfRElTQUJMRUQKK3N0YXRpYyBTdHJpbmcgdmFsaWRh
dGlvblBvbGljeUFzU3RyaW5nKFRpbGVHcmlkOjpUaWxlVmFsaWRhdGlvblBvbGljeSB2YWxpZGF0
aW9uUG9saWN5KQoreworICAgIFN0cmluZ0J1aWxkZXIgYnVpbGRlcjsKKyAgICBidWlsZGVyLmFw
cGVuZExpdGVyYWwoIlsiKTsKKyAgICBpZiAodmFsaWRhdGlvblBvbGljeSAmIFRpbGVHcmlkOjpQ
cnVuZVNlY29uZGFyeVRpbGVzKQorICAgICAgICBidWlsZGVyLmFwcGVuZExpdGVyYWwoInBydW5l
IHNlY29uZGFyeSIpOworCisgICAgaWYgKHZhbGlkYXRpb25Qb2xpY3kgJiBUaWxlR3JpZDo6VW5w
YXJlbnRBbGxUaWxlcykgeworICAgICAgICBpZiAodmFsaWRhdGlvblBvbGljeSAmIFRpbGVHcmlk
OjpQcnVuZVNlY29uZGFyeVRpbGVzKQorICAgICAgICAgICAgYnVpbGRlci5hcHBlbmRMaXRlcmFs
KCIsICIpOworICAgICAgICBidWlsZGVyLmFwcGVuZExpdGVyYWwoInVucGFyZW50IGFsbCIpOwor
ICAgIH0KKyAgICBidWlsZGVyLmFwcGVuZExpdGVyYWwoIl0iKTsKKworICAgIHJldHVybiBidWls
ZGVyLnRvU3RyaW5nKCk7Cit9CisjZW5kaWYKKwogVGlsZUdyaWQ6OlRpbGVHcmlkKFRpbGVDb250
cm9sbGVyJiBjb250cm9sbGVyKQogICAgIDogbV9jb250cm9sbGVyKGNvbnRyb2xsZXIpCiAgICAg
LCBtX2NvbnRhaW5lckxheWVyKCpjb250cm9sbGVyLnJvb3RMYXllcigpLmNyZWF0ZUNvbXBhdGli
bGVMYXllcihQbGF0Zm9ybUNBTGF5ZXI6OkxheWVyVHlwZUxheWVyLCBudWxscHRyKSkKQEAgLTc4
LDggKzk3LDcgQEAgdm9pZCBUaWxlR3JpZDo6c2V0U2NhbGUoZmxvYXQgc2NhbGUpCiAgICAgdHJh
bnNmb3JtLnNjYWxlKDEgLyBtX3NjYWxlKTsKICAgICBtX2NvbnRhaW5lckxheWVyLT5zZXRUcmFu
c2Zvcm0odHJhbnNmb3JtKTsKIAotICAgIC8vIEZJWE1FOiB3ZSBtYXkgcmV2YWxpZGF0ZVRpbGVz
IHR3aWNlIGluIHRoaXMgY29tbWl0LgotICAgIHJldmFsaWRhdGVUaWxlcyhQcnVuZVNlY29uZGFy
eVRpbGVzKTsKKyAgICBtX2NvbnRyb2xsZXIuc2V0TmVlZHNSZXZhbGlkYXRlVGlsZXMoKTsKIAog
ICAgIG1fY29udGFpbmVyTGF5ZXIuZ2V0KCkuc2V0Q29udGVudHNTY2FsZShtX2NvbnRyb2xsZXIu
ZGV2aWNlU2NhbGVGYWN0b3IoKSk7CiAKQEAgLTIxNSw2ICsyMzMsOCBAQCBib29sIFRpbGVHcmlk
Ojp0aWxlc1dvdWxkQ2hhbmdlRm9yQ292ZXJhZ2VSZWN0KGNvbnN0IEZsb2F0UmVjdCYgY292ZXJh
Z2VSZWN0KSBjbwogCiBib29sIFRpbGVHcmlkOjpwcmVwb3B1bGF0ZVJlY3QoY29uc3QgRmxvYXRS
ZWN0JiByZWN0KQogeworICAgIExPR19XSVRIX1NUUkVBTShUaWxpbmcsIHN0cmVhbSA8PCAiVGls
ZUdyaWQgIiA8PCB0aGlzIDw8ICIgcHJlcG9wdWxhdGVSZWN0OiAiIDw8IHJlY3QpOworCiAgICAg
SW50UmVjdCBlbmNsb3NpbmdDb3ZlcmFnZVJlY3QgPSBlbmNsb3NpbmdJbnRSZWN0KHJlY3QpOwog
ICAgIGlmIChtX3ByaW1hcnlUaWxlQ292ZXJhZ2VSZWN0LmNvbnRhaW5zKGVuY2xvc2luZ0NvdmVy
YWdlUmVjdCkpCiAgICAgICAgIHJldHVybiBmYWxzZTsKQEAgLTMzMiw2ICszNTIsOCBAQCB2b2lk
IFRpbGVHcmlkOjpyZXZhbGlkYXRlVGlsZXMoVGlsZVZhbGlkYXRpb25Qb2xpY3kgdmFsaWRhdGlv
blBvbGljeSkKICAgICBGbG9hdFJlY3QgY292ZXJhZ2VSZWN0ID0gbV9jb250cm9sbGVyLmNvdmVy
YWdlUmVjdCgpOwogICAgIEludFJlY3QgYm91bmRzID0gbV9jb250cm9sbGVyLmJvdW5kcygpOwog
CisgICAgTE9HX1dJVEhfU1RSRUFNKFRpbGluZywgc3RyZWFtIDw8ICJUaWxlR3JpZCAiIDw8IHRo
aXMgPDwgIiByZXZhbGlkYXRlVGlsZXM6IGJvdW5kcyAiIDw8IGJvdW5kcyA8PCAiIGNvdmVyYWdl
UmVjdCIgPDwgY292ZXJhZ2VSZWN0IDw8ICIgdmFsaWRhdGlvbjogIiA8PCB2YWxpZGF0aW9uUG9s
aWN5QXNTdHJpbmcodmFsaWRhdGlvblBvbGljeSkpOworCiAgICAgaWYgKGNvdmVyYWdlUmVjdC5p
c0VtcHR5KCkgfHwgYm91bmRzLmlzRW1wdHkoKSkKICAgICAgICAgcmV0dXJuOwogCkBAIC01MzQs
NiArNTU2LDggQEAgSW50UmVjdCBUaWxlR3JpZDo6ZW5zdXJlVGlsZXNGb3JSZWN0KGNvbnN0IEZs
b2F0UmVjdCYgcmVjdCwgQ292ZXJhZ2VUeXBlIG5ld1RpbGUKICAgICBpZiAoIW1fY29udHJvbGxl
ci5pc0luV2luZG93KCkpCiAgICAgICAgIHJldHVybiBJbnRSZWN0KCk7CiAKKyAgICBMT0dfV0lU
SF9TVFJFQU0oVGlsaW5nLCBzdHJlYW0gPDwgIlRpbGVHcmlkICIgPDwgdGhpcyA8PCAiIGVuc3Vy
ZVRpbGVzRm9yUmVjdDogIiA8PCByZWN0KTsKKwogICAgIEZsb2F0UmVjdCBzY2FsZWRSZWN0KHJl
Y3QpOwogICAgIHNjYWxlZFJlY3Quc2NhbGUobV9zY2FsZSk7CiAgICAgSW50UmVjdCByZWN0SW5U
aWxlQ29vcmRzKGVuY2xvc2luZ0ludFJlY3Qoc2NhbGVkUmVjdCkpOwo=
</data>
<flag name="review"
          id="322538"
          type_id="1"
          status="+"
          setter="thorton"
    />
          </attachment>
      

    </bug>

</bugzilla>