Drop FilterDataState::Applying

Drop FilterDataState::Applying

This state is never transitioned to, so it can be removed.

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

[fs, 2014-09-30 16:06:11]

fs@opera.com changed reviewers:
+ fmalita@chromium.org, schenney@chromium.org, senorblanco@chromium.org

[fs, 2014-09-30 16:06:12]

Probably something that should've been in
https://codereview.chromium.org/500233002. (Discovered while trying to figure
out what transitions there were.)

[f(malita), 2014-09-30 16:08:26]

lgtm

[Stephen White, 2014-09-30 16:09:01]

LGTM too. Thanks for this!

[Stephen Chennney, 2014-09-30 16:13:31]

Did I add this to fix a security problem? I seem to recall things going that
way.

[f(malita), 2014-09-30 16:22:08]

On 2014/09/30 16:13:31, Stephen Chenney wrote:
> Did I add this to fix a security problem? I seem to recall things going that
> way.

I think so, yes. But after switching to deferred filters, the old code is gone
(hopefully not the test that came with it).

[fs, 2014-09-30 16:26:24]

On 2014/09/30 16:22:08, Florin Malita wrote:
> On 2014/09/30 16:13:31, Stephen Chenney wrote:
> > Did I add this to fix a security problem? I seem to recall things going that
> > way.
> 
> I think so, yes. But after switching to deferred filters, the old code is gone
> (hopefully not the test that came with it).

Yepp schenney added it (sort of at least) when fixing
https://bugs.webkit.org/show_bug.cgi?id=94652 is seems. The Cl I mentioned
removed the last transition to said state though.

[fs, 2014-09-30 16:26:32]

The CQ bit was checked by fs@opera.com

[commit-bot: I haz the power, 2014-09-30 16:26:59]

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

[commit-bot: I haz the power, 2014-09-30 17:32:14]

Committed patchset #1 (id:1) as 182947