<?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>66566</bug_id>
          
          <creation_ts>2011-08-19 09:21:54 -0700</creation_ts>
          <short_desc>[CSSRegions]Speed up RenderFlowThread repaint of regions</short_desc>
          <delta_ts>2012-07-23 23:24:43 -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>CSS</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>
          
          <blocked>57312</blocked>
          <everconfirmed>0</everconfirmed>
          <reporter name="Mihnea Ovidenie">mihnea</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>webkit.review.bot</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>453874</commentid>
    <comment_count>0</comment_count>
    <who name="Mihnea Ovidenie">mihnea</who>
    <bug_when>2011-08-19 09:21:54 -0700</bug_when>
    <thetext>Improve RenderFlowThread::repaintRectangleInRegions by bailing out earlier if in printing or the repaint rectangle is empty. In this case, there is no need to iterate all the regions.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>453879</commentid>
    <comment_count>1</comment_count>
      <attachid>104517</attachid>
    <who name="Mihnea Ovidenie">mihnea</who>
    <bug_when>2011-08-19 09:31:41 -0700</bug_when>
    <thetext>Created attachment 104517
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>453943</commentid>
    <comment_count>2</comment_count>
      <attachid>104517</attachid>
    <who name="Dave Hyatt">hyatt</who>
    <bug_when>2011-08-19 11:33:26 -0700</bug_when>
    <thetext>Comment on attachment 104517
Patch

r=me</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>454001</commentid>
    <comment_count>3</comment_count>
      <attachid>104517</attachid>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2011-08-19 13:04:11 -0700</bug_when>
    <thetext>Comment on attachment 104517
Patch

Clearing flags on attachment: 104517

Committed r93432: &lt;http://trac.webkit.org/changeset/93432&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>454002</commentid>
    <comment_count>4</comment_count>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2011-08-19 13:04:15 -0700</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>104517</attachid>
            <date>2011-08-19 09:31:41 -0700</date>
            <delta_ts>2011-08-19 13:04:11 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>66566.patch</filename>
            <type>text/plain</type>
            <size>2450</size>
            <attacher name="Mihnea Ovidenie">mihnea</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZyBiL1NvdXJjZS9XZWJDb3JlL0No
