Revert of Fix the handling of user gestures for external protocol handler dialogs.

Revert of Fix the handling of user gestures for external protocol handler dialogs. (https://codereview.chromium.org/131783012/)

Reason for revert:
This change broke certain cases where external protocol handlers are launched. Consider relanding depending on whether http://crbug.com/78592 is to be fixed.

Original issue's description:
> Fix the handling of user gestures for external protocol handler dialogs.
> 
> - Remove browser state from external protocol handler.
> - Use gesture with a timeout.
> 
> BUG=173557
> 
> Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=261014

TBR=boliu@chromium.org

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=286301

[meacer, 2014-07-28 18:00:53]

Created Revert of Fix the handling of user gestures for external protocol
handler dialogs.

[meacer, 2014-07-28 19:31:12]

jochen, kalman: Can you please take a look as owners?

[not at google - send to devlin, 2014-07-28 19:33:12]

is it any more complicated than a plain revert? if not, lgtm.

[meacer, 2014-07-28 19:37:34]

On 2014/07/28 19:33:12, kalman wrote:
> is it any more complicated than a plain revert? if not, lgtm.

It's a plain revert but wanted to check if there's possibility that it could
break any new protocol handling stuff (shouldn't).

[not at google - send to devlin, 2014-07-28 19:41:28]

In my case - wouldn't really know where to look for such things.

[joi_joisig.com, 2014-07-28 22:06:49]

Minus myself for review as I'm no longer actively involved in
Chromium. I think Jochen can review the bits you might have intended
me for.

Cheers,
Jói


On Mon, Jul 28, 2014 at 7:41 PM,  <kalman@chromium.org> wrote:
> In my case - wouldn't really know where to look for such things.
>
> https://codereview.chromium.org/426713002/

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

[jochen (gone - plz use gerrit), 2014-07-29 09:14:59]

plain revert lgtm

you'd expect that new protocol handling stuff came with tests :)

[meacer, 2014-07-29 17:31:56]

On 2014/07/29 09:14:59, jochen wrote:
> plain revert lgtm
> 
> you'd expect that new protocol handling stuff came with tests :)

I agree, even though change in behavior was unintended more tests would be
better.

[meacer, 2014-07-29 17:32:00]

The CQ bit was checked by meacer@chromium.org

[meacer, 2014-07-29 17:32:07]

The CQ bit was unchecked by meacer@chromium.org

[meacer, 2014-07-29 17:33:01]

The CQ bit was checked by meacer@chromium.org

[commit-bot: I haz the power, 2014-07-29 17:34:15]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/meacer@chromium.org/426713002/150001

[commit-bot: I haz the power, 2014-07-29 21:39:33]

Change committed as 286301