<html>
    <head>
      <base href="https://bugs.webkit.org/">
    </head>
    <body><span class="vcard"><a class="email" href="mailto:bandou.yacine@gmail.com" title="Yacine Bandou <bandou.yacine@gmail.com>"> <span class="fn">Yacine Bandou</span></a>
</span> changed
          <a class="bz_bug_link 
          bz_status_RESOLVED  bz_closed"
   title="RESOLVED FIXED - [EME][GStreamer] Add a handler for GStreamer protection event"
   href="https://bugs.webkit.org/show_bug.cgi?id=185245">bug 185245</a>
          <br>
             <table border="1" cellspacing="0" cellpadding="8">
          <tr>
            <th>What</th>
            <th>Removed</th>
            <th>Added</th>
          </tr>

         <tr>
           <td style="text-align:right;">Blocks</td>
           <td>185535
           </td>
           <td>
                
           </td>
         </tr></table>
      <p>
        <div>
            <b><a class="bz_bug_link 
          bz_status_RESOLVED  bz_closed"
   title="RESOLVED FIXED - [EME][GStreamer] Add a handler for GStreamer protection event"
   href="https://bugs.webkit.org/show_bug.cgi?id=185245#c17">Comment # 17</a>
              on <a class="bz_bug_link 
          bz_status_RESOLVED  bz_closed"
   title="RESOLVED FIXED - [EME][GStreamer] Add a handler for GStreamer protection event"
   href="https://bugs.webkit.org/show_bug.cgi?id=185245">bug 185245</a>
              from <span class="vcard"><a class="email" href="mailto:bandou.yacine@gmail.com" title="Yacine Bandou <bandou.yacine@gmail.com>"> <span class="fn">Yacine Bandou</span></a>
</span></b>
        <pre>(In reply to Yacine Bandou from <a href="show_bug.cgi?id=185245#c16">comment #16</a>)
<span class="quote">> I noted that you changed the patch. You didn't land the last pushed patch
> and you landed the patch which has dependency with this <a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - [EME][GStreamer] Dispatch the protection event from AppendPipeline to MediaPlayer"
   href="show_bug.cgi?id=185420">bug 185420</a>.

> That provoked the crashes of encrypted-media layoutTests in debug mode as i
> commented in c10.</span >


Now, to fix these crashes, we should take this patch 185535.

I think there was a misunderstanding, now I implemented the patch that planned in 185420 in an other one 185535, 
in order to avoid all other misunderstanding.


For more detail, here is the history of this patch: 

1. I pushed a patch that adds a probe for GStreamer protection event and adds a handler for this event in MediaPlayerPrivate (Patch 1)

2. Calvaris asked me to split the code in two parts : 
   - A probe to catch the GStreamer protection event
   - A handler of  the event in MediaPlayerPrivate

3. I removed the handler of the event in MediaPlayerPrivate from this patch1  and I replaced it by removing the ASSERT_NOT_REACHED 
   because it is needed in debug mode. (patch 2)

4. Calvaris asked me to move the delete of the ASSERT in an other bug

5. I moved the ASSERT removal in the <a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - [EME][GStreamer] Dispatch the protection event from AppendPipeline to MediaPlayer"
   href="show_bug.cgi?id=185420">bug 185420</a> (patch 3)

6. Calvaris prefers to implement the handler of the protection event in <a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - [EME][GStreamer] Dispatch the protection event from AppendPipeline to MediaPlayer"
   href="show_bug.cgi?id=185420">bug 185420</a> instead to just remove the ASSERT. 

7. In order to take more time to implement the patch 185420. I made these two patches independent by removing the call of 
   MediaPlayerPrivate in the probe of the protection event. (Patch 4).

I close the <a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - [EME][GStreamer] Dispatch the protection event from AppendPipeline to MediaPlayer"
   href="show_bug.cgi?id=185420">bug 185420</a></pre>
        </div>
      </p>

        <div id="referenced">
          <hr style="border: 1px dashed #969696">
          <b>Referenced Bugs:</b>
          <ul>
              <li>
                [<a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - [EME][GStreamer] Handle the protection event in MediaPlayerPrivate"
   href="https://bugs.webkit.org/show_bug.cgi?id=185535">Bug 185535</a>] [EME][GStreamer] Handle the protection event in MediaPlayerPrivate
              </li>
          </ul>
        </div>
        <br>

      <hr>
      <span>You are receiving this mail because:</span>

      <ul>
          <li>You are the assignee for the bug.</li>
      </ul>
    </body>
</html>