Move the Google Payments URL functions out of content.

Move the Google Payments URL functions out of content.

According to a comment in the code, switches::kReduceSecurityForTesting
(which is the only requirement for this class to be in content) was used
for requestAutocomplete testing. Now that requestAutocomplete has been
removed, these functions can be moved out of components/autofill/content
into components/autofill/core.

Also, remove the kWalletServiceUrl and kWalletSecureServiceUrl switches.
The former isn't referenced by any code and the while the latter is still
referenced, the only mention I can find for the use of that flag is in a
3 year old thread* that later goes on to specify a different set of
instructions that don't include the use of the flag.

Finally, make some progress on s/Wallet/Payments.

BUG=613281
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation

Committed: https://crrev.com/65639ea0c7f964b96243603dec17ec449ec2ced4
Cr-Commit-Position: refs/heads/master@{#395343}

[Justin Donnelly, 2016-05-20 21:24:11]

Description was changed from

==========
Move the Google Payments URL functions out of content.

According to a comment in the code, switches::kReduceSecurityForTesting
(which is the only requirement for this class to be in content) was used
for requestAutocomplete testing. Now that requestAutocomplete has been
removed, these functions can be moved out of components/autofill/content
into components/autofill/core.

Also, remove the kWalletServiceUrl and kWalletSecureServiceUrl switches.
The former isn't referenced by any code and the while the latter is still
referenced, the only mention I can find for the use of that flag is in a
3 year old thread* that later goes on to specify a different set of
instructions that don't include the use of the flag.

BUG=613281
==========

to

==========
Move the Google Payments URL functions out of content.

According to a comment in the code, switches::kReduceSecurityForTesting
(which is the only requirement for this class to be in content) was used
for requestAutocomplete testing. Now that requestAutocomplete has been
removed, these functions can be moved out of components/autofill/content
into components/autofill/core.

Also, remove the kWalletServiceUrl and kWalletSecureServiceUrl switches.
The former isn't referenced by any code and the while the latter is still
referenced, the only mention I can find for the use of that flag is in a
3 year old thread* that later goes on to specify a different set of
instructions that don't include the use of the flag.

BUG=613281
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation
==========

[Justin Donnelly, 2016-05-20 21:25:44]

The thread referencing the old flag can be found at
https://goto.google.com/xjnbc.

[Justin Donnelly, 2016-05-20 21:41:54]

Description was changed from

==========
Move the Google Payments URL functions out of content.

According to a comment in the code, switches::kReduceSecurityForTesting
(which is the only requirement for this class to be in content) was used
for requestAutocomplete testing. Now that requestAutocomplete has been
removed, these functions can be moved out of components/autofill/content
into components/autofill/core.

Also, remove the kWalletServiceUrl and kWalletSecureServiceUrl switches.
The former isn't referenced by any code and the while the latter is still
referenced, the only mention I can find for the use of that flag is in a
3 year old thread* that later goes on to specify a different set of
instructions that don't include the use of the flag.

BUG=613281
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation
==========

to

==========
Move the Google Payments URL functions out of content.

According to a comment in the code, switches::kReduceSecurityForTesting
(which is the only requirement for this class to be in content) was used
for requestAutocomplete testing. Now that requestAutocomplete has been
removed, these functions can be moved out of components/autofill/content
into components/autofill/core.

Also, remove the kWalletServiceUrl and kWalletSecureServiceUrl switches.
The former isn't referenced by any code and the while the latter is still
referenced, the only mention I can find for the use of that flag is in a
3 year old thread* that later goes on to specify a different set of
instructions that don't include the use of the flag.

Finally, make some progress on s/Wallet/Payments.

BUG=613281
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation
==========

[Justin Donnelly, 2016-05-20 21:44:38]

Patchset #1 (id:1) has been deleted

[Justin Donnelly, 2016-05-20 21:46:01]

jdonnelly@chromium.org changed reviewers:
+ estade@chromium.org

[Evan Stade, 2016-05-20 21:48:24]

lgtm

[Evan Stade, 2016-05-20 21:48:42]

On 2016/05/20 21:48:24, Evan Stade wrote:
> lgtm

(rubber stamp)

[Justin Donnelly, 2016-05-20 22:31:47]

jdonnelly@chromium.org changed reviewers:
+ pkasting@chromium.org, scottmg@chromium.org

[Justin Donnelly, 2016-05-20 22:31:48]

+scottmg for build/common.gypi
+pkasting for chrome/browser/ui/startup/bad_flags_prompt.cc

[Peter Kasting, 2016-05-20 22:34:01]

LGTM

[scottmg, 2016-05-20 22:45:59]

build lgtm

[Justin Donnelly, 2016-05-23 15:45:31]

The CQ bit was checked by jdonnelly@chromium.org

[commit-bot: I haz the power, 2016-05-23 15:45:43]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1999923002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1999923002/20001

[commit-bot: I haz the power, 2016-05-23 17:03:06]

Description was changed from

==========
Move the Google Payments URL functions out of content.

According to a comment in the code, switches::kReduceSecurityForTesting
(which is the only requirement for this class to be in content) was used
for requestAutocomplete testing. Now that requestAutocomplete has been
removed, these functions can be moved out of components/autofill/content
into components/autofill/core.

Also, remove the kWalletServiceUrl and kWalletSecureServiceUrl switches.
The former isn't referenced by any code and the while the latter is still
referenced, the only mention I can find for the use of that flag is in a
3 year old thread* that later goes on to specify a different set of
instructions that don't include the use of the flag.

Finally, make some progress on s/Wallet/Payments.

BUG=613281
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation
==========

to

==========
Move the Google Payments URL functions out of content.

According to a comment in the code, switches::kReduceSecurityForTesting
(which is the only requirement for this class to be in content) was used
for requestAutocomplete testing. Now that requestAutocomplete has been
removed, these functions can be moved out of components/autofill/content
into components/autofill/core.

Also, remove the kWalletServiceUrl and kWalletSecureServiceUrl switches.
The former isn't referenced by any code and the while the latter is still
referenced, the only mention I can find for the use of that flag is in a
3 year old thread* that later goes on to specify a different set of
instructions that don't include the use of the flag.

Finally, make some progress on s/Wallet/Payments.

BUG=613281
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation
==========

[commit-bot: I haz the power, 2016-05-23 17:03:07]

Committed patchset #1 (id:20001)

[commit-bot: I haz the power, 2016-05-23 17:05:05]

Description was changed from

==========
Move the Google Payments URL functions out of content.

According to a comment in the code, switches::kReduceSecurityForTesting
(which is the only requirement for this class to be in content) was used
for requestAutocomplete testing. Now that requestAutocomplete has been
removed, these functions can be moved out of components/autofill/content
into components/autofill/core.

Also, remove the kWalletServiceUrl and kWalletSecureServiceUrl switches.
The former isn't referenced by any code and the while the latter is still
referenced, the only mention I can find for the use of that flag is in a
3 year old thread* that later goes on to specify a different set of
instructions that don't include the use of the flag.

Finally, make some progress on s/Wallet/Payments.

BUG=613281
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation
==========

to

==========
Move the Google Payments URL functions out of content.

According to a comment in the code, switches::kReduceSecurityForTesting
(which is the only requirement for this class to be in content) was used
for requestAutocomplete testing. Now that requestAutocomplete has been
removed, these functions can be moved out of components/autofill/content
into components/autofill/core.

Also, remove the kWalletServiceUrl and kWalletSecureServiceUrl switches.
The former isn't referenced by any code and the while the latter is still
referenced, the only mention I can find for the use of that flag is in a
3 year old thread* that later goes on to specify a different set of
instructions that don't include the use of the flag.

Finally, make some progress on s/Wallet/Payments.

BUG=613281
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation

Committed: https://crrev.com/65639ea0c7f964b96243603dec17ec449ec2ced4
Cr-Commit-Position: refs/heads/master@{#395343}
==========

[commit-bot: I haz the power, 2016-05-23 17:05:06]

Patchset 1 (id:??) landed as
https://crrev.com/65639ea0c7f964b96243603dec17ec449ec2ced4
Cr-Commit-Position: refs/heads/master@{#395343}