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

Issue 915229 link

Starred by 1 user
Status: Assigned
Owner:
jkrcal@chromium.org
Cc:
se...@chromium.org
mastiz@chromium.org
█ tschumann@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

[AF Wallet] In sync integration tests, PDM sometimes fails to convert a server profile into a local one

Project Member Reported by jkrcal@chromium.org, Dec 14 
Rare flakiness that always PASSes in the next try, so far. Never managed to repro locally. I need more DLOGs to debug the issue on the waterfall.
Project Member

Comment 1 by bugdroid1@chromium.org, Dec 14 

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

commit fc4148c5309d3e1d0a5af2f3a024d13d6001b058
Author: Jan Krcal <jkrcal@chromium.org>
Date: Fri Dec 14 16:02:20 2018

[AF Wallet] Log has_converted() in AutofillProfile::operator<<

Bug: 915229
Change-Id: I62bffaca4e7d335d8ad60bc4fa90cb71b2553ee3
Reviewed-on: https://chromium-review.googlesource.com/c/1377421
Commit-Queue: Jan Krcal <jkrcal@chromium.org>
Commit-Queue: Sebastien Seguin-Gagnon <sebsg@chromium.org>
Reviewed-by: Sebastien Seguin-Gagnon <sebsg@chromium.org>
Cr-Commit-Position: refs/heads/master@{#616697}
[modify] https://crrev.com/fc4148c5309d3e1d0a5af2f3a024d13d6001b058/components/autofill/core/browser/autofill_profile.cc
Project Member

Comment 2 by bugdroid1@chromium.org, Dec 14 

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

commit 643b8b4cf95936652eab0651ce115e0ca5927c3b
Author: Jan Krcal <jkrcal@chromium.org>
Date: Fri Dec 14 19:56:14 2018

[AF] Debug logs in wallet_metadata sync for flaky integration test

To be reverted as soon as the issue is found.

Bug: 915229
Change-Id: I81e9d1f7fa23eee957df8256fa858e210af620c8
Reviewed-on: https://chromium-review.googlesource.com/c/1377438
Commit-Queue: Jan Krcal <jkrcal@chromium.org>
Reviewed-by: Sebastien Seguin-Gagnon <sebsg@chromium.org>
Cr-Commit-Position: refs/heads/master@{#616798}
[modify] https://crrev.com/643b8b4cf95936652eab0651ce115e0ca5927c3b/components/autofill/core/browser/personal_data_manager.cc
[modify] https://crrev.com/643b8b4cf95936652eab0651ce115e0ca5927c3b/components/autofill/core/browser/webdata/autofill_wallet_metadata_syncable_service.cc
Project Member

Comment 3 by bugdroid1@chromium.org, Dec 17 

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

commit aff14da954903f2379381d096ce8793967186739
Author: Jan Krcal <jkrcal@chromium.org>
Date: Mon Dec 17 10:52:26 2018

[AF] Debug logs in wallet_metadata sync for flaky integration test, part 2

To be reverted as soon as this is confirmed as the source of the issue.

TBR=sebsg@chromium.org

Bug: 915229
Change-Id: I59925e538ea5547e3bfd089ed5bb4f1772dbdcc0
Reviewed-on: https://chromium-review.googlesource.com/c/1379759
Reviewed-by: Jan Krcal <jkrcal@chromium.org>
Commit-Queue: Jan Krcal <jkrcal@chromium.org>
Cr-Commit-Position: refs/heads/master@{#617080}
[modify] https://crrev.com/aff14da954903f2379381d096ce8793967186739/components/autofill/core/browser/personal_data_manager.cc
►

Sign in to add a comment