Enable PaymentRequest on Android by default.

Enable PaymentRequest on Android by default.

BUG=627648
Committed: https://crrev.com/59899871c544eb16d31c31b6eb761f83de00fd7d
Cr-Commit-Position: refs/heads/master@{#406631}

[please use gerrit instead, 2016-07-12 23:30:34]

rouslan@chromium.org changed reviewers:
+ dfalcantara@chromium.org, mek@chromium.org

[please use gerrit instead, 2016-07-12 23:30:35]

mek@, dfalcantara@, FYI. This is how I plan to enable PaymentRequest when it's
ready to ship. No need to review now.

[please use gerrit instead, 2016-07-15 23:19:28]

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

[commit-bot: I haz the power, 2016-07-15 23:19:53]

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

[please use gerrit instead, 2016-07-15 23:37:11]

mek@ & dfalcantara@, ptal patch 3.

PaymentRequest is in a shippable state. We are getting final launch approvals
now. zkoch@ has requested that we enable by default on trunk with a Finch kill
switch (only Android). This is accomplished by using the base::FeatureList API.

[commit-bot: I haz the power, 2016-07-16 00:36:14]

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

[commit-bot: I haz the power, 2016-07-16 00:36:15]

Dry run: This issue passed the CQ dry run.

[Marijn Kruisselbrink, 2016-07-18 16:43:56]

lgtm

[gone, 2016-07-18 19:15:42]

lgtm

[please use gerrit instead, 2016-07-18 23:16:41]

rouslan@chromium.org changed reviewers:
+ esprehn@chromium.org, isherman@chromium.org, nasko@chromium.org

[please use gerrit instead, 2016-07-18 23:16:42]

esprehn@, owners ptal in patch 3:

content/child/runtime_features.cc
third_party/WebKit/Source/web/WebRuntimeFeatures.cpp
third_party/WebKit/public/web/WebRuntimeFeatures.h

nasko@, owners ptal in patch 3:

content/public/common/content_features.h
content/public/common/content_features.cc

isherman@, owners ptal in patch 3:

tools/metrics/histograms/histograms.xml

[nasko, 2016-07-18 23:37:42]

https://codereview.chromium.org/2145793002/diff/40001/content/public/common/c...
File content/public/common/content_features.cc (right):

https://codereview.chromium.org/2145793002/diff/40001/content/public/common/c...
content/public/common/content_features.cc:155: const base::Feature
kPaymentRequest{"PaymentRequest",
"Web payments API" seems a bit far off from "PaymentRequest". Shouldn't we be
naming the features based on their actual feature name and not implementation
details?

[please use gerrit instead, 2016-07-19 00:20:08]

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

[commit-bot: I haz the power, 2016-07-19 00:21:14]

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

[please use gerrit instead, 2016-07-19 00:23:23]

nasko@, ptal patch 4.

https://codereview.chromium.org/2145793002/diff/40001/content/public/common/c...
File content/public/common/content_features.cc (right):

https://codereview.chromium.org/2145793002/diff/40001/content/public/common/c...
content/public/common/content_features.cc:155: const base::Feature
kPaymentRequest{"PaymentRequest",
On 2016/07/18 23:37:42, nasko wrote:
> "Web payments API" seems a bit far off from "PaymentRequest". Shouldn't we be
> naming the features based on their actual feature name and not implementation
> details?

Changed to kWebPayments.

[Ilya Sherman, 2016-07-19 01:17:19]

histograms.xml lgtm

[commit-bot: I haz the power, 2016-07-19 04:14:28]

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

[commit-bot: I haz the power, 2016-07-19 04:14:29]

Dry run: This issue passed the CQ dry run.

[nasko, 2016-07-19 16:16:00]

content/ LGTM

[esprehn, 2016-07-20 16:13:07]

lgtm

[please use gerrit instead, 2016-07-20 17:17:03]

The CQ bit was checked by rouslan@chromium.org

[please use gerrit instead, 2016-07-20 17:17:03]

The patchset sent to the CQ was uploaded after l-g-t-m from
dfalcantara@chromium.org, mek@chromium.org, esprehn@chromium.org,
isherman@chromium.org, nasko@chromium.org
Link to the patchset: https://chromiumcodereview.appspot.com/2145793002/#ps80001
(title: "Rebase")

[commit-bot: I haz the power, 2016-07-20 17:17:37]

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

[commit-bot: I haz the power, 2016-07-20 18:57:48]

Committed patchset #5 (id:80001)

[commit-bot: I haz the power, 2016-07-20 18:57:52]

CQ bit was unchecked.

[commit-bot: I haz the power, 2016-07-20 19:02:24]

Description was changed from

==========
Enable PaymentRequest on Android by default.

BUG=627648
==========

to

==========
Enable PaymentRequest on Android by default.

BUG=627648

Committed: https://crrev.com/59899871c544eb16d31c31b6eb761f83de00fd7d
Cr-Commit-Position: refs/heads/master@{#406631}
==========

[commit-bot: I haz the power, 2016-07-20 19:02:26]

Patchset 5 (id:??) landed as
https://crrev.com/59899871c544eb16d31c31b6eb761f83de00fd7d
Cr-Commit-Position: refs/heads/master@{#406631}