<?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>174020</bug_id>
          
          <creation_ts>2017-06-30 02:10:12 -0700</creation_ts>
          <short_desc>[EME] Solve a couple of compiler warnings</short_desc>
          <delta_ts>2017-07-04 04:12:31 -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="Xabier Rodríguez Calvar">calvaris</reporter>
          <assigned_to name="Xabier Rodríguez Calvar">calvaris</assigned_to>
          <cc>commit-queue</cc>
    
    <cc>mcatanzaro</cc>
    
    <cc>zan</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1324358</commentid>
    <comment_count>0</comment_count>
    <who name="Xabier Rodríguez Calvar">calvaris</who>
    <bug_when>2017-06-30 02:10:12 -0700</bug_when>
    <thetext>[EME] Solve a couple of compiler warnings</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1324360</commentid>
    <comment_count>1</comment_count>
      <attachid>314253</attachid>
    <who name="Xabier Rodríguez Calvar">calvaris</who>
    <bug_when>2017-06-30 02:13:03 -0700</bug_when>
    <thetext>Created attachment 314253
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1324934</commentid>
    <comment_count>2</comment_count>
      <attachid>314253</attachid>
    <who name="Michael Catanzaro">mcatanzaro</who>
    <bug_when>2017-07-02 09:23:09 -0700</bug_when>
    <thetext>Comment on attachment 314253
Patch

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

&gt; Source/WebCore/Modules/encryptedmedia/MediaKeySession.cpp:636
&gt;              break;

Remove this unreachable break</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1325091</commentid>
    <comment_count>3</comment_count>
      <attachid>314473</attachid>
    <who name="Xabier Rodríguez Calvar">calvaris</who>
    <bug_when>2017-07-03 04:48:42 -0700</bug_when>
    <thetext>Created attachment 314473
Patch for landing</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1325104</commentid>
    <comment_count>4</comment_count>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2017-07-03 05:36:03 -0700</bug_when>
    <thetext>The commit-queue encountered the following flaky tests while processing attachment 314473:

workers/bomb.html bug 171985 (author: fpizlo@apple.com)
The commit-queue is continuing to process your patch.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1325105</commentid>
    <comment_count>5</comment_count>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2017-07-03 05:36:04 -0700</bug_when>
    <thetext>The commit-queue encountered the following flaky tests while processing attachment 314473:

