<?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>125822</bug_id>
          
          <creation_ts>2013-12-16 18:35:57 -0800</creation_ts>
          <short_desc>Video element&apos;s width and height content attributes should not influence intrinsic width and height</short_desc>
          <delta_ts>2013-12-17 16:13:12 -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>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>BlinkMergeCandidate</keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          <blocked>125838</blocked>
          <everconfirmed>1</everconfirmed>
          <reporter name="Ryosuke Niwa">rniwa</reporter>
          <assigned_to name="Ryosuke Niwa">rniwa</assigned_to>
          <cc>commit-queue</cc>
    
    <cc>eric.carlson</cc>
    
    <cc>esprehn+autocc</cc>
    
    <cc>glenn</cc>
    
    <cc>jer.noble</cc>
    
    <cc>kondapallykalyan</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>960348</commentid>
    <comment_count>0</comment_count>
    <who name="Ryosuke Niwa">rniwa</who>
    <bug_when>2013-12-16 18:35:57 -0800</bug_when>
    <thetext>Video element&apos;s width and height content attributes should not influence intrinsic width and height</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>960351</commentid>
    <comment_count>1</comment_count>
      <attachid>219380</attachid>
    <who name="Ryosuke Niwa">rniwa</who>
    <bug_when>2013-12-16 18:40:51 -0800</bug_when>
    <thetext>Created attachment 219380
Fixes the bug</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>960539</commentid>
    <comment_count>2</comment_count>
    <who name="Eric Carlson">eric.carlson</who>
    <bug_when>2013-12-17 09:23:36 -0800</bug_when>
    <thetext>Note that the following comment in the test is incorrect:

    These tests assume that the default intrinsic width is 300x150, so that the default
    intrinsic ratio is 2:1. This is no longer per spec, but is what is implemented.

The intrinsic width and height of a &lt;video&gt; element is 300x150 if poster and media file dimensions are unavailable: http://www.w3.org/TR/2013/WD-html51-20130528/embedded-content-0.html#concept-video-intrinsic-width</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>960603</commentid>
    <comment_count>3</comment_count>
      <attachid>219380</attachid>
    <who name="Ryosuke Niwa">rniwa</who>
    <bug_when>2013-12-17 11:43:49 -0800</bug_when>
    <thetext>Comment on attachment 219380
Fixes the bug

Oops, let me correct that comment.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>960707</commentid>
    <comment_count>4</comment_count>
    <who name="Ryosuke Niwa">rniwa</who>
    <bug_when>2013-12-17 16:13:12 -0800</bug_when>
    <thetext>Committed r160734: &lt;http://trac.webkit.org/changeset/160734&gt;</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>219380</attachid>
            <date>2013-12-16 18:40:51 -0800</date>
            <delta_ts>2013-12-17 11:43:49 -0800</delta_ts>
            <desc>Fixes the bug</desc>
            <filename>bug-125822-20131216184050.patch</filename>
            <type>text/plain</type>
            <size>5807</size>
            <attacher name="Ryosuke Niwa">rniwa</attacher>
            
              <data encoding="base64">SW5kZXg6IFNvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2UvV2Vi
