Commit da248227 authored by Findit's avatar Findit

Revert "Add GPayAppDynamicUpdate feature flag."

This reverts commit 02d1e955.

Reason for revert:

Findit (https://goo.gl/kROfz5) identified CL at revision 844274 as the
culprit for failures in the build cycles as shown on:
https://analysis.chromium.org/waterfall/culprit?key=ag9zfmZpbmRpdC1mb3ItbWVyRAsSDVdmU3VzcGVjdGVkQ0wiMWNocm9taXVtLzAyZDFlOTU1YTczNjg0ZGI5ODU2Y2Q4MjAzYThjNGUzODRjOTIwZGMM

Sample Failed Build: https://ci.chromium.org/b/8857990510232294496

Sample Failed Step: compile

Original change's description:
> Add GPayAppDynamicUpdate feature flag.
> 
> The feature flag controls the GPay native app integration for dynamic update on Android.
> If this feature flag is enabled, the GooglePayPaymentApp would enable a callback service and handle communications between the native GPay app and the browser for dynamic updates on shipping and payment data.
> Design doc is go/dynamic-update-mweb.
> 
> Bug: 1074423
> Change-Id: I898199446179d6c314b833c3ecb8f017481fc596
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2628745
> Commit-Queue: Kaiyu Chen <kylechenkyc@google.com>
> Reviewed-by: Sahel Sharify <sahel@chromium.org>
> Reviewed-by: Rouslan Solomakhin <rouslan@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#844274}


Change-Id: I72abe5615c0ee585679600e2b757c0d10ea60ca9
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 1074423
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2632842
Cr-Commit-Position: refs/heads/master@{#844357}
parent a2547f04
......@@ -18,7 +18,6 @@ public class PaymentFeatureList {
/** Alphabetical: */
public static final String ANDROID_APP_PAYMENT_UPDATE_EVENTS = "AndroidAppPaymentUpdateEvents";
public static final String ENFORCE_FULL_DELEGATION = "EnforceFullDelegation";
public static final String GPAY_APP_DYNAMIC_UPDATE = "GPayAppDynamicUpdate";
public static final String PAYMENT_REQUEST_SKIP_TO_GPAY = "PaymentRequestSkipToGPay";
public static final String PAYMENT_REQUEST_SKIP_TO_GPAY_IF_NO_CARD =
"PaymentRequestSkipToGPayIfNoCard";
......
......@@ -27,7 +27,6 @@ const base::Feature* kFeaturesExposedToJava[] = {
&features::kAppStoreBilling,
&features::kAppStoreBillingDebug,
&features::kEnforceFullDelegation,
&features::kGPayAppDynamicUpdate,
&features::kPaymentRequestSkipToGPay,
&features::kPaymentRequestSkipToGPayIfNoCard,
&features::kReturnGooglePayInBasicCard,
......
......@@ -89,8 +89,5 @@ const base::Feature kSecurePaymentConfirmation {
#endif // OS_MAC
};
const base::Feature kGPayAppDynamicUpdate{"GPayAppDynamicUpdate",
base::FEATURE_DISABLED_BY_DEFAULT};
} // namespace features
} // namespace payments
......@@ -90,10 +90,6 @@ extern const base::Feature kEnforceFullDelegation;
// SecurePaymentConfirmation to be available.
extern const base::Feature kSecurePaymentConfirmation;
// If enabled, the GooglePayPaymentApp handles communications between the native
// GPay app and the browser for dynamic updates on shipping and payment data.
extern const base::Feature kGPayAppDynamicUpdate;
} // namespace features
} // namespace payments
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment