New issue
Advanced search Search tips

Issue 913125 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Dec 12
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Mac
Pri: 1
Type: Bug



Sign in to add a comment

Record Wallet Sync Transport opt-in pref when user accepts upstream offer.

Project Member Reported by se...@chromium.org, Dec 7

Issue description

^^^
 
Labels: Merge-Request-72
Project Member

Comment 3 by sheriffbot@chromium.org, Dec 12

Labels: -Merge-Request-72 Hotlist-Merge-Approved Merge-Approved-72
Your change meets the bar and is auto-approved for M72. Please go ahead and merge the CL to branch 3626 manually. Please contact milestone owner if you have questions.
Owners: govind@(Android), kariahda@(iOS), djmm@(ChromeOS), abdulsyed@(Desktop)

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Project Member

Comment 4 by bugdroid1@chromium.org, Dec 12

Labels: -merge-approved-72 merge-merged-3626
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/4afe52d43e5f7143a72a9297da25034a68b89c7f

commit 4afe52d43e5f7143a72a9297da25034a68b89c7f
Author: sebsg <sebsg@chromium.org>
Date: Wed Dec 12 20:19:32 2018

Merge [AF]Record Wallet Sync Transport opt-in on upstream acceptance.

TBR=sebsg@chromium.org

(cherry picked from commit deba185250c1a680c462eb091d69135a6c7890af)

Bug:  913125 
Change-Id: I638007991590656b7374d3838c9b140f0d48e1f5
Reviewed-on: https://chromium-review.googlesource.com/c/1368953
Commit-Queue: Sebastien Seguin-Gagnon <sebsg@chromium.org>
Reviewed-by: Tommy Martino <tmartino@chromium.org>
Cr-Original-Commit-Position: refs/heads/master@{#615327}
Reviewed-on: https://chromium-review.googlesource.com/c/1374434
Reviewed-by: Sebastien Seguin-Gagnon <sebsg@chromium.org>
Cr-Commit-Position: refs/branch-heads/3626@{#301}
Cr-Branched-From: d897fb137fbaaa9355c0c93124cc048824eb1e65-refs/heads/master@{#612437}
[modify] https://crrev.com/4afe52d43e5f7143a72a9297da25034a68b89c7f/components/autofill/core/browser/credit_card_save_manager.cc
[modify] https://crrev.com/4afe52d43e5f7143a72a9297da25034a68b89c7f/components/autofill/core/browser/credit_card_save_manager_unittest.cc
[modify] https://crrev.com/4afe52d43e5f7143a72a9297da25034a68b89c7f/components/autofill/core/browser/personal_data_manager.cc
[modify] https://crrev.com/4afe52d43e5f7143a72a9297da25034a68b89c7f/components/autofill/core/browser/personal_data_manager.h
[modify] https://crrev.com/4afe52d43e5f7143a72a9297da25034a68b89c7f/components/autofill/core/browser/personal_data_manager_unittest.cc

Status: Fixed (was: Started)
Labels: Merge-Merged-72-3626
The following revision refers to this bug: 
https://chromium.googlesource.com/chromium/src.git/+/4afe52d43e5f7143a72a9297da25034a68b89c7f

Commit: 4afe52d43e5f7143a72a9297da25034a68b89c7f
Author: sebsg@chromium.org
Commiter: sebsg@chromium.org
Date: 2018-12-12 20:19:32 +0000 UTC

Merge [AF]Record Wallet Sync Transport opt-in on upstream acceptance.

TBR=sebsg@chromium.org

(cherry picked from commit deba185250c1a680c462eb091d69135a6c7890af)

Bug:  913125 
Change-Id: I638007991590656b7374d3838c9b140f0d48e1f5
Reviewed-on: https://chromium-review.googlesource.com/c/1368953
Commit-Queue: Sebastien Seguin-Gagnon <sebsg@chromium.org>
Reviewed-by: Tommy Martino <tmartino@chromium.org>
Cr-Original-Commit-Position: refs/heads/master@{#615327}
Reviewed-on: https://chromium-review.googlesource.com/c/1374434
Reviewed-by: Sebastien Seguin-Gagnon <sebsg@chromium.org>
Cr-Commit-Position: refs/branch-heads/3626@{#301}
Cr-Branched-From: d897fb137fbaaa9355c0c93124cc048824eb1e65-refs/heads/master@{#612437}

Sign in to add a comment