<?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>158980</bug_id>
          
          <creation_ts>2016-06-21 02:31:12 -0700</creation_ts>
          <short_desc>[GStreamer] top/bottom black bars added needlessly in fullscreen</short_desc>
          <delta_ts>2016-06-27 02:22:38 -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>Platform</component>
          <version>Other</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="Philippe Normand">pnormand</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>cgarcia</cc>
    
    <cc>pnormand</cc>
    
    <cc>yoon</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1204091</commentid>
    <comment_count>0</comment_count>
    <who name="Philippe Normand">pnormand</who>
    <bug_when>2016-06-21 02:31:12 -0700</bug_when>
    <thetext>When playing a 1280x720 video fullscreen on a monitor with the same resolution we should see the video fullscreen without top/bottom black bars.

It seems this regression was added in r201076.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1204092</commentid>
    <comment_count>1</comment_count>
      <attachid>281725</attachid>
    <who name="Philippe Normand">pnormand</who>
    <bug_when>2016-06-21 02:34:28 -0700</bug_when>
    <thetext>Created attachment 281725
[GStreamer] Trigger video resize upon first sample rendering

The natural video size calculation depends on the validity of the
current sample, so whenever the first sample reached the sink it&apos;s a
good idea to reflect this on the player which will update its natural
size accordingly.

Fixes an issue where black borders were added on top and bottom of
fullscreen video.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1205499</commentid>
    <comment_count>2</comment_count>
    <who name="Philippe Normand">pnormand</who>
    <bug_when>2016-06-27 02:22:38 -0700</bug_when>
    <thetext>Committed r202482: &lt;http://trac.webkit.org/changeset/202482&gt;</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>281725</attachid>
            <date>2016-06-21 02:34:28 -0700</date>
            <delta_ts>2016-06-27 02:15:03 -0700</delta_ts>
            <desc>[GStreamer] Trigger video resize upon first sample rendering</desc>
            <filename>GStreamer-Trigger-video-resize-upon-first-sample-r.patch</filename>
            <type>text/plain</type>
            <size>2687</size>
            <attacher name="Philippe Normand">pnormand</attacher>
            
              <data encoding="base64">RnJvbSBhYzI3YmQ1MTQ1NThiMzhkYzBjMjAyNGQxZDg2Y2M2MmE5YWZjZThjIE1vbiBTZXAgMTcg
