Fix condition of FileReader::hasPendingActivity

Fix condition of FileReader::hasPendingActivity

Before this patch, |hasPendingActiviy| checked |m_loadingState| to check if it is stopped.
However, when called |abort()|, FileReader schedules a delayed task to call |doAbort()|, while setting |m_loadingState| to |LoadingStateAborted|. This may cause the FileReader to go away when |doAbort()| is called.

This patch avoids the issue by having |hasPendingActivity| look at |m_state| instead.

BUG=393425
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=178019

[haraken, 2014-07-14 00:34:55]

Would you give me a permission to see the bug?

Also add more to the CL description.

[blink-reviews, 2014-07-14 00:37:41]

Adding a LayoutTest now. Please hold on review.


On Mon, Jul 14, 2014 at 9:34 AM, <haraken@chromium.org> wrote:

> Would you give me a permission to see the bug?
>
> Also add more to the CL description.
>
>
> https://codereview.chromium.org/389193002/
>



-- 
Kouhei Ueno

To unsubscribe from this group and stop receiving emails from it, send an email
to blink-reviews+unsubscribe@chromium.org.

[kouhei (in TOK), 2014-07-14 00:54:19]

Updated description and added a test.
PTAL.

[haraken, 2014-07-14 01:06:56]

LGTM

[kouhei (in TOK), 2014-07-14 01:07:25]

The CQ bit was checked by kouhei@chromium.org

[commit-bot: I haz the power, 2014-07-14 01:07:54]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/kouhei@chromium.org/389193002/20001

[commit-bot: I haz the power, 2014-07-14 01:55:00]

Change committed as 178019