<?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>164211</bug_id>
          
          <creation_ts>2016-10-30 16:54:03 -0700</creation_ts>
          <short_desc>[CSS Parser] Miscellaneous bug fixes</short_desc>
          <delta_ts>2016-10-31 08:04:22 -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>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></keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Dave Hyatt">hyatt</reporter>
          <assigned_to name="Dave Hyatt">hyatt</assigned_to>
          
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1246292</commentid>
    <comment_count>0</comment_count>
    <who name="Dave Hyatt">hyatt</who>
    <bug_when>2016-10-30 16:54:03 -0700</bug_when>
    <thetext>[CSS Parser] Miscellaneous bug fixes</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1246293</commentid>
    <comment_count>1</comment_count>
      <attachid>293372</attachid>
    <who name="Dave Hyatt">hyatt</who>
    <bug_when>2016-10-30 16:57:27 -0700</bug_when>
    <thetext>Created attachment 293372
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1246306</commentid>
    <comment_count>2</comment_count>
      <attachid>293372</attachid>
    <who name="Darin Adler">darin</who>
    <bug_when>2016-10-30 17:59:22 -0700</bug_when>
    <thetext>Comment on attachment 293372
Patch

Strange to see all these patches for bug fixes, but no test cases. I assume that these fixes are needed to pass existing WebKit tests with the new parse?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1246310</commentid>
    <comment_count>3</comment_count>
      <attachid>293372</attachid>
    <who name="Build Bot">buildbot</who>
    <bug_when>2016-10-30 18:18:19 -0700</bug_when>
    <thetext>Comment on attachment 293372
Patch

Attachment 293372 did not pass ios-sim-ews (ios-simulator-wk2):
Output: http://webkit-queues.webkit.org/results/2407920

New failing tests:
canvas/philip/tests/2d.canvas.reference.html</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1246311</commentid>
    <comment_count>4</comment_count>
      <attachid>293379</attachid>
    <who name="Build Bot">buildbot</who>
    <bug_when>2016-10-30 18:18:22 -0700</bug_when>
    <thetext>Created attachment 293379
Archive of layout-test-results from ews125 for ios-simulator-wk2

