<?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>130124</bug_id>
          
          <creation_ts>2014-03-12 05:24:11 -0700</creation_ts>
          <short_desc>[CSS Grid layout] Initial position in span not correctly computed sometimes</short_desc>
          <delta_ts>2014-03-15 16:42:29 -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>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Sergio Villar Senin">svillar</reporter>
          <assigned_to name="Sergio Villar Senin">svillar</assigned_to>
          <cc>darin</cc>
    
    <cc>dino</cc>
    
    <cc>esprehn+autocc</cc>
    
    <cc>glenn</cc>
    
    <cc>hyatt</cc>
    
    <cc>jfernandez</cc>
    
    <cc>kling</cc>
    
    <cc>kondapallykalyan</cc>
    
    <cc>rego</cc>
    
    <cc>simon.fraser</cc>
    
    <cc>svillar</cc>
    
    <cc>webkit-ews</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>989599</commentid>
    <comment_count>0</comment_count>
    <who name="Sergio Villar Senin">svillar</who>
    <bug_when>2014-03-12 05:24:11 -0700</bug_when>
    <thetext>We&apos;re using std::lower_bound to find the first line before the opposite resolved position. The returned iterator might point to a resolved position which is actually greater than the opposite position (since lower_bound returns the first non less than). In those cases we need to move the iterator to the previous line so that the span can be correctly computed.

For example, let&apos;s imagine that we try to compute the span for a given named line which is defined in these tree positions (tracks)

[2, 5, 8]

Suppose that the opposite final position to the span was computed as 7. Then std::lower_bound will return an iterator pointing to 8. We&apos;re looking for the first position before 7, and obviously 8 is after, so we have to move the iterator to the previous position which would be 5. That&apos;s because how std::lower_bound works, if the iterator points to a value &lt;= then we use it, but if it&apos;s &gt; then we have to use the previous one.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>989602</commentid>
    <comment_count>1</comment_count>
      <attachid>226497</attachid>
    <who name="Sergio Villar Senin">svillar</who>
    <bug_when>2014-03-12 06:06:15 -0700</bug_when>
    <thetext>Created attachment 226497
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>990081</commentid>
    <comment_count>2</comment_count>
      <attachid>226497</attachid>
    <who name="Darin Adler">darin</who>
    <bug_when>2014-03-13 11:16:52 -0700</bug_when>
    <thetext>Comment on attachment 226497
Patch

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

&gt; Source/WebCore/rendering/RenderGrid.cpp:1035
&gt;      const size_t* firstLineBeforeOppositePosition = std::lower_bound(gridLines.begin(), gridLines.end(), resolvedOppositePosition);
&gt; -    if (firstLineBeforeOppositePosition != gridLines.end())
&gt; +    if (firstLineBeforeOppositePosition != gridLines.end()) {
&gt; +        if (*firstLineBeforeOppositePosition &gt; resolvedOppositePosition &amp;&amp; firstLineBeforeOppositePosition != gridLines.begin())
&gt; +            --firstLineBeforeOppositePosition;
&gt; +
&gt;          firstLineBeforeOppositePositionIndex = firstLineBeforeOppositePosition - gridLines.begin();
&gt; +    }

Awkward. Makes me think we should put this whole thing into a helper function rather than invoking lower_bound directly in-line.

&gt; LayoutTests/fast/css-grid-layout/grid-item-named-grid-line-resolution-expected.txt:26
&gt; +PASS
&gt; +PASS
&gt; +PASS
&gt; +PASS
&gt; +PASS
&gt; +PASS
&gt; +PASS
&gt; +PASS
&gt; +PASS
&gt; +PASS