YW5nZUxvZwppbmRleCBiMjcyNWZmLi5kMWNhOWM5IDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViQ29y
ZS9DaGFuZ2VMb2cKKysrIGIvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTkg
QEAKKzIwMTEtMDgtMTkgIE1paG5lYSBPdmlkZW5pZSAgPG1paG5lYUBhZG9iZS5jb20+CisKKyAg
ICAgICAgW0NTU1JlZ2lvbnNdU3BlZWQgdXAgUmVuZGVyRmxvd1RocmVhZCByZXBhaW50IG9mIHJl
Z2lvbnMKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTY2
NTY2CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgV2hl
biBpbiBwcmludGluZyBvciB0aGUgcmVwYWludCByZWN0YW5nbGUgaXMgZW1wdHksIHRoZXJlIGlz
IG5vIG5lZWQgdG8gaXRlcmF0ZQorICAgICAgICB0aGUgcmVuZGVyIGZsb3cgcmVnaW9ucyBpbiBv
cmRlciB0byByZXBhaW50LgorICAgICAgICBDb3ZlcmVkIGJ5IGV4aXN0aW5nIHRlc3RzLgorCisg
ICAgICAgICogcmVuZGVyaW5nL1JlbmRlckZsb3dUaHJlYWQuY3BwOgorICAgICAgICAoV2ViQ29y
ZTo6UmVuZGVyRmxvd1RocmVhZDo6c2hvdWxkUmVwYWludCk6CisgICAgICAgIChXZWJDb3JlOjpS
ZW5kZXJGbG93VGhyZWFkOjpyZXBhaW50UmVjdGFuZ2xlSW5SZWdpb25zKToKKyAgICAgICAgKiBy
ZW5kZXJpbmcvUmVuZGVyRmxvd1RocmVhZC5oOgorCiAyMDExLTA4LTE5ICBQYXZlbCBGZWxkbWFu
ICA8cGZlbGRtYW5AZ29vZ2xlLmNvbT4KIAogICAgICAgICBXZWIgSW5zcGVjdG9yOiBnZXRBdHRy
aWJ1dGVzIHNob3VsZCB3b3JrIG9uIGEgc2luZ2xlIG5vZGUsIG5vdCBhcnJheS4KZGlmZiAtLWdp
dCBhL1NvdXJjZS9XZWJDb3JlL3JlbmRlcmluZy9SZW5kZXJGbG93VGhyZWFkLmNwcCBiL1NvdXJj
ZS9XZWJDb3JlL3JlbmRlcmluZy9SZW5kZXJGbG93VGhyZWFkLmNwcAppbmRleCBkOWI4ZDk1Li4z
NDlmOTkwIDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViQ29yZS9yZW5kZXJpbmcvUmVuZGVyRmxvd1Ro
cmVhZC5jcHAKKysrIGIvU291cmNlL1dlYkNvcmUvcmVuZGVyaW5nL1JlbmRlckZsb3dUaHJlYWQu
Y3BwCkBAIC0zMCw2ICszMCw3IEBACiAjaW5jbHVkZSAiY29uZmlnLmgiCiAKICNpbmNsdWRlICJS
ZW5kZXJGbG93VGhyZWFkLmgiCisKICNpbmNsdWRlICJIaXRUZXN0UmVxdWVzdC5oIgogI2luY2x1
ZGUgIkhpdFRlc3RSZXN1bHQuaCIKICNpbmNsdWRlICJOb2RlLmgiCkBAIC00MDEsOCArNDAyLDE5
IEBAIGJvb2wgUmVuZGVyRmxvd1RocmVhZDo6aGl0VGVzdFJlZ2lvbihjb25zdCBMYXlvdXRSZWN0
JiByZWdpb25SZWN0LCBjb25zdCBIaXRUZXN0CiAgICAgcmV0dXJuIGlzUG9pbnRJbnNpZGVGbG93
VGhyZWFkOwogfQogCitib29sIFJlbmRlckZsb3dUaHJlYWQ6OnNob3VsZFJlcGFpbnQoY29uc3Qg
TGF5b3V0UmVjdCYgcikgY29uc3QKK3sKKyAgICBpZiAodmlldygpLT5wcmludGluZygpIHx8IHIu
aXNFbXB0eSgpKQorICAgICAgICByZXR1cm4gZmFsc2U7CisKKyAgICByZXR1cm4gdHJ1ZTsKK30K
Kwogdm9pZCBSZW5kZXJGbG93VGhyZWFkOjpyZXBhaW50UmVjdGFuZ2xlSW5SZWdpb25zKGNvbnN0
IExheW91dFJlY3QmIHJlcGFpbnRSZWN0LCBib29sIGltbWVkaWF0ZSkKIHsKKyAgICBpZiAoIXNo
b3VsZFJlcGFpbnQocmVwYWludFJlY3QpKQorICAgICAgICByZXR1cm47CisKICAgICBmb3IgKFJl
bmRlclJlZ2lvbkxpc3Q6Oml0ZXJhdG9yIGl0ZXIgPSBtX3JlZ2lvbkxpc3QuYmVnaW4oKTsgaXRl
ciAhPSBtX3JlZ2lvbkxpc3QuZW5kKCk7ICsraXRlcikgewogICAgICAgICBSZW5kZXJSZWdpb24q
IHJlZ2lvbiA9ICppdGVyOwogICAgICAgICBpZiAoIXJlZ2lvbi0+aXNWYWxpZCgpKQpkaWZmIC0t
Z2l0IGEvU291cmNlL1dlYkNvcmUvcmVuZGVyaW5nL1JlbmRlckZsb3dUaHJlYWQuaCBiL1NvdXJj
ZS9XZWJDb3JlL3JlbmRlcmluZy9SZW5kZXJGbG93VGhyZWFkLmgKaW5kZXggZTExYjFmYS4uZjM3
MzUzNSAxMDA2NDQKLS0tIGEvU291cmNlL1dlYkNvcmUvcmVuZGVyaW5nL1JlbmRlckZsb3dUaHJl
YWQuaAorKysgYi9Tb3VyY2UvV2ViQ29yZS9yZW5kZXJpbmcvUmVuZGVyRmxvd1RocmVhZC5oCkBA
IC0xMDQsNiArMTA0LDggQEAgcHJpdmF0ZToKICAgICB2b2lkIHJlbW92ZURlcGVuZGVuY3lPbkZs
b3dUaHJlYWQoUmVuZGVyRmxvd1RocmVhZCopOwogICAgIHZvaWQgY2hlY2tJbnZhbGlkUmVnaW9u
cygpOwogCisgICAgYm9vbCBzaG91bGRSZXBhaW50KGNvbnN0IExheW91dFJlY3QmKSBjb25zdDsK
KwogICAgIHR5cGVkZWYgTGlzdEhhc2hTZXQ8UmVuZGVyT2JqZWN0Kj4gRmxvd1RocmVhZENoaWxk
TGlzdDsKICAgICBGbG93VGhyZWFkQ2hpbGRMaXN0IG1fZmxvd1RocmVhZENoaWxkTGlzdDsKIAo=
</data>

          </attachment>
      

    </bug>

</bugzilla>