cros: Fix "Add person" long delay regression

cros: Fix "Add person" long delay regression

The long delay is caused by plugin data removal timed out,
which is caused by the fake Flash plugin. ChromeOS should
not use the fake Flash plugin since its Flash is bundled.

BUG=644400

[xiyuan, 2016-09-13 21:57:47]

The CQ bit was checked by xiyuan@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-09-13 21:58:10]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[xiyuan, 2016-09-13 21:59:59]

xiyuan@chromium.org changed reviewers:
+ waffles@chromium.org

[xiyuan, 2016-09-13 21:59:59]

This fixes http://crbug.com/644400. WDYT?

[commit-bot: I haz the power, 2016-09-13 22:56:32]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-09-13 22:56:33]

Dry run: This issue passed the CQ dry run.

[xiyuan, 2016-09-15 16:10:11]

ping?

[waffles, 2016-09-15 17:19:22]

On 2016/09/15 16:10:11, xiyuan wrote:
> ping?

Sorry, I was on leave.

It looks like this will also be fixed by
https://codereview.chromium.org/2340643002/, the idea being that
FLAPPER_AVAILABLE will no longer be defined on ChromeOS - how do you feel about
checking in that change only?

[waffles, 2016-09-15 17:19:33]

Description was changed from

==========
cros: Fix "Add person" long delay regression

The long delay is caused by plugin data removal timed out,
which is caused by the fake Flash plugin. ChromeOS should
not use the fake Flash plugin since its Flash is bundled.

BUG=644400
==========

to

==========
cros: Fix "Add person" long delay regression

The long delay is caused by plugin data removal timed out,
which is caused by the fake Flash plugin. ChromeOS should
not use the fake Flash plugin since its Flash is bundled.

BUG=644400
==========

[xiyuan, 2016-09-15 17:22:33]

On 2016/09/15 17:19:22, waffles wrote:
> On 2016/09/15 16:10:11, xiyuan wrote:
> > ping?
> 
> Sorry, I was on leave.
> 
> It looks like this will also be fixed by
> https://codereview.chromium.org/2340643002/, the idea being that
> FLAPPER_AVAILABLE will no longer be defined on ChromeOS - how do you feel
about
> checking in that change only?

I am fine with the CL as long as it fixes the long delay problem. Thanks for the
pointer. I will give the CL try.

[xiyuan, 2016-09-15 17:57:19]

On 2016/09/15 17:22:33, xiyuan wrote:
> On 2016/09/15 17:19:22, waffles wrote:
> > On 2016/09/15 16:10:11, xiyuan wrote:
> > > ping?
> > 
> > Sorry, I was on leave.
> > 
> > It looks like this will also be fixed by
> > https://codereview.chromium.org/2340643002/, the idea being that
> > FLAPPER_AVAILABLE will no longer be defined on ChromeOS - how do you feel
> about
> > checking in that change only?
> 
> I am fine with the CL as long as it fixes the long delay problem. Thanks for
the
> pointer. I will give the CL try.

Yep, the CL fixes the long delay problem. I will abandon this CL then.