[Autofill Offer] Temporarily remove offer_details_url validation

We don't need this field in the MVP scope, and we are not 100% sure
what format of the text will be. So remove it for now. Will add it back
when we really use this.

Bug: 1112095
Change-Id: I919c5520cf14da24d9474fff0b3f95716fd31935
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2432322Reviewed-by: Marc Treib <treib@chromium.org>
Commit-Queue: Siyu An <siyua@chromium.org>
Cr-Commit-Position: refs/heads/master@{#811434}

components/autofill/core/browser/webdata/autofill_sync_bridge_util.cc

@@ -499,12 +499,6 @@ bool IsOfferSpecificsValid(const sync_pb::AutofillOfferSpecifics specifics) {
   if (!specifics.has_id())
     return false;
 
-  // A valid offer has a non-empty offer details url and the url must be valid.
-  if (!specifics.has_offer_details_url() ||
-      !GURL(specifics.offer_details_url()).is_valid()) {
-    return false;
-  }
-
   // A valid offer has at least one merchant domain.
   if (specifics.merchant_domain().size() == 0) {
     return false;

components/autofill/core/browser/webdata/autofill_sync_bridge_util_unittest.cc

@@ -316,15 +316,6 @@ TEST_F(AutofillSyncBridgeUtilTest, IsOfferSpecificsValid) {
   // Expects specifics without id to be invalid.
   EXPECT_FALSE(IsOfferSpecificsValid(specifics));
 
-  SetAutofillOfferSpecificsFromOfferData(test::GetCardLinkedOfferData1(),
-                                         &specifics);
-  specifics.set_offer_details_url("invalid url");
-  // Expects specifics with invalid offer_details_url to be invalid.
-  EXPECT_FALSE(IsOfferSpecificsValid(specifics));
-  specifics.clear_offer_details_url();
-  // Expects specifics without offer_details_url to be invalid.
-  EXPECT_FALSE(IsOfferSpecificsValid(specifics));
-
   SetAutofillOfferSpecificsFromOfferData(test::GetCardLinkedOfferData1(),
                                          &specifics);
   specifics.clear_merchant_domain();