Q29yZS9DaGFuZ2VMb2cJKHJldmlzaW9uIDE2MDY4NykKKysrIFNvdXJjZS9XZWJDb3JlL0NoYW5n
ZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwzICsxLDI0IEBACisyMDEzLTEyLTE2ICBSeW9zdWtl
IE5pd2EgIDxybml3YUB3ZWJraXQub3JnPgorCisgICAgICAgIFZpZGVvIGVsZW1lbnQncyB3aWR0
aCBhbmQgaGVpZ2h0IGNvbnRlbnQgYXR0cmlidXRlcyBzaG91bGQgbm90IGluZmx1ZW5jZSBpbnRy
aW5zaWMgd2lkdGggYW5kIGhlaWdodAorICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9z
aG93X2J1Zy5jZ2k/aWQ9MTI1ODIyCisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BT
ISkuCisKKyAgICAgICAgTWVyZ2UgaHR0cHM6Ly9jaHJvbWl1bS5nb29nbGVzb3VyY2UuY29tL2No
cm9taXVtL2JsaW5rLysvMDIyY2UzNGVmYjViNzBjYjk2NGMzY2EyOWYyM2M4OTgwZmZhZWYwNQor
CisgICAgICAgIFRoZSB3aWR0aC9oZWlnaHQgY29udGVudCBhdHRyaWJ1dGVzIGFscmVhZHkgaW5m
bHVlbmNlIHNwZWNpZmllZCBzdHlsZSB2aWEKKyAgICAgICAgSFRNTFZpZGVvRWxlbWVudDo6Y29s
bGVjdFN0eWxlRm9yUHJlc2VudGF0aW9uQXR0cmlidXRlLCB0byBhbHNvIGluZmx1ZW5jZSB0aGUg
aW50cmluc2ljIHNpemUgaGFzIG5ldmVyCisgICAgICAgIGJlZW4gcGFydCBvZiB0aGUgc3BlYzog
aHR0cDovL3d3dy53My5vcmcvVFIvMjAxMy9XRC1odG1sNTEtMjAxMzA1MjgvZW1iZWRkZWQtY29u
dGVudC0wLmh0bWwjZG9tLWRpbS13aWR0aAorCisgICAgICAgIFRoZSB0ZXN0IGNhc2UgcGFzc2Vz
IGluIEZpcmVmb3ggTmlnaHRseSBhbmQgSUUxMSBSZWxlYXNlIFByZXZpZXcsIGJ1dCBmYWlscyBp
biBPcGVyYSBQcmVzdG8sIHdoaWNoIGhhcworICAgICAgICBubyBkZWZhdWx0IGludHJpbnNpYyBz
aXplLgorCisgICAgICAgIFRlc3Q6IG1lZGlhL3ZpZGVvLWludHJpbnNpYy13aWR0aC1oZWlnaHQu
aHRtbAorCisgICAgICAgICogcmVuZGVyaW5nL1JlbmRlclZpZGVvLmNwcDoKKyAgICAgICAgKFdl
YkNvcmU6OlJlbmRlclZpZGVvOjpjYWxjdWxhdGVJbnRyaW5zaWNTaXplKToKKwogMjAxMy0xMi0x
NiAgQW5kZXJzIENhcmxzc29uICA8YW5kZXJzY2FAYXBwbGUuY29tPgogCiAgICAgICAgIFJlbW92
ZSBFbnVtQ2xhc3MuaCBmcm9tIFdURgpJbmRleDogU291cmNlL1dlYkNvcmUvcmVuZGVyaW5nL1Jl
bmRlclZpZGVvLmNwcAo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2UvV2ViQ29yZS9yZW5kZXJpbmcvUmVu
ZGVyVmlkZW8uY3BwCShyZXZpc2lvbiAxNjA2MjQpCisrKyBTb3VyY2UvV2ViQ29yZS9yZW5kZXJp
bmcvUmVuZGVyVmlkZW8uY3BwCSh3b3JraW5nIGNvcHkpCkBAIC0xMTYsMTQgKzExNiw2IEBAIExh
eW91dFNpemUgUmVuZGVyVmlkZW86OmNhbGN1bGF0ZUludHJpbnMKICAgICBpZiAodmlkZW9FbGVt
ZW50KCkuc2hvdWxkRGlzcGxheVBvc3RlckltYWdlKCkgJiYgIW1fY2FjaGVkSW1hZ2VTaXplLmlz
RW1wdHkoKSAmJiAhaW1hZ2VSZXNvdXJjZSgpLT5lcnJvck9jY3VycmVkKCkpCiAgICAgICAgIHJl
dHVybiBtX2NhY2hlZEltYWdlU2l6ZTsKIAotI2lmICFQTEFURk9STShJT1MpCi0gICAgLy8gV2hl
biB0aGUgbmF0dXJhbCBzaXplIG9mIHRoZSB2aWRlbyBpcyB1bmF2YWlsYWJsZSwgd2UgdXNlIHRo
ZSBwcm92aWRlZAotICAgIC8vIHdpZHRoIGFuZCBoZWlnaHQgYXR0cmlidXRlcyBvZiB0aGUgdmlk
ZW8gZWxlbWVudCBhcyB0aGUgaW50cmluc2ljIHNpemUgdW50aWwKLSAgICAvLyBiZXR0ZXIgdmFs
dWVzIGJlY29tZSBhdmFpbGFibGUuCi0gICAgaWYgKHZpZGVvRWxlbWVudCgpLmhhc0F0dHJpYnV0
ZSh3aWR0aEF0dHIpICYmIHZpZGVvRWxlbWVudCgpLmhhc0F0dHJpYnV0ZShoZWlnaHRBdHRyKSkK
LSAgICAgICAgcmV0dXJuIExheW91dFNpemUodmlkZW9FbGVtZW50KCkud2lkdGgoKSwgdmlkZW9F
bGVtZW50KCkuaGVpZ2h0KCkpOwotI2VuZGlmCi0KICAgICAvLyA8dmlkZW8+IGluIHN0YW5kYWxv
bmUgbWVkaWEgZG9jdW1lbnRzIHNob3VsZCBub3QgdXNlIHRoZSBkZWZhdWx0IDMwMHgxNTAKICAg
ICAvLyBzaXplIHNpbmNlIHRoZXkgYWxzbyBoYXZlIGF1ZGlvLW9ubHkgZmlsZXMuIEJ5IHNldHRp
bmcgdGhlIGludHJpbnNpYwogICAgIC8vIHNpemUgdG8gMzAweDEgdGhlIHZpZGVvIHdpbGwgcmVz
aXplIGl0c2VsZiBpbiB0aGVzZSBjYXNlcywgYW5kIGF1ZGlvIHdpbGwKSW5kZXg6IExheW91dFRl
c3RzL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBMYXlvdXRUZXN0cy9DaGFuZ2VMb2cJKHJldmlz
aW9uIDE2MDY4NykKKysrIExheW91dFRlc3RzL0NoYW5nZUxvZwkod29ya2luZyBjb3B5KQpAQCAt
MSwzICsxLDEzIEBACisyMDEzLTEyLTE2ICBSeW9zdWtlIE5pd2EgIDxybml3YUB3ZWJraXQub3Jn
PgorCisgICAgICAgIFZpZGVvIGVsZW1lbnQncyB3aWR0aCBhbmQgaGVpZ2h0IGNvbnRlbnQgYXR0
cmlidXRlcyBzaG91bGQgbm90IGluZmx1ZW5jZSBpbnRyaW5zaWMgd2lkdGggYW5kIGhlaWdodAor
ICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MTI1ODIyCisK
KyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgKiBtZWRpYS92
aWRlby1pbnRyaW5zaWMtd2lkdGgtaGVpZ2h0LWV4cGVjdGVkLnR4dDogQWRkZWQuCisgICAgICAg
ICogbWVkaWEvdmlkZW8taW50cmluc2ljLXdpZHRoLWhlaWdodC5odG1sOiBBZGRlZC4KKwogMjAx
My0xMi0xNiAgT2xpdmVyIEh1bnQgIDxvbGl2ZXJAYXBwbGUuY29tPgogCiAgICAgICAgIENhY2hl
IGdldHRlcnMgYW5kIGN1c3RvbSBhY2Nlc3NvcnMgb24gdGhlIHByb3RvdHlwZSBjaGFpbgpJbmRl
eDogTGF5b3V0VGVzdHMvbWVkaWEvdmlkZW8taW50cmluc2ljLXdpZHRoLWhlaWdodC1leHBlY3Rl
ZC50eHQKPT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PQotLS0gTGF5b3V0VGVzdHMvbWVkaWEvdmlkZW8taW50cmluc2ljLXdp
ZHRoLWhlaWdodC1leHBlY3RlZC50eHQJKHJldmlzaW9uIDApCisrKyBMYXlvdXRUZXN0cy9tZWRp
YS92aWRlby1pbnRyaW5zaWMtd2lkdGgtaGVpZ2h0LWV4cGVjdGVkLnR4dAkod29ya2luZyBjb3B5
KQpAQCAtMCwwICsxLDggQEAKKyAgICAgICAKKworUEFTUyBubyB3aWR0aC9oZWlnaHQgYXR0cmli
dXRlcyAKK1BBU1Mgb25seSB3aWR0aCBhdHRyaWJ1dGUgCitQQVNTIG9ubHkgaGVpZ2h0IGF0dHJp
YnV0ZSAKK1BBU1MgYm90aCB3aWR0aC9oZWlnaHQgYXR0cmlidXRlcyAKK1BBU1MgYm90aCB3aWR0
aC9oZWlnaHQgYXR0cmlidXRlcyBhbmQgc3R5bGUgCisKSW5kZXg6IExheW91dFRlc3RzL21lZGlh
L3ZpZGVvLWludHJpbnNpYy13aWR0aC1oZWlnaHQuaHRtbAo9PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBMYXlvdXRU
ZXN0cy9tZWRpYS92aWRlby1pbnRyaW5zaWMtd2lkdGgtaGVpZ2h0Lmh0bWwJKHJldmlzaW9uIDAp
CisrKyBMYXlvdXRUZXN0cy9tZWRpYS92aWRlby1pbnRyaW5zaWMtd2lkdGgtaGVpZ2h0Lmh0bWwJ
KHdvcmtpbmcgY29weSkKQEAgLTAsMCArMSwzOSBAQAorPCFET0NUWVBFIGh0bWw+Cis8aHRtbD4K
KyAgICA8aGVhZD4KKyAgICAgICAgPHRpdGxlPnZpZGVvIGVsZW1lbnQgaW50cmluc2ljIHdpZHRo
L2hlaWdodDwvdGl0bGU+CisgICAgICAgIDxzY3JpcHQgc3JjPSIuLi9yZXNvdXJjZXMvdGVzdGhh
cm5lc3MuanMiPjwvc2NyaXB0PgorICAgICAgICA8c2NyaXB0IHNyYz0iLi4vcmVzb3VyY2VzL3Rl
c3RoYXJuZXNzcmVwb3J0LmpzIj48L3NjcmlwdD4KKyAgICA8L2hlYWQ+CisgICAgPGJvZHk+Cisg
ICAgICAgIDxkaXYgaWQ9ImxvZyI+PC9kaXY+CisgICAgICAgIDwhLS0gVGhlc2UgdGVzdHMgYXNz
dW1lIHRoYXQgdGhlIGRlZmF1bHQgaW50cmluc2ljIHdpZHRoIGlzIDMwMHgxNTAsIHNvIHRoYXQg
dGhlIGRlZmF1bHQKKyAgICAgICAgICAgICBpbnRyaW5zaWMgcmF0aW8gaXMgMjoxLiBUaGlzIGlz
IG5vIGxvbmdlciBwZXIgc3BlYywgYnV0IGlzIHdoYXQgaXMgaW1wbGVtZW50ZWQuIC0tPgorICAg
ICAgICA8dmlkZW8gdGl0bGU9Im5vIHdpZHRoL2hlaWdodCBhdHRyaWJ1dGVzIgorICAgICAgICAg
ICAgICAgZGF0YS1leHBlY3RlZC13aWR0aD0iMzAwIiBkYXRhLWV4cGVjdGVkLWhlaWdodD0iMTUw
Ij48L3ZpZGVvPgorICAgICAgICA8dmlkZW8gdGl0bGU9Im9ubHkgd2lkdGggYXR0cmlidXRlIgor
ICAgICAgICAgICAgICAgZGF0YS1leHBlY3RlZC13aWR0aD0iMTAwIiBkYXRhLWV4cGVjdGVkLWhl
aWdodD0iNTAiCisgICAgICAgICAgICAgICB3aWR0aD0iMTAwIj48L3ZpZGVvPgorICAgICAgICA8
dmlkZW8gdGl0bGU9Im9ubHkgaGVpZ2h0IGF0dHJpYnV0ZSIKKyAgICAgICAgICAgICAgIGRhdGEt
ZXhwZWN0ZWQtd2lkdGg9IjIwMCIgZGF0YS1leHBlY3RlZC1oZWlnaHQ9IjEwMCIKKyAgICAgICAg
ICAgICAgIGhlaWdodD0iMTAwIj48L3ZpZGVvPgorICAgICAgICA8dmlkZW8gdGl0bGU9ImJvdGgg
d2lkdGgvaGVpZ2h0IGF0dHJpYnV0ZXMiCisgICAgICAgICAgICAgICBkYXRhLWV4cGVjdGVkLXdp
ZHRoPSIxMDAiIGRhdGEtZXhwZWN0ZWQtaGVpZ2h0PSIxMDAiCisgICAgICAgICAgICAgICB3aWR0
aD0iMTAwIiBoZWlnaHQ9IjEwMCI+PC92aWRlbz4KKyAgICAgICAgPCEtLSBBIHdpZHRoOmhlaWdo
dCByYXRpbyBvdGhlciB0aGFuIDI6MSBhbmQgb3ZlcnJpZGluZyB0aGUgc3BlY2lmaWVkIHN0eWxl
IG11c3QgYmUgdXNlZCB0bworICAgICAgICAgICAgIHZlcmlmeSB0aGF0IHdpZHRoL2hlaWdodCBk
b2VzIG5vdCBpbmZsdWVuY2UgaW50cmluc2ljIHJhdGlvIC0tPgorICAgICAgICA8dmlkZW8gdGl0
bGU9ImJvdGggd2lkdGgvaGVpZ2h0IGF0dHJpYnV0ZXMgYW5kIHN0eWxlIgorICAgICAgICAgICAg
ICAgZGF0YS1leHBlY3RlZC13aWR0aD0iMzAwIiBkYXRhLWV4cGVjdGVkLWhlaWdodD0iMTUwIgor
ICAgICAgICAgICAgICAgd2lkdGg9IjEwMCIgaGVpZ2h0PSIxMDAiIHN0eWxlPSJ3aWR0aDogYXV0
bzsgaGVpZ2h0OiBhdXRvIj48L3ZpZGVvPgorICAgICAgICA8c2NyaXB0PgorICAgICAgICAgICAg
QXJyYXkucHJvdG90eXBlLmZvckVhY2guY2FsbChkb2N1bWVudC5xdWVyeVNlbGVjdG9yQWxsKCd2
aWRlbycpLCBmdW5jdGlvbih2aWRlbykKKyAgICAgICAgICAgIHsKKyAgICAgICAgICAgICAgICB0
ZXN0KGZ1bmN0aW9uKCkKKyAgICAgICAgICAgICAgICB7CisgICAgICAgICAgICAgICAgICAgIGFz
c2VydF9lcXVhbHModmlkZW8uY2xpZW50V2lkdGgsIHBhcnNlSW50KHZpZGVvLmRhdGFzZXQuZXhw
ZWN0ZWRXaWR0aCksICJ3aWR0aCIpOworICAgICAgICAgICAgICAgICAgICBhc3NlcnRfZXF1YWxz
KHZpZGVvLmNsaWVudEhlaWdodCwgcGFyc2VJbnQodmlkZW8uZGF0YXNldC5leHBlY3RlZEhlaWdo
dCksICJoZWlnaHQiKTsKKyAgICAgICAgICAgICAgICB9LCB2aWRlby50aXRsZSk7CisgICAgICAg
ICAgICB9KTsKKyAgICAgICAgPC9zY3JpcHQ+CisgICAgPC9ib2R5PgorPC9odG1sPgo=
</data>
<flag name="review"
          id="242997"
          type_id="1"
          status="+"
          setter="darin"
    />
    <flag name="commit-queue"
          id="243063"
          type_id="3"
          status="-"
          setter="rniwa"
    />
          </attachment>
      

    </bug>

</bugzilla>