Not great test output here. Worth thinking in the future about things we can include in tests to make it clear what we are testing rather than just a lot of PASS/FAIL. Helpful in many cases, for example, when some tests are failing it helps you see which ones they are.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>990082</commentid>
    <comment_count>3</comment_count>
    <who name="Darin Adler">darin</who>
    <bug_when>2014-03-13 11:17:17 -0700</bug_when>
    <thetext>Naming that helper function would help clarify what this code is doing at a mechanical level.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>990364</commentid>
    <comment_count>4</comment_count>
    <who name="Sergio Villar Senin">svillar</who>
    <bug_when>2014-03-14 02:42:16 -0700</bug_when>
    <thetext>(In reply to comment #2)
&gt; (From update of attachment 226497 [details])
&gt; View in context: https://bugs.webkit.org/attachment.cgi?id=226497&amp;action=review
&gt; &gt; LayoutTests/fast/css-grid-layout/grid-item-named-grid-line-resolution-expected.txt:26
&gt; &gt; +PASS
&gt; &gt; +PASS
&gt; &gt; +PASS
&gt; &gt; +PASS
&gt; &gt; +PASS
&gt; &gt; +PASS
&gt; &gt; +PASS
&gt; &gt; +PASS
&gt; &gt; +PASS
&gt; &gt; +PASS
&gt; 
&gt; Not great test output here. Worth thinking in the future about things we can include in tests to make it clear what we are testing rather than just a lot of PASS/FAIL. Helpful in many cases, for example, when some tests are failing it helps you see which ones they are.

Although I agree that the output is far from optimal, in this case the testing code is the one that is very descriptive. We&apos;re using checkLayout() to check the offsets and sizes of grid items. That allows us to test with great detail the positioning of grid items. In case of error, that JS helper prints the reason why the test fails (like expected 30 for width but found 50) and also the original testing code so it&apos;s quite easy to check the reason of the failure.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>990377</commentid>
    <comment_count>5</comment_count>
    <who name="Sergio Villar Senin">svillar</who>
    <bug_when>2014-03-14 03:19:00 -0700</bug_when>
    <thetext>Committed r165612: &lt;http://trac.webkit.org/changeset/165612&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>990900</commentid>
    <comment_count>6</comment_count>
    <who name="Darin Adler">darin</who>
    <bug_when>2014-03-15 16:42:29 -0700</bug_when>
    <thetext>(In reply to comment #4)
&gt; that JS helper prints the reason why the test fails (like expected 30 for width but found 50) and also the original testing code so it&apos;s quite easy to check the reason of the failure.

If it just printed slightly more of the same when passing, I’d be happy.

When a test fails it’s good to be reminded of the similar tests that are passing.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>226497</attachid>
            <date>2014-03-12 06:06:15 -0700</date>
            <delta_ts>2014-03-13 11:16:52 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-130124-20140312140549.patch</filename>
            <type>text/plain</type>
            <size>7614</size>
            <attacher name="Sergio Villar Senin">svillar</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMTY1NDUyCmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9D
aGFuZ2VMb2cgYi9Tb3VyY2UvV2ViQ29yZS9DaGFuZ2VMb2cKaW5kZXggYThjMmZlZmExNmM1YzJi
YjhlOGFmMTEzZTc5OTM5OTJkMDc5N2M1OS4uODE5NDlmY2RmYzJlZjdjNjkwMmZhN2RlMDg5ZjRj
ZGM2MzU3MGRlMiAxMDA2NDQKLS0tIGEvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCisrKyBiL1Nv
dXJjZS9XZWJDb3JlL0NoYW5nZUxvZwpAQCAtMSwzICsxLDI5IEBACisyMDE0LTAzLTEyICBTZXJn
aW8gVmlsbGFyIFNlbmluICA8c3ZpbGxhckBpZ2FsaWEuY29tPgorCisgICAgICAgIFtDU1MgR3Jp
ZCBsYXlvdXRdIEluaXRpYWwgcG9zaXRpb24gaW4gc3BhbiBub3QgY29ycmVjdGx5IGNvbXB1dGVk
IHNvbWV0aW1lcworICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/
aWQ9MTMwMTI0CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAg
ICAgVGhlIGNvZGUgdXNlcyBzdGQ6Omxvd2VyX2JvdW5kKCkgdG8gZ2V0IHRoZSBmaXJzdCBncmlk
IGxpbmUKKyAgICAgICAgcG9zaXRpb24gYmVmb3JlIGEgZ2l2ZW4gcmVzb2x2ZWQgcG9zaXRpb24g
aW4gYSBWZWN0b3IuIFRoYXQKKyAgICAgICAgZnVuY3Rpb24gcmV0dXJucyBhbiBpdGVyYXRvciBw
b2ludGluZyB0byB0aGUgZmlyc3QgdmFsdWUgbm90CisgICAgICAgICJzdHJpY3RseSBsZXNzIiB0
aGFuIHRoZSBnaXZlbiBvbmUuIENvbnNlcXVlbnRseSwgdGhlIHJldHVybmVkCisgICAgICAgIHBv
c2l0aW9uIG1pZ2h0IGJlIGFjdHVhbGx5IGxvY2F0ZWQgYWZ0ZXIgdGhlIHJlc29sdmVkIGZpbmFs
CisgICAgICAgIHBvc2l0aW9uLiBJbiB0aG9zZSBjYXNlcyB3ZSBuZWVkIHRvIHNlbGVjdCB0aGUg
cHJldmlvdXMgcG9zaXRpb24KKyAgICAgICAgaW4gdGhlIFZlY3RvciAod2hpY2ggaXMgZ3VhcmFu
dGVkIHRvIGJlICI8IHJlc29sdmVkIHBvc2l0aW9uIiBkdWUKKyAgICAgICAgdG8gaG93IHN0ZDo6
bG93ZXJfYm91bmQoKSB3b3JrcykuCisKKyAgICAgICAgRm9yIGV4YW1wbGUsIGlmIHdlIGhhdmUg
dmVjdG9yPVsyLDUsOF0gYXMgZ3JpZCBwb3NpdGlvbnMsIGFuZCB3ZQorICAgICAgICBuZWVkIHRo
ZSBmaXJzdCBwb3NpdGlvbiBiZWZvcmUgNywgc3RkOjpsb3dlcl9ib3VuZCh2ZWN0b3IsIDcpIHdp
bGwKKyAgICAgICAgcG9pbnQgdG8gOCAoZmlyc3QgdmFsdWUgbm90IDwgNykuIEl0J3Mgb2J2aW91
cyB0aGF0IHdlIHNob3VsZAorICAgICAgICBzZWxlY3QgNSBpbnN0ZWFkLiBTaG91bGQgdGhlIHBv
aW50ZWQgdmFsdWUgYmUgZXF1YWwsIHRoZW4gd2UKKyAgICAgICAgc2hvdWxkIGRvIG5vdGhpbmcs
IGJlY2F1c2UgdGhlIGluZGV4ZXMgbWVhbiB0cmFja3MsIHNvIHRoZSBpdGVtCisgICAgICAgIHdp
bGwgYmUgcGxhY2VkIGluIGp1c3Qgb25lIGNlbGwgb2YgdGhlIGdyaWQuCisKKyAgICAgICAgKiBy
ZW5kZXJpbmcvUmVuZGVyR3JpZC5jcHA6CisgICAgICAgIChXZWJDb3JlOjpSZW5kZXJHcmlkOjpy
ZXNvbHZlUm93U3RhcnRDb2x1bW5TdGFydE5hbWVkR3JpZExpbmVQb3NpdGlvbkFnYWluc3RPcHBv
c2l0ZVBvc2l0aW9uKToKKwogMjAxNC0wMy0xMSAgUnl1YW4gQ2hvaSAgPHJ5dWFuLmNob2lAc2Ft
c3VuZy5jb20+CiAKICAgICAgICAgUmVtb3ZlIHVudXNlZCBtZXRob2QgZnJvbSBCYXR0ZXJ5Q29u
dHJvbGxlcgpkaWZmIC0tZ2l0IGEvU291cmNlL1dlYkNvcmUvcmVuZGVyaW5nL1JlbmRlckdyaWQu
Y3BwIGIvU291cmNlL1dlYkNvcmUvcmVuZGVyaW5nL1JlbmRlckdyaWQuY3BwCmluZGV4IGY1MjYx
NjQ3MzU0YTg2YTIwOTM1NTdkNzgzZGRkMDM4MmU5YmIzNjUuLmMxYTk4MWQ0NWMwMjVmZDg4NmI2
MmRkNjdlYWM0N2ExZGJlZDQ2YzAgMTAwNjQ0Ci0tLSBhL1NvdXJjZS9XZWJDb3JlL3JlbmRlcmlu
Zy9SZW5kZXJHcmlkLmNwcAorKysgYi9Tb3VyY2UvV2ViQ29yZS9yZW5kZXJpbmcvUmVuZGVyR3Jp
ZC5jcHAKQEAgLTEwMjcsOCArMTAyNywxMiBAQCBQYXNzT3duUHRyPEdyaWRTcGFuPiBSZW5kZXJH
cmlkOjpyZXNvbHZlUm93U3RhcnRDb2x1bW5TdGFydE5hbWVkR3JpZExpbmVQb3NpdGlvbgogICAg
IC8vIGlzIGFscmVhZHkgY29udmVydGVkIHRvIGFuIGluZGV4IGluIG91ciBncmlkIHJlcHJlc2Vu
dGF0aW9uIChpZSBvbmUgd2FzIHJlbW92ZWQgZnJvbSB0aGUgZ3JpZCBsaW5lIHRvIGFjY291bnQg
Zm9yIHRoZSBzaWRlKS4KICAgICBzaXplX3QgZmlyc3RMaW5lQmVmb3JlT3Bwb3NpdGVQb3NpdGlv
bkluZGV4ID0gMDsKICAgICBjb25zdCBzaXplX3QqIGZpcnN0TGluZUJlZm9yZU9wcG9zaXRlUG9z
aXRpb24gPSBzdGQ6Omxvd2VyX2JvdW5kKGdyaWRMaW5lcy5iZWdpbigpLCBncmlkTGluZXMuZW5k
KCksIHJlc29sdmVkT3Bwb3NpdGVQb3NpdGlvbik7Ci0gICAgaWYgKGZpcnN0TGluZUJlZm9yZU9w
cG9zaXRlUG9zaXRpb24gIT0gZ3JpZExpbmVzLmVuZCgpKQorICAgIGlmIChmaXJzdExpbmVCZWZv
cmVPcHBvc2l0ZVBvc2l0aW9uICE9IGdyaWRMaW5lcy5lbmQoKSkgeworICAgICAgICBpZiAoKmZp
cnN0TGluZUJlZm9yZU9wcG9zaXRlUG9zaXRpb24gPiByZXNvbHZlZE9wcG9zaXRlUG9zaXRpb24g
JiYgZmlyc3RMaW5lQmVmb3JlT3Bwb3NpdGVQb3NpdGlvbiAhPSBncmlkTGluZXMuYmVnaW4oKSkK
KyAgICAgICAgICAgIC0tZmlyc3RMaW5lQmVmb3JlT3Bwb3NpdGVQb3NpdGlvbjsKKwogICAgICAg
ICBmaXJzdExpbmVCZWZvcmVPcHBvc2l0ZVBvc2l0aW9uSW5kZXggPSBmaXJzdExpbmVCZWZvcmVP
cHBvc2l0ZVBvc2l0aW9uIC0gZ3JpZExpbmVzLmJlZ2luKCk7CisgICAgfQogCiAgICAgc2l6ZV90
IGdyaWRMaW5lSW5kZXggPSBzdGQ6Om1heDxpbnQ+KDAsIGZpcnN0TGluZUJlZm9yZU9wcG9zaXRl
UG9zaXRpb25JbmRleCAtIHBvc2l0aW9uLnNwYW5Qb3NpdGlvbigpICsgMSk7CiAgICAgc2l6ZV90
IHJlc29sdmVkR3JpZExpbmVQb3NpdGlvbiA9IGdyaWRMaW5lc1tncmlkTGluZUluZGV4XTsKZGlm
ZiAtLWdpdCBhL0xheW91dFRlc3RzL0NoYW5nZUxvZyBiL0xheW91dFRlc3RzL0NoYW5nZUxvZwpp
bmRleCAxOGNjNmI1MGMwYTBjNDgxYWJhOTkwMjg4YjZiNWUxMDAxY2FjZjg4Li45YmI4NzZhMTMw
ZmExZTBhYzQxYTMxYzIwOGJlNTAyNzkyZWYwNjBjIDEwMDY0NAotLS0gYS9MYXlvdXRUZXN0cy9D
aGFuZ2VMb2cKKysrIGIvTGF5b3V0VGVzdHMvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTYgQEAKKzIw
MTQtMDMtMTIgIFNlcmdpbyBWaWxsYXIgU2VuaW4gIDxzdmlsbGFyQGlnYWxpYS5jb20+CisKKyAg
ICAgICAgW0NTUyBHcmlkIGxheW91dF0gSW5pdGlhbCBwb3NpdGlvbiBpbiBzcGFuIG5vdCBjb3Jy
ZWN0bHkgY29tcHV0ZWQgc29tZXRpbWVzCisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3Jn
L3Nob3dfYnVnLmNnaT9pZD0xMzAxMjQKKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9P
UFMhKS4KKworICAgICAgICBBZGRlZCB0aGUgZ3JpZCBwb3NpdGlvbmluZyBleGFtcGxlIG1lbnRp
b25lZCBpbiB0aGUgc3BlY3Mgd2hpY2gKKyAgICAgICAgY292ZXJzIHRoaXMgYnVnIGFuZCBzb21l
IG90aGVyIGNhc2VzLgorCisgICAgICAgICogZmFzdC9jc3MtZ3JpZC1sYXlvdXQvZ3JpZC1pdGVt
LW5hbWVkLWdyaWQtbGluZS1yZXNvbHV0aW9uLWV4cGVjdGVkLnR4dDoKKyAgICAgICAgKiBmYXN0
L2Nzcy1ncmlkLWxheW91dC9ncmlkLWl0ZW0tbmFtZWQtZ3JpZC1saW5lLXJlc29sdXRpb24uaHRt
bDoKKwogMjAxNC0wMy0xMSAgTWFudWVsIFJlZ28gQ2FzYXNub3ZhcyAgPHJlZ29AaWdhbGlhLmNv
bT4KIAogICAgICAgICBbQ1NTIEdyaWQgTGF5b3V0XSBBZGQgbGF5b3V0IHRlc3QgZm9yIGFub255
bW91cyBncmlkIGl0ZW1zCmRpZmYgLS1naXQgYS9MYXlvdXRUZXN0cy9mYXN0L2Nzcy1ncmlkLWxh
eW91dC9ncmlkLWl0ZW0tbmFtZWQtZ3JpZC1saW5lLXJlc29sdXRpb24tZXhwZWN0ZWQudHh0IGIv
TGF5b3V0VGVzdHMvZmFzdC9jc3MtZ3JpZC1sYXlvdXQvZ3JpZC1pdGVtLW5hbWVkLWdyaWQtbGlu
ZS1yZXNvbHV0aW9uLWV4cGVjdGVkLnR4dAppbmRleCBkMjkzNjY4ZTI4MWU0YWY3MWEzZGU4NTk1
ZmJlNmFkNmJmMjkzNDNmLi5kYWQ2ZjBmODFhYjVkNzc1MzNlOWE4YjhmMjllOTNiOTZmZTdkY2Iw
IDEwMDY0NAotLS0gYS9MYXlvdXRUZXN0cy9mYXN0L2Nzcy1ncmlkLWxheW91dC9ncmlkLWl0ZW0t
bmFtZWQtZ3JpZC1saW5lLXJlc29sdXRpb24tZXhwZWN0ZWQudHh0CisrKyBiL0xheW91dFRlc3Rz
L2Zhc3QvY3NzLWdyaWQtbGF5b3V0L2dyaWQtaXRlbS1uYW1lZC1ncmlkLWxpbmUtcmVzb2x1dGlv
bi1leHBlY3RlZC50eHQKQEAgLTE0LDMgKzE0LDEzIEBAIFBBU1MKIFBBU1MKIFBBU1MKIFBBU1MK
K1BBU1MKK1BBU1MKK1BBU1MKK1BBU1MKK1BBU1MKK1BBU1MKK1BBU1MKK1BBU1MKK1BBU1MKK1BB
U1MKZGlmZiAtLWdpdCBhL0xheW91dFRlc3RzL2Zhc3QvY3NzLWdyaWQtbGF5b3V0L2dyaWQtaXRl
bS1uYW1lZC1ncmlkLWxpbmUtcmVzb2x1dGlvbi5odG1sIGIvTGF5b3V0VGVzdHMvZmFzdC9jc3Mt
Z3JpZC1sYXlvdXQvZ3JpZC1pdGVtLW5hbWVkLWdyaWQtbGluZS1yZXNvbHV0aW9uLmh0bWwKaW5k
ZXggNWM5YTQ0NzAzMmZkNmFjMzhmYzU3NWNhMTAyMzJmMDVlYmM5YThmNC4uMmM3ZGFiNTkxYmNi
OWE1MjE3NmQyYjc3YjMyYWEyZTYzZTU0OWNiMyAxMDA2NDQKLS0tIGEvTGF5b3V0VGVzdHMvZmFz
dC9jc3MtZ3JpZC1sYXlvdXQvZ3JpZC1pdGVtLW5hbWVkLWdyaWQtbGluZS1yZXNvbHV0aW9uLmh0
bWwKKysrIGIvTGF5b3V0VGVzdHMvZmFzdC9jc3MtZ3JpZC1sYXlvdXQvZ3JpZC1pdGVtLW5hbWVk
LWdyaWQtbGluZS1yZXNvbHV0aW9uLmh0bWwKQEAgLTE3LDYgKzE3LDExIEBAIGlmICh3aW5kb3cu
dGVzdFJ1bm5lcikKICAgICAtd2Via2l0LWdyaWQtdGVtcGxhdGUtcm93czogJ2cnIDUwcHggJ2cn
IDEwMHB4ICdnJyAyMDBweCAnZyc7CiB9CiAKKy5ncmlkRnJvbVNwZWNzIHsKKyAgICAtd2Via2l0
LWdyaWQtdGVtcGxhdGUtY29sdW1uczogJ0EnIDEwcHggJ0InIDIwcHggJ0MnIDMwcHggJ0EnIDQw
cHggJ0InIDUwcHggJ0MnIDYwcHggJ0EnIDcwcHggJ0InIDgwcHggJ0MnOworICAgIC13ZWJraXQt
Z3JpZC10ZW1wbGF0ZS1yb3dzOiAxMDBweDsKK30KKwogLmdyaWRJdGVtQlRvRCB7CiAgICAgLXdl
YmtpdC1ncmlkLWNvbHVtbjogJ2InIC8gJ2QnOwogICAgIC13ZWJraXQtZ3JpZC1yb3c6IDE7CkBA
IC0xNTcsNSArMTYyLDY1IEBAIGlmICh3aW5kb3cudGVzdFJ1bm5lcikKICAgICA8L2Rpdj4KIDwv
ZGl2PgogCis8ZGl2IHN0eWxlPSJwb3NpdGlvbjogcmVsYXRpdmUiPgorICAgIDxkaXYgY2xhc3M9
ImdyaWQgZ3JpZEZyb21TcGVjcyI+CisgICAgICA8ZGl2IGNsYXNzPSJzaXplZFRvR3JpZEFyZWEi
IHN0eWxlPSItd2Via2l0LWdyaWQtY29sdW1uOiA0IC8gYXV0bzsiIGRhdGEtb2Zmc2V0LXg9IjYw
cHgiIGRhdGEtZXhwZWN0ZWQtd2lkdGg9IjQwcHgiPjwvZGl2PgorICAgIDwvZGl2PgorPC9kaXY+
CisKKzxkaXYgc3R5bGU9InBvc2l0aW9uOiByZWxhdGl2ZSI+CisgICAgPGRpdiBjbGFzcz0iZ3Jp
ZCBncmlkRnJvbVNwZWNzIj4KKyAgICAgIDxkaXYgY2xhc3M9InNpemVkVG9HcmlkQXJlYSIgc3R5
bGU9Ii13ZWJraXQtZ3JpZC1jb2x1bW46IGF1dG8gLyA2OyIgZGF0YS1vZmZzZXQteD0iMTAwcHgi
IGRhdGEtZXhwZWN0ZWQtd2lkdGg9IjUwcHgiPjwvZGl2PgorICAgIDwvZGl2PgorPC9kaXY+CisK
KzxkaXYgc3R5bGU9InBvc2l0aW9uOiByZWxhdGl2ZSI+CisgICAgPGRpdiBjbGFzcz0iZ3JpZCBn
cmlkRnJvbVNwZWNzIj4KKyAgICAgIDxkaXYgY2xhc3M9InNpemVkVG9HcmlkQXJlYSIgc3R5bGU9
Ii13ZWJraXQtZ3JpZC1jb2x1bW46ICdDJyAvICdDJyAtMTsiIGRhdGEtb2Zmc2V0LXg9IjMwcHgi
IGRhdGEtZXhwZWN0ZWQtd2lkdGg9IjMzMHB4Ij48L2Rpdj4KKyAgICA8L2Rpdj4KKzwvZGl2Pgor
Cis8ZGl2IHN0eWxlPSJwb3NpdGlvbjogcmVsYXRpdmUiPgorICAgIDxkaXYgY2xhc3M9ImdyaWQg
Z3JpZEZyb21TcGVjcyI+CisgICAgICA8ZGl2IGNsYXNzPSJzaXplZFRvR3JpZEFyZWEiIHN0eWxl
PSItd2Via2l0LWdyaWQtY29sdW1uOiAnQycgLyBzcGFuICdDJzsiIGRhdGEtb2Zmc2V0LXg9IjMw
cHgiIGRhdGEtZXhwZWN0ZWQtd2lkdGg9IjEyMHB4Ij48L2Rpdj4KKyAgICA8L2Rpdj4KKzwvZGl2
PgorCis8ZGl2IHN0eWxlPSJwb3NpdGlvbjogcmVsYXRpdmUiPgorICAgIDxkaXYgY2xhc3M9Imdy
aWQgZ3JpZEZyb21TcGVjcyI+CisgICAgICA8ZGl2IGNsYXNzPSJzaXplZFRvR3JpZEFyZWEiIHN0
eWxlPSItd2Via2l0LWdyaWQtY29sdW1uOiBzcGFuICdDJyAvICdDJyAtMTsiIGRhdGEtb2Zmc2V0
LXg9IjE1MHB4IiBkYXRhLWV4cGVjdGVkLXdpZHRoPSIyMTBweCI+PC9kaXY+CisgICAgPC9kaXY+
Cis8L2Rpdj4KKworPGRpdiBzdHlsZT0icG9zaXRpb246IHJlbGF0aXZlIj4KKyAgICA8ZGl2IGNs
YXNzPSJncmlkIGdyaWRGcm9tU3BlY3MiPgorICAgICAgPGRpdiBjbGFzcz0ic2l6ZWRUb0dyaWRB
cmVhIiBzdHlsZT0iLXdlYmtpdC1ncmlkLWNvbHVtbjogc3BhbiAnQycgLyBzcGFuICdDJzsiIGRh
dGEtb2Zmc2V0LXg9IjBweCIgZGF0YS1leHBlY3RlZC13aWR0aD0iMTBweCI+PC9kaXY+CisgICAg
PC9kaXY+Cis8L2Rpdj4KKworPGRpdiBzdHlsZT0icG9zaXRpb246IHJlbGF0aXZlIj4KKyAgICA8
ZGl2IGNsYXNzPSJncmlkIGdyaWRGcm9tU3BlY3MiPgorICAgICAgPGRpdiBjbGFzcz0ic2l6ZWRU
b0dyaWRBcmVhIiBzdHlsZT0iLXdlYmtpdC1ncmlkLWNvbHVtbjogNSAvICdDJyAtMTsiIGRhdGEt
b2Zmc2V0LXg9IjEwMHB4IiBkYXRhLWV4cGVjdGVkLXdpZHRoPSIyNjBweCI+PC9kaXY+CisgICAg
PC9kaXY+Cis8L2Rpdj4KKworPGRpdiBzdHlsZT0icG9zaXRpb246IHJlbGF0aXZlIj4KKyAgICA8
ZGl2IGNsYXNzPSJncmlkIGdyaWRGcm9tU3BlY3MiPgorICAgICAgPGRpdiBjbGFzcz0ic2l6ZWRU
b0dyaWRBcmVhIiBzdHlsZT0iLXdlYmtpdC1ncmlkLWNvbHVtbjogNSAvIHNwYW4gJ0MnOyIgZGF0
YS1vZmZzZXQteD0iMTAwcHgiIGRhdGEtZXhwZWN0ZWQtd2lkdGg9IjUwcHgiPjwvZGl2PgorICAg
IDwvZGl2PgorPC9kaXY+CisKKzxkaXYgc3R5bGU9InBvc2l0aW9uOiByZWxhdGl2ZSI+CisgICAg
PGRpdiBjbGFzcz0iZ3JpZCBncmlkRnJvbVNwZWNzIj4KKyAgICAgIDxkaXYgY2xhc3M9InNpemVk
VG9HcmlkQXJlYSIgc3R5bGU9Ii13ZWJraXQtZ3JpZC1jb2x1bW46IDggLyA4OyIgZGF0YS1vZmZz
ZXQteD0iMjgwcHgiIGRhdGEtZXhwZWN0ZWQtd2lkdGg9IjgwcHgiPjwvZGl2PgorICAgIDwvZGl2
PgorPC9kaXY+CisKKzxkaXYgc3R5bGU9InBvc2l0aW9uOiByZWxhdGl2ZSI+CisgICAgPGRpdiBj
bGFzcz0iZ3JpZCBncmlkRnJvbVNwZWNzIj4KKyAgICAgIDxkaXYgY2xhc3M9InNpemVkVG9Hcmlk
QXJlYSIgc3R5bGU9Ii13ZWJraXQtZ3JpZC1jb2x1bW46ICdCJyAyIC8gc3BhbiAxOyIgZGF0YS1v
ZmZzZXQteD0iMTAwcHgiIGRhdGEtZXhwZWN0ZWQtd2lkdGg9IjUwcHgiPjwvZGl2PgorICAgIDwv
ZGl2PgorPC9kaXY+CisKIDwvYm9keT4KIDwvaHRtbD4K
</data>
<flag name="review"
          id="250737"
          type_id="1"
          status="+"
          setter="darin"
    />
          </attachment>
      

    </bug>

</bugzilla>