The commit-queue is continuing to process your patch.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1325118</commentid>
    <comment_count>6</comment_count>
    <who name="Xabier Rodríguez Calvar">calvaris</who>
    <bug_when>2017-07-03 07:33:49 -0700</bug_when>
    <thetext>(In reply to WebKit Commit Bot from comment #5)
&gt; The commit-queue is continuing to process your patch.

The errors seems unrelated, I&apos;ll try to land tomorrow.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1325407</commentid>
    <comment_count>7</comment_count>
      <attachid>314473</attachid>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2017-07-04 04:12:30 -0700</bug_when>
    <thetext>Comment on attachment 314473
Patch for landing

Clearing flags on attachment: 314473

Committed r219116: &lt;http://trac.webkit.org/changeset/219116&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1325408</commentid>
    <comment_count>8</comment_count>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2017-07-04 04:12:31 -0700</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>314253</attachid>
            <date>2017-06-30 02:13:03 -0700</date>
            <delta_ts>2017-07-03 04:48:37 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-174020-20170630111301.patch</filename>
            <type>text/plain</type>
            <size>2450</size>
            <attacher name="Xabier Rodríguez Calvar">calvaris</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMjE4ODc0CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9D
aGFuZ2VMb2cgYi9Tb3VyY2UvV2ViQ29yZS9DaGFuZ2VMb2cKaW5kZXggZThkY2U3MmFmNTFiMzRk
Njk4ZmQ1MTk4ZTU4YzczNjQzMzc0NWRjYi4uYjhlYjgxM2Y3ZmFlMjI1Yzk1NGJhYmYzZGU3NDE0
MzRkNjg5OTI3ZiAxMDA2NDQKLS0tIGEvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCisrKyBiL1Nv
dXJjZS9XZWJDb3JlL0NoYW5nZUxvZwpAQCAtMSwzICsxLDE4IEBACisyMDE3LTA2LTMwICBYYWJp
ZXIgUm9kcmlndWV6IENhbHZhciAgPGNhbHZhcmlzQGlnYWxpYS5jb20+CisKKyAgICAgICAgW0VN
RV0gU29sdmUgYSBjb3VwbGUgb2YgY29tcGlsZXIgd2FybmluZ3MKKyAgICAgICAgaHR0cHM6Ly9i
dWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTE3NDAyMAorCisgICAgICAgIFJldmlld2Vk
IGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgICogTW9kdWxlcy9lbmNyeXB0ZWRtZWRpYS9D
RE0uY3BwOgorICAgICAgICAoV2ViQ29yZTo6Q0RNOjppc1BlcnNpc3RlbnRUeXBlKTogQWRkZWQg
ZGVmYXVsdCByZXR1cm4gYW5kCisgICAgICAgIGFzc2VydGlvbi4KKyAgICAgICAgKiBNb2R1bGVz
L2VuY3J5cHRlZG1lZGlhL01lZGlhS2V5U2Vzc2lvbi5jcHA6CisgICAgICAgIChXZWJDb3JlOjpN
ZWRpYUtleVNlc3Npb246OnVwZGF0ZUtleVN0YXR1c2VzKTogVGhpcyB3YXJuaW5nIHdhcworICAg
ICAgICBhbHJlYWR5IHNvbHZlZCBidXQgSSB0aGluayBhZGRpbmcgYW4gYXNzZXJ0aW9uIGZvciB0
aGUgZGVmYXVsdAorICAgICAgICBjYXNlIGNhbiBoZWxwIGNhdGNoIGVycm9ycyBpbiB0aGUgZnV0
dXJlLgorCiAyMDE3LTA2LTI4ICBDYXJsb3MgR2FyY2lhIENhbXBvcyAgPGNnYXJjaWFAaWdhbGlh
LmNvbT4KIAogICAgICAgICBSRUdSRVNTSU9OKHIyMTg3OTkpOiBbR1RLXVtXUEVdIENyaXRpY2Fs
IHdhcm5pbmcgYXQgZXhpdApkaWZmIC0tZ2l0IGEvU291cmNlL1dlYkNvcmUvTW9kdWxlcy9lbmNy
eXB0ZWRtZWRpYS9DRE0uY3BwIGIvU291cmNlL1dlYkNvcmUvTW9kdWxlcy9lbmNyeXB0ZWRtZWRp
YS9DRE0uY3BwCmluZGV4IGFlM2ZlMDZkMjJhNDEwMmE4M2EwYTdjNTlmZGM3MjU2NGVhODAyNGMu
LmQzZDA5NjY0MmFmZDc3NzQ5YThhMzIyOWZjOTJkNmMxOWYzYjk3NjMgMTAwNjQ0Ci0tLSBhL1Nv
dXJjZS9XZWJDb3JlL01vZHVsZXMvZW5jcnlwdGVkbWVkaWEvQ0RNLmNwcAorKysgYi9Tb3VyY2Uv
V2ViQ29yZS9Nb2R1bGVzL2VuY3J5cHRlZG1lZGlhL0NETS5jcHAKQEAgLTE4MSw2ICsxODEsOSBA
QCBib29sIENETTo6aXNQZXJzaXN0ZW50VHlwZShNZWRpYUtleVNlc3Npb25UeXBlIHNlc3Npb25U
eXBlKQogICAgICAgICAvLyDihrMgInBlcnNpc3RlbnQtbGljZW5zZSIKICAgICAgICAgcmV0dXJu
IHRydWU7CiAgICAgfQorCisgICAgQVNTRVJUX05PVF9SRUFDSEVEKCk7CisgICAgcmV0dXJuIGZh
bHNlOwogfQogCiBzdGQ6Om9wdGlvbmFsPE1lZGlhS2V5U3lzdGVtQ29uZmlndXJhdGlvbj4gQ0RN
OjpnZXRTdXBwb3J0ZWRDb25maWd1cmF0aW9uKGNvbnN0IE1lZGlhS2V5U3lzdGVtQ29uZmlndXJh
dGlvbiYgY2FuZGlkYXRlQ29uZmlndXJhdGlvbiwgTWVkaWFLZXlzUmVzdHJpY3Rpb25zJiByZXN0
cmljdGlvbnMpCmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9Nb2R1bGVzL2VuY3J5cHRlZG1l
ZGlhL01lZGlhS2V5U2Vzc2lvbi5jcHAgYi9Tb3VyY2UvV2ViQ29yZS9Nb2R1bGVzL2VuY3J5cHRl
ZG1lZGlhL01lZGlhS2V5U2Vzc2lvbi5jcHAKaW5kZXggZDhhZDlkZGYxMmU3MWYxZWY4ZTA5Nzg5
ZjRmYmQxMTRjNWU1MmU0Ni4uYTM2Zjk5ZmIyNjg1Y2JiODM4NzhjYzliNTMxZWQ4ZmQ3N2JlNDEw
MCAxMDA2NDQKLS0tIGEvU291cmNlL1dlYkNvcmUvTW9kdWxlcy9lbmNyeXB0ZWRtZWRpYS9NZWRp
YUtleVNlc3Npb24uY3BwCisrKyBiL1NvdXJjZS9XZWJDb3JlL01vZHVsZXMvZW5jcnlwdGVkbWVk
aWEvTWVkaWFLZXlTZXNzaW9uLmNwcApAQCAtNjMyLDggKzYzMiwxMSBAQCB2b2lkIE1lZGlhS2V5
U2Vzc2lvbjo6dXBkYXRlS2V5U3RhdHVzZXMoQ0RNSW5zdGFuY2U6OktleVN0YXR1c1ZlY3RvciYm
IGlucHV0U3RhdAogICAgICAgICBjYXNlIENETUluc3RhbmNlOjpLZXlTdGF0dXM6OlN0YXR1c1Bl
bmRpbmc6CiAgICAgICAgICAgICByZXR1cm4gTWVkaWFLZXlTdGF0dXM6OlN0YXR1c1BlbmRpbmc7
CiAgICAgICAgIGNhc2UgQ0RNSW5zdGFuY2U6OktleVN0YXR1czo6SW50ZXJuYWxFcnJvcjoKKyAg
ICAgICAgICAgIHJldHVybiBNZWRpYUtleVN0YXR1czo6SW50ZXJuYWxFcnJvcjsKICAgICAgICAg
ICAgIGJyZWFrOwogICAgICAgICB9OworCisgICAgICAgIEFTU0VSVF9OT1RfUkVBQ0hFRCgpOwog
ICAgICAgICByZXR1cm4gTWVkaWFLZXlTdGF0dXM6OkludGVybmFsRXJyb3I7CiAgICAgfTsKIAo=
</data>

          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>314473</attachid>
            <date>2017-07-03 04:48:42 -0700</date>
            <delta_ts>2017-07-04 04:12:30 -0700</delta_ts>
            <desc>Patch for landing</desc>
            <filename>bug-174020-20170703134841.patch</filename>
            <type>text/plain</type>
            <size>2434</size>
            <attacher name="Xabier Rodríguez Calvar">calvaris</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMjE5MDY1CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9D
aGFuZ2VMb2cgYi9Tb3VyY2UvV2ViQ29yZS9DaGFuZ2VMb2cKaW5kZXggMmFkM2IxZjE2MGQ5OWU2
ZGJkZGRmZDUwZWI1ZjM3YThlZDAwYmFiMy4uZWJhMTNjMWFkYzU4NzRiOGUzN2EzMTdjZTgwYWE3
YzIwNWU4MzA5ZiAxMDA2NDQKLS0tIGEvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCisrKyBiL1Nv
dXJjZS9XZWJDb3JlL0NoYW5nZUxvZwpAQCAtMSwzICsxLDE4IEBACisyMDE3LTA2LTMwICBYYWJp
ZXIgUm9kcmlndWV6IENhbHZhciAgPGNhbHZhcmlzQGlnYWxpYS5jb20+CisKKyAgICAgICAgW0VN
RV0gU29sdmUgYSBjb3VwbGUgb2YgY29tcGlsZXIgd2FybmluZ3MKKyAgICAgICAgaHR0cHM6Ly9i
dWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTE3NDAyMAorCisgICAgICAgIFJldmlld2Vk
IGJ5IE1pY2hhZWwgQ2F0YW56YXJvLgorCisgICAgICAgICogTW9kdWxlcy9lbmNyeXB0ZWRtZWRp
YS9DRE0uY3BwOgorICAgICAgICAoV2ViQ29yZTo6Q0RNOjppc1BlcnNpc3RlbnRUeXBlKTogQWRk
ZWQgZGVmYXVsdCByZXR1cm4gYW5kCisgICAgICAgIGFzc2VydGlvbi4KKyAgICAgICAgKiBNb2R1
bGVzL2VuY3J5cHRlZG1lZGlhL01lZGlhS2V5U2Vzc2lvbi5jcHA6CisgICAgICAgIChXZWJDb3Jl
OjpNZWRpYUtleVNlc3Npb246OnVwZGF0ZUtleVN0YXR1c2VzKTogVGhpcyB3YXJuaW5nIHdhcwor
ICAgICAgICBhbHJlYWR5IHNvbHZlZCBidXQgSSB0aGluayBhZGRpbmcgYW4gYXNzZXJ0aW9uIGZv
ciB0aGUgZGVmYXVsdAorICAgICAgICBjYXNlIGNhbiBoZWxwIGNhdGNoIGVycm9ycyBpbiB0aGUg
ZnV0dXJlLgorCiAyMDE3LTA3LTAzICBaYW4gRG9iZXJzZWsgIDx6ZG9iZXJzZWtAaWdhbGlhLmNv
bT4KIAogICAgICAgICBbR0NyeXB0XSBJbXBsZW1lbnQgQ3J5cHRvS2V5RUMgU1BLSSBleHBvcnRz
CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9Nb2R1bGVzL2VuY3J5cHRlZG1lZGlhL0NETS5j
cHAgYi9Tb3VyY2UvV2ViQ29yZS9Nb2R1bGVzL2VuY3J5cHRlZG1lZGlhL0NETS5jcHAKaW5kZXgg
YWUzZmUwNmQyMmE0MTAyYTgzYTBhN2M1OWZkYzcyNTY0ZWE4MDI0Yy4uZDNkMDk2NjQyYWZkNzc3
NDlhOGEzMjI5ZmM5MmQ2YzE5ZjNiOTc2MyAxMDA2NDQKLS0tIGEvU291cmNlL1dlYkNvcmUvTW9k
dWxlcy9lbmNyeXB0ZWRtZWRpYS9DRE0uY3BwCisrKyBiL1NvdXJjZS9XZWJDb3JlL01vZHVsZXMv
ZW5jcnlwdGVkbWVkaWEvQ0RNLmNwcApAQCAtMTgxLDYgKzE4MSw5IEBAIGJvb2wgQ0RNOjppc1Bl
cnNpc3RlbnRUeXBlKE1lZGlhS2V5U2Vzc2lvblR5cGUgc2Vzc2lvblR5cGUpCiAgICAgICAgIC8v
IOKGsyAicGVyc2lzdGVudC1saWNlbnNlIgogICAgICAgICByZXR1cm4gdHJ1ZTsKICAgICB9CisK
KyAgICBBU1NFUlRfTk9UX1JFQUNIRUQoKTsKKyAgICByZXR1cm4gZmFsc2U7CiB9CiAKIHN0ZDo6
b3B0aW9uYWw8TWVkaWFLZXlTeXN0ZW1Db25maWd1cmF0aW9uPiBDRE06OmdldFN1cHBvcnRlZENv
bmZpZ3VyYXRpb24oY29uc3QgTWVkaWFLZXlTeXN0ZW1Db25maWd1cmF0aW9uJiBjYW5kaWRhdGVD
b25maWd1cmF0aW9uLCBNZWRpYUtleXNSZXN0cmljdGlvbnMmIHJlc3RyaWN0aW9ucykKZGlmZiAt
LWdpdCBhL1NvdXJjZS9XZWJDb3JlL01vZHVsZXMvZW5jcnlwdGVkbWVkaWEvTWVkaWFLZXlTZXNz
aW9uLmNwcCBiL1NvdXJjZS9XZWJDb3JlL01vZHVsZXMvZW5jcnlwdGVkbWVkaWEvTWVkaWFLZXlT
ZXNzaW9uLmNwcAppbmRleCBkOGFkOWRkZjEyZTcxZjFlZjhlMDk3ODlmNGZiZDExNGM1ZTUyZTQ2
Li5iZDU1NzY4Yjc4MjZmNzAwZjRhYWQzY2M2YTU5ZmRmYjYzYWQxYWE5IDEwMDY0NAotLS0gYS9T
b3VyY2UvV2ViQ29yZS9Nb2R1bGVzL2VuY3J5cHRlZG1lZGlhL01lZGlhS2V5U2Vzc2lvbi5jcHAK
KysrIGIvU291cmNlL1dlYkNvcmUvTW9kdWxlcy9lbmNyeXB0ZWRtZWRpYS9NZWRpYUtleVNlc3Np
b24uY3BwCkBAIC02MzIsOCArNjMyLDEwIEBAIHZvaWQgTWVkaWFLZXlTZXNzaW9uOjp1cGRhdGVL
ZXlTdGF0dXNlcyhDRE1JbnN0YW5jZTo6S2V5U3RhdHVzVmVjdG9yJiYgaW5wdXRTdGF0CiAgICAg
ICAgIGNhc2UgQ0RNSW5zdGFuY2U6OktleVN0YXR1czo6U3RhdHVzUGVuZGluZzoKICAgICAgICAg
ICAgIHJldHVybiBNZWRpYUtleVN0YXR1czo6U3RhdHVzUGVuZGluZzsKICAgICAgICAgY2FzZSBD
RE1JbnN0YW5jZTo6S2V5U3RhdHVzOjpJbnRlcm5hbEVycm9yOgotICAgICAgICAgICAgYnJlYWs7
CisgICAgICAgICAgICByZXR1cm4gTWVkaWFLZXlTdGF0dXM6OkludGVybmFsRXJyb3I7CiAgICAg
ICAgIH07CisKKyAgICAgICAgQVNTRVJUX05PVF9SRUFDSEVEKCk7CiAgICAgICAgIHJldHVybiBN
ZWRpYUtleVN0YXR1czo6SW50ZXJuYWxFcnJvcjsKICAgICB9OwogCg==
</data>

          </attachment>
      

    </bug>

</bugzilla>