move rAc from behind flags

move rAc from behind flags

It was on by default anyway, and I don't see a reason to keep this flag around any more.

BUG=none
R=isherman@chromium.org, thestig@chromium.org

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

[Ilya Sherman, 2014-03-26 03:57:50]

I'm fine with removing these; but while you're here, maybe there's more we can
remove?

https://codereview.chromium.org/197333008/diff/30001/chrome/renderer/chrome_r...
File chrome/renderer/chrome_render_process_observer.cc (right):

https://codereview.chromium.org/197333008/diff/30001/chrome/renderer/chrome_r...
chrome/renderer/chrome_render_process_observer.cc:261: #if
defined(ENABLE_AUTOFILL_DIALOG)
Is this #define still needed?

https://codereview.chromium.org/197333008/diff/30001/chrome/renderer/chrome_r...
chrome/renderer/chrome_render_process_observer.cc:262:
WebRuntimeFeatures::enableRequestAutocomplete(true);
Is this method still needed?

[Evan Stade, 2014-03-26 17:41:59]

https://codereview.chromium.org/197333008/diff/30001/chrome/renderer/chrome_r...
File chrome/renderer/chrome_render_process_observer.cc (right):

https://codereview.chromium.org/197333008/diff/30001/chrome/renderer/chrome_r...
chrome/renderer/chrome_render_process_observer.cc:261: #if
defined(ENABLE_AUTOFILL_DIALOG)
On 2014/03/26 03:57:50, Ilya Sherman wrote:
> Is this #define still needed?

For now yes, I believe, for example it's false on non-aura Linux. It's used in a
lot of places so I think it makes sense to make removing it a separate task.

https://codereview.chromium.org/197333008/diff/30001/chrome/renderer/chrome_r...
chrome/renderer/chrome_render_process_observer.cc:262:
WebRuntimeFeatures::enableRequestAutocomplete(true);
On 2014/03/26 03:57:50, Ilya Sherman wrote:
> Is this method still needed?

Yes, as long as ENABLE_AUTOFILL_DIALOG exists

[Ilya Sherman, 2014-03-26 23:13:31]

Fair enough.  LGTM.

[Evan Stade, 2014-03-26 23:14:10]

The CQ bit was checked by estade@chromium.org

[commit-bot: I haz the power, 2014-03-26 23:14:23]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/estade@chromium.org/197333008/30001

[commit-bot: I haz the power, 2014-03-26 23:43:03]

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

[commit-bot: I haz the power, 2014-03-26 23:43:03]

Retried try job too often on chromium_presubmit for step(s) presubmit
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=chromium_p...

[Evan Stade, 2014-03-27 19:29:24]

+Lei for OWNERS on chrome/renderer/

[Lei Zhang, 2014-03-27 19:35:36]

lgtm

[Evan Stade, 2014-04-08 21:32:54]

The CQ bit was checked by estade@chromium.org

[commit-bot: I haz the power, 2014-04-08 21:33:29]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/estade@chromium.org/197333008/30001

[commit-bot: I haz the power, 2014-04-08 23:09:29]

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

[commit-bot: I haz the power, 2014-04-08 23:09:29]

Try jobs failed on following builders:
  tryserver.chromium on mac_chromium_rel

[Evan Stade, 2014-04-10 01:04:25]

Committed patchset #2 manually as r262883 (presubmit successful).