MDA6MDA6MDAgMjAwMQpGcm9tOiBQaGlsaXBwZSBOb3JtYW5kIDxwaGlsbkBpZ2FsaWEuY29tPgpE
YXRlOiBUdWUsIDIxIEp1biAyMDE2IDExOjIyOjI3ICswMjAwClN1YmplY3Q6IFtQQVRDSF0gW0dT
dHJlYW1lcl0gVHJpZ2dlciB2aWRlbyByZXNpemUgdXBvbiBmaXJzdCBzYW1wbGUgcmVuZGVyaW5n
CgpUaGUgbmF0dXJhbCB2aWRlbyBzaXplIGNhbGN1bGF0aW9uIGRlcGVuZHMgb24gdGhlIHZhbGlk
aXR5IG9mIHRoZQpjdXJyZW50IHNhbXBsZSwgc28gd2hlbmV2ZXIgdGhlIGZpcnN0IHNhbXBsZSBy
ZWFjaGVkIHRoZSBzaW5rIGl0J3MgYQpnb29kIGlkZWEgdG8gcmVmbGVjdCB0aGlzIG9uIHRoZSBw
bGF5ZXIgd2hpY2ggd2lsbCB1cGRhdGUgaXRzIG5hdHVyYWwKc2l6ZSBhY2NvcmRpbmdseS4KCkZp
eGVzIGFuIGlzc3VlIHdoZXJlIGJsYWNrIGJvcmRlcnMgd2VyZSBhZGRlZCBvbiB0b3AgYW5kIGJv
dHRvbSBvZgpmdWxsc2NyZWVuIHZpZGVvLgotLS0KIFNvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZyAg
ICAgICAgICAgICAgICAgICAgICAgICAgICAgICB8IDE4ICsrKysrKysrKysrKysrKysrKwogLi4u
L2dzdHJlYW1lci9NZWRpYVBsYXllclByaXZhdGVHU3RyZWFtZXJCYXNlLmNwcCAgICAgIHwgIDcg
KysrKysrKwogMiBmaWxlcyBjaGFuZ2VkLCAyNSBpbnNlcnRpb25zKCspCgpkaWZmIC0tZ2l0IGEv
U291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nIGIvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCmluZGV4
IDgyZTlmMWYuLmIzZGQwYTMgMTAwNjQ0Ci0tLSBhL1NvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZwor
KysgYi9Tb3VyY2UvV2ViQ29yZS9DaGFuZ2VMb2cKQEAgLTEsMyArMSwyMSBAQAorMjAxNi0wNi0y
MSAgUGhpbGlwcGUgTm9ybWFuZCAgPHBoaWxuQGlnYWxpYS5jb20+CisKKyAgICAgICAgW0dTdHJl
YW1lcl0gdG9wL2JvdHRvbSBibGFjayBiYXJzIGFkZGVkIG5lZWRsZXNzbHkgaW4gZnVsbHNjcmVl
bgorICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MTU4OTgw
CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgVGhlIG5h
dHVyYWwgdmlkZW8gc2l6ZSBjYWxjdWxhdGlvbiBkZXBlbmRzIG9uIHRoZSB2YWxpZGl0eSBvZiB0
aGUKKyAgICAgICAgY3VycmVudCBzYW1wbGUsIHNvIHdoZW5ldmVyIHRoZSBmaXJzdCBzYW1wbGUg
cmVhY2hlZCB0aGUgc2luayBpdCdzIGEKKyAgICAgICAgZ29vZCBpZGVhIHRvIHJlZmxlY3QgdGhp
cyBvbiB0aGUgcGxheWVyIHdoaWNoIHdpbGwgdXBkYXRlIGl0cyBuYXR1cmFsCisgICAgICAgIHNp
emUgYWNjb3JkaW5nbHkuCisKKyAgICAgICAgRml4ZXMgYW4gaXNzdWUgd2hlcmUgYmxhY2sgYm9y
ZGVycyB3ZXJlIGFkZGVkIG9uIHRvcCBhbmQgYm90dG9tIG9mCisgICAgICAgIGZ1bGxzY3JlZW4g
dmlkZW8uCisKKyAgICAgICAgKiBwbGF0Zm9ybS9ncmFwaGljcy9nc3RyZWFtZXIvTWVkaWFQbGF5
ZXJQcml2YXRlR1N0cmVhbWVyQmFzZS5jcHA6CisgICAgICAgIChXZWJDb3JlOjpNZWRpYVBsYXll
clByaXZhdGVHU3RyZWFtZXJCYXNlOjp0cmlnZ2VyUmVwYWludCk6CisKIDIwMTYtMDYtMjEgIE1p
Z3VlbCBHb21leiAgPG1hZ29tZXpAaWdhbGlhLmNvbT4KIAogICAgICAgICBbR1N0cmVhbWVyXSB2
aWRlbyBvcmllbnRhdGlvbiBzdXBwb3J0CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9wbGF0
Zm9ybS9ncmFwaGljcy9nc3RyZWFtZXIvTWVkaWFQbGF5ZXJQcml2YXRlR1N0cmVhbWVyQmFzZS5j
cHAgYi9Tb3VyY2UvV2ViQ29yZS9wbGF0Zm9ybS9ncmFwaGljcy9nc3RyZWFtZXIvTWVkaWFQbGF5
ZXJQcml2YXRlR1N0cmVhbWVyQmFzZS5jcHAKaW5kZXggMGRkNTY1Ni4uNjQ5NGQwZSAxMDA2NDQK
LS0tIGEvU291cmNlL1dlYkNvcmUvcGxhdGZvcm0vZ3JhcGhpY3MvZ3N0cmVhbWVyL01lZGlhUGxh
eWVyUHJpdmF0ZUdTdHJlYW1lckJhc2UuY3BwCisrKyBiL1NvdXJjZS9XZWJDb3JlL3BsYXRmb3Jt
L2dyYXBoaWNzL2dzdHJlYW1lci9NZWRpYVBsYXllclByaXZhdGVHU3RyZWFtZXJCYXNlLmNwcApA
QCAtNTM0LDExICs1MzQsMTggQEAgdm9pZCBNZWRpYVBsYXllclByaXZhdGVHU3RyZWFtZXJCYXNl
OjpyZXBhaW50KCkKIAogdm9pZCBNZWRpYVBsYXllclByaXZhdGVHU3RyZWFtZXJCYXNlOjp0cmln
Z2VyUmVwYWludChHc3RTYW1wbGUqIHNhbXBsZSkKIHsKKyAgICBib29sIHRyaWdnZXJSZXNpemU7
CiAgICAgewogICAgICAgICBXVEY6OkdNdXRleExvY2tlcjxHTXV0ZXg+IGxvY2sobV9zYW1wbGVN
dXRleCk7CisgICAgICAgIHRyaWdnZXJSZXNpemUgPSAhbV9zYW1wbGU7CiAgICAgICAgIG1fc2Ft
cGxlID0gc2FtcGxlOwogICAgIH0KIAorICAgIGlmICh0cmlnZ2VyUmVzaXplKSB7CisgICAgICAg
IExPR19NRURJQV9NRVNTQUdFKCJGaXJzdCBzYW1wbGUgcmVhY2hlZCB0aGUgc2luaywgdHJpZ2dl
cmluZyB2aWRlbyBkaW1lbnNpb25zIHVwZGF0ZSIpOworICAgICAgICBtX3BsYXllci0+c2l6ZUNo
YW5nZWQoKTsKKyAgICB9CisKICNpZiBVU0UoQ09PUkRJTkFURURfR1JBUEhJQ1NfVEhSRUFERUQp
CiAjaWYgVVNFKEdTVFJFQU1FUl9HTCkKICAgICBwdXNoVGV4dHVyZVRvQ29tcG9zaXRvcigpOwot
LSAKMi44LjE=
</data>
<flag name="review"
          id="305548"
          type_id="1"
          status="+"
          setter="cgarcia"
    />
          </attachment>
      

    </bug>

</bugzilla>