The attached test failures were seen while running run-webkit-tests on the ios-sim-ews.
Bot: ews125  Port: ios-simulator-wk2  Platform: Mac OS X 10.11.6</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1246417</commentid>
    <comment_count>5</comment_count>
    <who name="Dave Hyatt">hyatt</who>
    <bug_when>2016-10-31 08:03:14 -0700</bug_when>
    <thetext>(In reply to comment #2)
&gt; Comment on attachment 293372 [details]
&gt; Patch
&gt; 
&gt; Strange to see all these patches for bug fixes, but no test cases. I assume
&gt; that these fixes are needed to pass existing WebKit tests with the new parse?

Yeah, all these fixes are about getting the new parser to pass our existing tests.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1246418</commentid>
    <comment_count>6</comment_count>
    <who name="Dave Hyatt">hyatt</who>
    <bug_when>2016-10-31 08:04:22 -0700</bug_when>
    <thetext>Fixed in r208142.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>293372</attachid>
            <date>2016-10-30 16:57:27 -0700</date>
            <delta_ts>2016-10-30 18:18:19 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-164211-20161030185332.patch</filename>
            <type>text/plain</type>
            <size>4644</size>
            <attacher name="Dave Hyatt">hyatt</attacher>
            
              <data encoding="base64">SW5kZXg6IFNvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2UvV2Vi
Q29yZS9DaGFuZ2VMb2cJKHJldmlzaW9uIDIwODEzNikKKysrIFNvdXJjZS9XZWJDb3JlL0NoYW5n
ZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwzICsxLDI4IEBACisyMDE2LTEwLTMwICBEYXZlIEh5
YXR0ICA8aHlhdHRAYXBwbGUuY29tPgorCisgICAgICAgIFtDU1MgUGFyc2VyXSBNaXNjZWxsYW5l
b3VzIGJ1ZyBmaXhlcworICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5j
Z2k/aWQ9MTY0MjExCisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAg
ICAgICAgKiBjc3MvcGFyc2VyL0NTU1Byb3BlcnR5UGFyc2VyLmNwcDoKKyAgICAgICAgKFdlYkNv
cmU6OkNTU1Byb3BlcnR5UGFyc2VyOjpjb25zdW1lVHJhbnNmb3JtT3JpZ2luKToKKyAgICAgICAg
TWFrZSBzdXJlIHRvIHByb3Blcmx5IHJlamVjdCBpbnZhbGlkIHRyYW5zZm9ybS1vcmlnaW4teiB2
YWx1ZXMuCisKKyAgICAgICAgKiBjc3MvcGFyc2VyL0NTU1NlbGVjdG9yUGFyc2VyLmNwcDoKKyAg
ICAgICAgKFdlYkNvcmU6OkNTU1NlbGVjdG9yUGFyc2VyOjpjb25zdW1lQXR0cmlidXRlKToKKyAg
ICAgICAgTWFrZSBhdHRyaWJ1dGUgc2VsZWN0b3IgcGFyc2luZyBzdHJpY3QgYWJvdXQgcmVxdWly
aW5nIGEgXSB0byBlbmQKKyAgICAgICAgdGhlIHNlbGVjdG9yLgorCisgICAgICAgIChXZWJDb3Jl
OjpDU1NTZWxlY3RvclBhcnNlcjo6Y29uc3VtZVBzZXVkbyk6CisgICAgICAgIEZvcmNlIHRoZSBu
dGgtY2hpbGQgIm9mIiBzeW50YXggdG8gaGF2ZSB3aGl0ZXNwYWNlIGFmdGVyICJvZiIgYnV0Cisg
ICAgICAgIGJlZm9yZSB0aGUgc2VsZWN0b3IuIEl0IGlzIHVuY2xlYXIgaWYgdGhpcyBzaG91bGQg
YmUgYSByZXF1aXJlbWVudAorICAgICAgICBvciBub3QgKHNwZWMgaXMgYW1iaWd1b3VzKSwgYnV0
IGZvciBub3cgd2UgbWF0Y2ggdGhlIG9sZCBwYXJzZXIuCisKKyAgICAgICAgKFdlYkNvcmU6OkNT
U1NlbGVjdG9yUGFyc2VyOjpjb25zdW1lQ29tYmluYXRvcik6CisgICAgICAgIEZpeCBhIGJ1ZyB0
aGF0IGNhdXNlZCB0aGUgZG91YmxlIGNoaWxkIGNvbWJpbmF0b3IgdG8gbWF0Y2ggZXZlbgorICAg
ICAgICB3aGVuIHRoZXJlIHdhcyB3aGl0ZXNwYWNlIGJldHdlZW4gdGhlIHR3byA+IHN5bWJvbHMu
CisKIDIwMTYtMTAtMzAgIERhcmluIEFkbGVyICA8ZGFyaW5AYXBwbGUuY29tPgogCiAgICAgICAg
IE1vdmUgRWxlbWVudCwgTmFtZWROb2RlTWFwLCBhbmQgRE9NU3RyaW5nTWFwIGZyb20gRXhjZXB0
aW9uQ29kZSB0byBFeGNlcHRpb24KSW5kZXg6IFNvdXJjZS9XZWJDb3JlL2Nzcy9wYXJzZXIvQ1NT
UHJvcGVydHlQYXJzZXIuY3BwCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFNvdXJjZS9XZWJDb3JlL2Nzcy9wYXJz
ZXIvQ1NTUHJvcGVydHlQYXJzZXIuY3BwCShyZXZpc2lvbiAyMDgxMzApCisrKyBTb3VyY2UvV2Vi
Q29yZS9jc3MvcGFyc2VyL0NTU1Byb3BlcnR5UGFyc2VyLmNwcAkod29ya2luZyBjb3B5KQpAQCAt
MzY0LDcgKzM2NCwxMSBAQCBib29sIENTU1Byb3BlcnR5UGFyc2VyOjpjb25zdW1lVHJhbnNmb3Jt
CiAgICAgUmVmUHRyPENTU1ByaW1pdGl2ZVZhbHVlPiByZXN1bHRYOwogICAgIFJlZlB0cjxDU1NQ
cmltaXRpdmVWYWx1ZT4gcmVzdWx0WTsKICAgICBpZiAoY29uc3VtZU9uZU9yVHdvVmFsdWVkUG9z
aXRpb24obV9yYW5nZSwgbV9jb250ZXh0Lm1vZGUsIFVuaXRsZXNzUXVpcms6OkZvcmJpZCwgcmVz
dWx0WCwgcmVzdWx0WSkpIHsKKyAgICAgICAgbV9yYW5nZS5jb25zdW1lV2hpdGVzcGFjZSgpOwor
ICAgICAgICBib29sIGF0RW5kID0gbV9yYW5nZS5hdEVuZCgpOwogICAgICAgICBSZWZQdHI8Q1NT
UHJpbWl0aXZlVmFsdWU+IHJlc3VsdFogPSBjb25zdW1lTGVuZ3RoKG1fcmFuZ2UsIG1fY29udGV4
dC5tb2RlLCBWYWx1ZVJhbmdlQWxsKTsKKyAgICAgICAgaWYgKCFyZXN1bHRaICYmICFhdEVuZCkK
KyAgICAgICAgICAgIHJldHVybiBmYWxzZTsKICAgICAgICAgaWYgKCFyZXN1bHRaKQogICAgICAg
ICAgICAgcmVzdWx0WiA9IENTU1ZhbHVlUG9vbDo6c2luZ2xldG9uKCkuY3JlYXRlVmFsdWUoMCwg
Q1NTUHJpbWl0aXZlVmFsdWU6OlVuaXRUeXBlczo6Q1NTX1BYKTsKICAgICAgICAgYWRkUHJvcGVy
dHkoQ1NTUHJvcGVydHlUcmFuc2Zvcm1PcmlnaW5YLCBDU1NQcm9wZXJ0eVRyYW5zZm9ybU9yaWdp
biwgcmVzdWx0WC5yZWxlYXNlTm9uTnVsbCgpLCBpbXBvcnRhbnQpOwpJbmRleDogU291cmNlL1dl
YkNvcmUvY3NzL3BhcnNlci9DU1NTZWxlY3RvclBhcnNlci5jcHAKPT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gU291
cmNlL1dlYkNvcmUvY3NzL3BhcnNlci9DU1NTZWxlY3RvclBhcnNlci5jcHAJKHJldmlzaW9uIDIw
ODEzMykKKysrIFNvdXJjZS9XZWJDb3JlL2Nzcy9wYXJzZXIvQ1NTU2VsZWN0b3JQYXJzZXIuY3Bw
CSh3b3JraW5nIGNvcHkpCkBAIC00MjAsNiArNDIwLDkgQEAgc3RkOjp1bmlxdWVfcHRyPENTU1Bh
cnNlclNlbGVjdG9yPiBDU1NTZQogewogICAgIEFTU0VSVChyYW5nZS5wZWVrKCkudHlwZSgpID09
IExlZnRCcmFja2V0VG9rZW4pOwogICAgIENTU1BhcnNlclRva2VuUmFuZ2UgYmxvY2sgPSByYW5n
ZS5jb25zdW1lQmxvY2soKTsKKyAgICBpZiAoYmxvY2suZW5kKCkgPT0gcmFuZ2UuZW5kKCkpCisg
ICAgICAgIHJldHVybiBudWxscHRyOyAvLyBObyBdIHdhcyBmb3VuZC4gQmUgc3RyaWN0IGFib3V0
IHRoaXMuCisKICAgICBibG9jay5jb25zdW1lV2hpdGVzcGFjZSgpOwogCiAgICAgQXRvbWljU3Ry
aW5nIG5hbWVzcGFjZVByZWZpeDsKQEAgLTU3Miw2ICs1NzUsOCBAQCBzdGQ6OnVuaXF1ZV9wdHI8
Q1NTUGFyc2VyU2VsZWN0b3I+IENTU1NlCiAgICAgICAgICAgICAgICAgY29uc3QgQ1NTUGFyc2Vy
VG9rZW4mIGlkZW50ID0gYmxvY2suY29uc3VtZSgpOwogICAgICAgICAgICAgICAgIGlmICghZXF1
YWxJZ25vcmluZ0FTQ0lJQ2FzZShpZGVudC52YWx1ZSgpLCAib2YiKSkKICAgICAgICAgICAgICAg
ICAgICAgcmV0dXJuIG51bGxwdHI7CisgICAgICAgICAgICAgICAgaWYgKGJsb2NrLnBlZWsoKS50
eXBlKCkgIT0gV2hpdGVzcGFjZVRva2VuKQorICAgICAgICAgICAgICAgICAgICByZXR1cm4gbnVs
bHB0cjsKICAgICAgICAgICAgICAgICBEaXNhbGxvd1BzZXVkb0VsZW1lbnRzU2NvcGUgc2NvcGUo
dGhpcyk7CiAgICAgICAgICAgICAgICAgYmxvY2suY29uc3VtZVdoaXRlc3BhY2UoKTsKICAgICAg
ICAgICAgICAgICBzdGQ6OnVuaXF1ZV9wdHI8Q1NTU2VsZWN0b3JMaXN0PiBzZWxlY3Rvckxpc3Qg
PSBzdGQ6OnVuaXF1ZV9wdHI8Q1NTU2VsZWN0b3JMaXN0PihuZXcgQ1NTU2VsZWN0b3JMaXN0KCkp
OwpAQCAtNjcxLDE2ICs2NzYsMjUgQEAgQ1NTU2VsZWN0b3I6OlJlbGF0aW9uVHlwZSBDU1NTZWxl
Y3RvclBhcgogICAgIFVDaGFyIGRlbGltaXRlciA9IHJhbmdlLnBlZWsoKS5kZWxpbWl0ZXIoKTsK
IAogICAgIGlmIChkZWxpbWl0ZXIgPT0gJysnIHx8IGRlbGltaXRlciA9PSAnficgfHwgZGVsaW1p
dGVyID09ICc+JykgewotICAgICAgICByYW5nZS5jb25zdW1lSW5jbHVkaW5nV2hpdGVzcGFjZSgp
OwotICAgICAgICBpZiAoZGVsaW1pdGVyID09ICcrJykKKyAgICAgICAgaWYgKGRlbGltaXRlciA9
PSAnKycpIHsKKyAgICAgICAgICAgIHJhbmdlLmNvbnN1bWVJbmNsdWRpbmdXaGl0ZXNwYWNlKCk7
CiAgICAgICAgICAgICByZXR1cm4gQ1NTU2VsZWN0b3I6OkRpcmVjdEFkamFjZW50OwotICAgICAg
ICBpZiAoZGVsaW1pdGVyID09ICd+JykKKyAgICAgICAgfQorICAgICAgICAKKyAgICAgICAgaWYg
KGRlbGltaXRlciA9PSAnficpIHsKKyAgICAgICAgICAgIHJhbmdlLmNvbnN1bWVJbmNsdWRpbmdX
aGl0ZXNwYWNlKCk7CiAgICAgICAgICAgICByZXR1cm4gQ1NTU2VsZWN0b3I6OkluZGlyZWN0QWRq
YWNlbnQ7CisgICAgICAgIH0KKyAgICAgICAgCiAjaWYgRU5BQkxFX0NTU19TRUxFQ1RPUlNfTEVW
RUw0Ci0gICAgICAgIGlmIChkZWxpbWl0ZXIgPT0gJz4nICYmIHJhbmdlLnBlZWsoKS50eXBlKCkg
PT0gRGVsaW1pdGVyVG9rZW4gJiYgcmFuZ2UucGVlaygpLmRlbGltaXRlcigpID09ICc+Jykgewor
ICAgICAgICByYW5nZS5jb25zdW1lKCk7CisgICAgICAgIGlmIChyYW5nZS5wZWVrKCkudHlwZSgp
ID09IERlbGltaXRlclRva2VuICYmIHJhbmdlLnBlZWsoKS5kZWxpbWl0ZXIoKSA9PSAnPicpIHsK
ICAgICAgICAgICAgIHJhbmdlLmNvbnN1bWVJbmNsdWRpbmdXaGl0ZXNwYWNlKCk7CiAgICAgICAg
ICAgICByZXR1cm4gQ1NTU2VsZWN0b3I6OkRlc2NlbmRhbnREb3VibGVDaGlsZDsKICAgICAgICAg
fQorICAgICAgICByYW5nZS5jb25zdW1lV2hpdGVzcGFjZSgpOworI2Vsc2UKKyAgICAgICAgcmFu
Z2UuY29uc3VtZUluY2x1ZGluZ1doaXRlc3BhY2UoKTsKICNlbmRpZgogICAgICAgICByZXR1cm4g
Q1NTU2VsZWN0b3I6OkNoaWxkOwogICAgIH0K
</data>
<flag name="review"
          id="316260"
          type_id="1"
          status="+"
          setter="darin"
    />
    <flag name="commit-queue"
          id="316266"
          type_id="3"
          status="-"
          setter="buildbot"
    />
          </attachment>
          <attachment
              isobsolete="0"
              ispatch="0"
              isprivate="0"
          >
            <attachid>293379</attachid>
            <date>2016-10-30 18:18:22 -0700</date>
            <delta_ts>2016-10-30 18:18:22 -0700</delta_ts>
            <desc>Archive of layout-test-results from ews125 for ios-simulator-wk2</desc>
            <filename>layout-test-results.zip</filename>
            <type>application/zip</type>
            <size>0</size>
            <attacher name="Build Bot">buildbot</attacher>
            
              <data encoding="base64"></data>

          </attachment>
      

    </bug>

</bugzilla>