<?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>97108</bug_id>
          
          <creation_ts>2012-09-19 06:51:22 -0700</creation_ts>
          <short_desc>[CoordinatedGraphics] Don&apos;t reset m_shouldSyncFrame in flushPendingLayerChanges</short_desc>
          <delta_ts>2012-09-20 04:13:50 -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>Layout and Rendering</component>
          <version>528+ (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></keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Balazs Kelemen">kbalazs</reporter>
          <assigned_to name="Balazs Kelemen">kbalazs</assigned_to>
          <cc>jturcotte</cc>
    
    <cc>noam</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>723526</commentid>
    <comment_count>0</comment_count>
    <who name="Balazs Kelemen">kbalazs</who>
    <bug_when>2012-09-19 06:51:22 -0700</bug_when>
    <thetext>We should consider the changes resulted in syncLayerState, syncLayerChildren, etc., that&apos;s why those methods set it.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>723531</commentid>
    <comment_count>1</comment_count>
      <attachid>164733</attachid>
    <who name="Balazs Kelemen">kbalazs</who>
    <bug_when>2012-09-19 06:54:50 -0700</bug_when>
    <thetext>Created attachment 164733
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>723533</commentid>
    <comment_count>2</comment_count>
      <attachid>164733</attachid>
    <who name="Noam Rosenthal">noam</who>
    <bug_when>2012-09-19 07:00:05 -0700</bug_when>
    <thetext>Comment on attachment 164733
Patch

Please say in the changelog why this is important (e.g. changes can happen during layout, like deleting a layer)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>724309</commentid>
    <comment_count>3</comment_count>
      <attachid>164866</attachid>
    <who name="Balazs Kelemen">kbalazs</who>
    <bug_when>2012-09-20 02:27:37 -0700</bug_when>
    <thetext>Created attachment 164866
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>724397</commentid>
    <comment_count>4</comment_count>
      <attachid>164866</attachid>
    <who name="Balazs Kelemen">kbalazs</who>
    <bug_when>2012-09-20 04:13:46 -0700</bug_when>
    <thetext>Comment on attachment 164866
Patch

Clearing flags on attachment: 164866

Committed r129116: &lt;http://trac.webkit.org/changeset/129116&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>724398</commentid>
    <comment_count>5</comment_count>
    <who name="Balazs Kelemen">kbalazs</who>
    <bug_when>2012-09-20 04:13:50 -0700</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>164733</attachid>
            <date>2012-09-19 06:54:50 -0700</date>
            <delta_ts>2012-09-20 02:27:33 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-97108-20120919155358.patch</filename>
            <type>text/plain</type>
            <size>1827</size>
            <attacher name="Balazs Kelemen">kbalazs</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMTI4OTc5CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViS2l0Mi9D
aGFuZ2VMb2cgYi9Tb3VyY2UvV2ViS2l0Mi9DaGFuZ2VMb2cKaW5kZXggN2RmZDcwNmUzMzBjOWNl
Zjc3MDdmMzRiYTU2M2E2MmNlOWNjNzdhMC4uMzQxNjc2Mjc5ZjI2YzcwNzg0MmI4Y2ZiNDcwNTYx
MGJmY2U5NjZkNCAxMDA2NDQKLS0tIGEvU291cmNlL1dlYktpdDIvQ2hhbmdlTG9nCisrKyBiL1Nv
dXJjZS9XZWJLaXQyL0NoYW5nZUxvZwpAQCAtMSw1ICsxLDE5IEBACiAyMDEyLTA5LTE5ICBCYWxh
enMgS2VsZW1lbiAgPGtiYWxhenNAd2Via2l0Lm9yZz4KIAorICAgICAgICBbQ29vcmRpbmF0ZWRH
cmFwaGljc10gRG9uJ3QgcmVzZXQgbV9zaG91bGRTeW5jRnJhbWUgaW4gZmx1c2hQZW5kaW5nTGF5
ZXJDaGFuZ2VzCisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9p
ZD05NzEwOAorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAg
IFN0b3AgaWdub3JpbmcgdGhvc2UgY2hhbmdlcyB0aGF0IGhhcHBlbmVkIGFmdGVyIHRoZSBsYXN0
IGZsdXNoCisgICAgICAgIGFuZCByZXN1bHRlZCBpbiBzZXR0aW5nIG1fc2hvdWxkU3luY0ZyYW1l
LiBXZSBzaG91bGQgc2VuZCBhCisgICAgICAgIERpZFJlbmRlckZyYW1lIG1lc3NhZ2UgaWYgc3Vj
aCBhIGNoYW5nZSBvY2N1cmVkLgorCisgICAgICAgICogV2ViUHJvY2Vzcy9XZWJQYWdlL0Nvb3Jk
aW5hdGVkR3JhcGhpY3MvTGF5ZXJUcmVlQ29vcmRpbmF0b3IuY3BwOgorICAgICAgICAoV2ViS2l0
OjpMYXllclRyZWVDb29yZGluYXRvcjo6Zmx1c2hQZW5kaW5nTGF5ZXJDaGFuZ2VzKToKKworMjAx
Mi0wOS0xOSAgQmFsYXpzIEtlbGVtZW4gIDxrYmFsYXpzQHdlYmtpdC5vcmc+CisKICAgICAgICAg
W1F0XSBJbWxlbWVudCBwaXhlbCBzbmFwc2hvdCBnZW5lcmF0aW9uIGluIFdUUgogICAgICAgICBo
dHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9OTU5OTIKIApkaWZmIC0tZ2l0
IGEvU291cmNlL1dlYktpdDIvV2ViUHJvY2Vzcy9XZWJQYWdlL0Nvb3JkaW5hdGVkR3JhcGhpY3Mv
TGF5ZXJUcmVlQ29vcmRpbmF0b3IuY3BwIGIvU291cmNlL1dlYktpdDIvV2ViUHJvY2Vzcy9XZWJQ
YWdlL0Nvb3JkaW5hdGVkR3JhcGhpY3MvTGF5ZXJUcmVlQ29vcmRpbmF0b3IuY3BwCmluZGV4IDBk
MmJkNGQzZjRjNzU0MzY5OGFjMDFiZDM4YmNiOTk3YTBhMDVhNDQuLmIyNWNhZjA3N2Y0NTY3Mjgw
MzljZDgwOWFiOTliNGQzNjIwNTcyMmEgMTAwNjQ0Ci0tLSBhL1NvdXJjZS9XZWJLaXQyL1dlYlBy
b2Nlc3MvV2ViUGFnZS9Db29yZGluYXRlZEdyYXBoaWNzL0xheWVyVHJlZUNvb3JkaW5hdG9yLmNw
cAorKysgYi9Tb3VyY2UvV2ViS2l0Mi9XZWJQcm9jZXNzL1dlYlBhZ2UvQ29vcmRpbmF0ZWRHcmFw
aGljcy9MYXllclRyZWVDb29yZGluYXRvci5jcHAKQEAgLTI1NCw3ICsyNTQsNiBAQCBib29sIExh
eWVyVHJlZUNvb3JkaW5hdG9yOjpmbHVzaFBlbmRpbmdMYXllckNoYW5nZXMoKQogICAgIGlmICht
X3dhaXRpbmdGb3JVSVByb2Nlc3MpCiAgICAgICAgIHJldHVybiBmYWxzZTsKIAotICAgIG1fc2hv
dWxkU3luY0ZyYW1lID0gZmFsc2U7CiAgICAgYm9vbCBkaWRTeW5jID0gbV93ZWJQYWdlLT5jb3Jl
UGFnZSgpLT5tYWluRnJhbWUoKS0+dmlldygpLT5zeW5jQ29tcG9zaXRpbmdTdGF0ZUluY2x1ZGlu
Z1N1YmZyYW1lcygpOwogICAgIG1fbm9uQ29tcG9zaXRlZENvbnRlbnRMYXllci0+c3luY0NvbXBv
c2l0aW5nU3RhdGVGb3JUaGlzTGF5ZXJPbmx5KCk7CiAgICAgaWYgKG1fcGFnZU92ZXJsYXlMYXll
cikK
</data>

          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>164866</attachid>
            <date>2012-09-20 02:27:37 -0700</date>
            <delta_ts>2012-09-20 04:13:46 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-97108-20120920112643.patch</filename>
            <type>text/plain</type>
            <size>1956</size>
            <attacher name="Balazs Kelemen">kbalazs</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMTI5MTA5CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViS2l0Mi9D
aGFuZ2VMb2cgYi9Tb3VyY2UvV2ViS2l0Mi9DaGFuZ2VMb2cKaW5kZXggNDNhYmJmOWZjZTYzYzhm
MjI4MjAzNDE3MzlmNDM0YmJhYmYzN2RjNy4uZThlNzQ0OTUwOWU0ZTBjZWMyNGM3NjFkNThhODg5
OGRmMDcxOTQxZCAxMDA2NDQKLS0tIGEvU291cmNlL1dlYktpdDIvQ2hhbmdlTG9nCisrKyBiL1Nv
dXJjZS9XZWJLaXQyL0NoYW5nZUxvZwpAQCAtMSwzICsxLDE5IEBACisyMDEyLTA5LTIwICBCYWxh
enMgS2VsZW1lbiAgPGtiYWxhenNAd2Via2l0Lm9yZz4KKworICAgICAgICBbQ29vcmRpbmF0ZWRH
cmFwaGljc10gRG9uJ3QgcmVzZXQgbV9zaG91bGRTeW5jRnJhbWUgaW4gZmx1c2hQZW5kaW5nTGF5
ZXJDaGFuZ2VzCisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9p
ZD05NzEwOAorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAg
IFN0b3AgaWdub3JpbmcgaWYgbV9zaG91bGRTeW5jRnJhbWUgaGFzIGJlZW4gc2V0IGJldHdlZW4g
dGhlIHR3bworICAgICAgICBsYXllciBmbHVzaC4gSXQgY2FuIGJlIHNldCBkdXJpbmcgbGF5b3V0
IGluIHNldmVyYWwgc2l0dWF0aW9ucywKKyAgICAgICAgZm9yIGV4YW1wbGUgd2hlbiBhIGxheWVy
IGlzIGRlbGV0ZWQgb3IgY2hhbmdlZCBzdGF0ZS4gV2Ugd2FudCB0bworICAgICAgICBzZW5kIHRo
ZSBEaWRSZW5kZXJGcmFtZSBtZXNzYWdlIGF0IHRoZSBuZXh0IGZsdXNoIGluIHRob3NlIHNpdHVh
dGlvbnMKKyAgICAgICAgc28gdGhlIFVJIHByb2Nlc3Mgd2lsbCBhcHBseSB0aGUgY2hhbmdlcyBh
cyBzb29uIGFzIHBvc3NpYmxlLgorCisgICAgICAgICogV2ViUHJvY2Vzcy9XZWJQYWdlL0Nvb3Jk
aW5hdGVkR3JhcGhpY3MvTGF5ZXJUcmVlQ29vcmRpbmF0b3IuY3BwOgorICAgICAgICAoV2ViS2l0
OjpMYXllclRyZWVDb29yZGluYXRvcjo6Zmx1c2hQZW5kaW5nTGF5ZXJDaGFuZ2VzKToKKwogMjAx
Mi0wOS0yMCAgSmlud29vIFNvbmcgIDxqaW53b283LnNvbmdAc2Ftc3VuZy5jb20+CiAKICAgICAg
ICAgW0VGTF0gQ2hhbmdlIHRoZSBsb2cgbWFjcm8gbmFtZXMgdG8gYmUgbW9yZSBjb25zaXN0ZW50
IHdpdGggRUlOQSBMT0cKZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJLaXQyL1dlYlByb2Nlc3MvV2Vi
UGFnZS9Db29yZGluYXRlZEdyYXBoaWNzL0xheWVyVHJlZUNvb3JkaW5hdG9yLmNwcCBiL1NvdXJj
ZS9XZWJLaXQyL1dlYlByb2Nlc3MvV2ViUGFnZS9Db29yZGluYXRlZEdyYXBoaWNzL0xheWVyVHJl
ZUNvb3JkaW5hdG9yLmNwcAppbmRleCBkNGJmMDk5YWIwNTc3MTZiOWEzYTBmYjRkMDJhY2FhNTQ4
MzE4YjY5Li5kOTc5YWQ3MWM5YTBlNWRjODRjNGZmZWQ3ODEwMWI2ZjJmOTA3YzI4IDEwMDY0NAot
LS0gYS9Tb3VyY2UvV2ViS2l0Mi9XZWJQcm9jZXNzL1dlYlBhZ2UvQ29vcmRpbmF0ZWRHcmFwaGlj
cy9MYXllclRyZWVDb29yZGluYXRvci5jcHAKKysrIGIvU291cmNlL1dlYktpdDIvV2ViUHJvY2Vz
cy9XZWJQYWdlL0Nvb3JkaW5hdGVkR3JhcGhpY3MvTGF5ZXJUcmVlQ29vcmRpbmF0b3IuY3BwCkBA
IC0yNTQsOCArMjU0LDYgQEAgYm9vbCBMYXllclRyZWVDb29yZGluYXRvcjo6Zmx1c2hQZW5kaW5n
TGF5ZXJDaGFuZ2VzKCkKICAgICBpZiAobV93YWl0aW5nRm9yVUlQcm9jZXNzKQogICAgICAgICBy
ZXR1cm4gZmFsc2U7CiAKLSAgICBtX3Nob3VsZFN5bmNGcmFtZSA9IGZhbHNlOwotCiAgICAgZm9y
IChzaXplX3QgaSA9IDA7IGkgPCBtX2RldGFjaGVkTGF5ZXJzLnNpemUoKTsgKytpKQogICAgICAg
ICBtX3dlYlBhZ2UtPnNlbmQoTWVzc2FnZXM6OkxheWVyVHJlZUNvb3JkaW5hdG9yUHJveHk6OkRl
bGV0ZUNvbXBvc2l0aW5nTGF5ZXIobV9kZXRhY2hlZExheWVyc1tpXSkpOwogICAgIG1fZGV0YWNo
ZWRMYXllcnMuY2xlYXIoKTsK
</data>

          </attachment>
      

    </bug>

</bugzilla>