New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 876300 link

Starred by 1 user

Issue metadata

Status: Assigned
Owner:
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug

Blocking:
issue 876301



Sign in to add a comment

Write integration test to test that Wallet Bridge reads correct metadata

Project Member Reported by feuunk@google.com, Aug 21

Issue description

The CL:
https://chromium-review.googlesource.com/c/chromium/src/+/1183437

Fixes Wallet Bridge to read wallet metadata instead of Address metadata. Before launching, we should make sure that we have integration test coverage for this.
 
Blocking: 876301
Project Member

Comment 2 by bugdroid1@chromium.org, Aug 21

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/7e5581fb2aabac5970f6bf8ea1012ad2487a4b15

commit 7e5581fb2aabac5970f6bf8ea1012ad2487a4b15
Author: Florian Uunk <feuunk@chromium.org>
Date: Tue Aug 21 15:06:20 2018

Use the wallet datatype to read wallet metadata

Fixing a bug where the wallet data bridge was reading address metadata.

Change-Id: I1a8dc41a59b4bb8f5855eb7c5f0fa81a3611a419
Bug: 876300
Reviewed-on: https://chromium-review.googlesource.com/1183437
Commit-Queue: Florian Uunk <feuunk@chromium.org>
Reviewed-by: Tim Schumann <tschumann@chromium.org>
Cr-Commit-Position: refs/heads/master@{#584743}
[modify] https://crrev.com/7e5581fb2aabac5970f6bf8ea1012ad2487a4b15/components/autofill/core/browser/webdata/autofill_wallet_sync_bridge.cc
[modify] https://crrev.com/7e5581fb2aabac5970f6bf8ea1012ad2487a4b15/components/autofill/core/browser/webdata/autofill_wallet_sync_bridge_unittest.cc

Sign in to add a comment