Remove the gyp flag for registerProtocolHandler.

Remove the gyp flag for registerProtocolHandler.

We've been shipping RPH for a while with no adverse effects so it should be
safe to remove the flag.

BUG=126569
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=136129

[tony, 2012-05-08 22:56:47]

This depends on https://bugs.webkit.org/show_bug.cgi?id=85924 , so I'll wait for
that to land before running a try job.

[koz (OOO until 15th September), 2012-05-08 23:00:12]

lgtm

[benwells, 2012-05-08 23:37:07]

lgtm

[commit-bot: I haz the power, 2012-05-09 19:03:01]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/tony@chromium.org/10381064/1

[commit-bot: I haz the power, 2012-05-09 19:03:11]

Presubmit check for 10381064-1 failed and returned exit status 1.

Running presubmit commit checks ...

** Presubmit Messages **
If this change requires manual test instructions to QA team, add
TEST=[instructions].

** Presubmit ERRORS **
Missing LGTM from an OWNER for files in these directories:
    chrome/browser/ui/webui/options2
    chrome/browser/resources/options2

Presubmit checks took 1.7s to calculate.

Was the presubmit check useful? Please send feedback & hate mail to
maruel@chromium.org!

[tony, 2012-05-09 19:12:48]

+estade,jhawkins for OWNERS

[James Hawkins, 2012-05-09 19:24:27]

LGTM with nit.

https://chromiumcodereview.appspot.com/10381064/diff/1/chrome/browser/resourc...
File chrome/browser/resources/options2/options.html (right):

https://chromiumcodereview.appspot.com/10381064/diff/1/chrome/browser/resourc...
chrome/browser/resources/options2/options.html:130: <include
src="handler_options.html">
nit: Move this up to ~line 117-118.

[tony, 2012-05-09 19:38:34]

On 2012/05/09 19:24:27, James Hawkins wrote:
> LGTM with nit.
> 
>
https://chromiumcodereview.appspot.com/10381064/diff/1/chrome/browser/resourc...
> File chrome/browser/resources/options2/options.html (right):
> 
>
https://chromiumcodereview.appspot.com/10381064/diff/1/chrome/browser/resourc...
> chrome/browser/resources/options2/options.html:130: <include
> src="handler_options.html">
> nit: Move this up to ~line 117-118.

Done.

[commit-bot: I haz the power, 2012-05-09 19:39:01]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/tony@chromium.org/10381064/2003

[commit-bot: I haz the power, 2012-05-09 21:34:27]

Change committed as 136129