Fix a crash in MimeHandlerServiceImpl::AbortStream().

Fix a crash in MimeHandlerServiceImpl::AbortStream().

BUG=452799
Committed: https://crrev.com/1cd6baf6813f91375098d1b74016575d2b135e33
Cr-Commit-Position: refs/heads/master@{#313472}

[Sam McNally, 2015-01-28 05:18:27]

sammc@chromium.org changed reviewers:
+ benwells@chromium.org

[benwells, 2015-01-28 05:22:01]

On 2015/01/28 05:18:28, Sam McNally wrote:

Is there a bug logged for this? I'm wondering how it happens. Also, can we get a
test?

[Sam McNally, 2015-01-28 06:40:00]

On 2015/01/28 05:22:01, benwells wrote:
> On 2015/01/28 05:18:28, Sam McNally wrote:
> 
> Is there a bug logged for this? 
No.

> I'm wondering how it happens.
It happens when the mime guest is destroyed while an AbortStream message is in
flight. This can occur if a PDF is closed or navigated away from at the right
point in time after opening. (For some large PDFs, the plugin aborts the stream
and does its own download)

> Also, can we get a test?
Done.

[benwells, 2015-01-28 07:08:35]

On 2015/01/28 06:40:00, Sam McNally wrote:
> On 2015/01/28 05:22:01, benwells wrote:
> > On 2015/01/28 05:18:28, Sam McNally wrote:
> > 
> > Is there a bug logged for this? 
> No.

OK. Can you file one please?

> 
> > I'm wondering how it happens.
> It happens when the mime guest is destroyed while an AbortStream message is in
> flight. This can occur if a PDF is closed or navigated away from at the right
> point in time after opening. (For some large PDFs, the plugin aborts the
stream
> and does its own download)
> 
> > Also, can we get a test?
> Done.

[Sam McNally, 2015-01-28 07:17:40]

On 2015/01/28 07:08:35, benwells wrote:
> On 2015/01/28 06:40:00, Sam McNally wrote:
> > On 2015/01/28 05:22:01, benwells wrote:
> > > On 2015/01/28 05:18:28, Sam McNally wrote:
> > > 
> > > Is there a bug logged for this? 
> > No.
> 
> OK. Can you file one please?
Done.
> 
> > 
> > > I'm wondering how it happens.
> > It happens when the mime guest is destroyed while an AbortStream message is
in
> > flight. This can occur if a PDF is closed or navigated away from at the
right
> > point in time after opening. (For some large PDFs, the plugin aborts the
> stream
> > and does its own download)
> > 
> > > Also, can we get a test?
> > Done.

[benwells, 2015-01-28 07:54:15]

lgtm

[Sam McNally, 2015-01-28 09:27:05]

The CQ bit was checked by sammc@chromium.org

[commit-bot: I haz the power, 2015-01-28 09:27:49]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/885563002/20002

[commit-bot: I haz the power, 2015-01-28 10:16:37]

Committed patchset #3 (id:20002)

[commit-bot: I haz the power, 2015-01-28 10:17:47]

Patchset 3 (id:??) landed as
https://crrev.com/1cd6baf6813f91375098d1b74016575d2b135e33
Cr-Commit-Position: refs/heads/master@{#313472}