New issue
Advanced search Search tips

Issue 799028 link

Starred by 2 users

Issue metadata

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



Sign in to add a comment

PaymentRequestShippingAddressEditorTest.EnterAcceleratorSyncData in browser_tests failing on chromium.memory/Linux ChromiumOS MSan Tests

Project Member Reported by sheriff-...@appspot.gserviceaccount.com, Jan 4 2018

Issue description

Filed by sheriff-o-matic@appspot.gserviceaccount.com on behalf of battre@google.com

PaymentRequestShippingAddressEditorTest.EnterAcceleratorSyncData in browser_tests failing on chromium.memory/Linux ChromiumOS MSan Tests

Builders failed on: 
- Linux ChromiumOS MSan Tests: 
  https://build.chromium.org/p/chromium.memory/builders/Linux%20ChromiumOS%20MSan%20Tests


 
Components: UI>Browser>Payments Tests>Disabled
Owner: rouslan@chromium.org
Status: Assigned (was: Available)
This failed multiple times in a row.

https://logs.chromium.org/v/?s=chromium%2Fbb%2Fchromium.memory%2FLinux_ChromiumOS_MSan_Tests%2F5055%2F%2B%2Frecipes%2Fsteps%2Fbrowser_tests%2F0%2Flogs%2FPaymentRequestShippingAddressEditorTest.EnterAcceleratorSyncData%2F0

[ RUN      ] PaymentRequestShippingAddressEditorTest.EnterAcceleratorSyncData
[27777:27777:0104/032414.006913:WARNING:chrome_browser_main_chromeos.cc(662)] Running as stub user with profile dir: test-user
[27777:27777:0104/032417.062746:ERROR:wallpaper_controller.cc(1201)] User is ephemeral or guest! Fallback to default wallpaper.
[27777:28056:0104/032417.606685:WARNING:embedded_test_server.cc(228)] Request not handled. Returning 404: /favicon.ico
[27777:27777:0104/032417.655703:INFO:CONSOLE(27)] "Card issuer network ("amex", "diners", "discover", "jcb", "mastercard", "mir", "unionpay", "visa") as payment method is deprecated and will be removed in M64, around January 2018. Please use payment method name "basic-card" with issuer network in the "supportedNetworks" field instead. See https://www.chromestatus.com/features/5725727580225536 for more details.", source: https://a.com:53906/dynamic_shipping.js (27)
../../chrome/browser/ui/views/autofill/dialog_event_waiter.h:66: Failure
Expected equality of these values:
  events_.front()
    Which is: 21
  event
    Which is: 1
[27777:27777:0104/032418.365919:INFO:chrome_cryptauth_service.cc(241)] Refresh token not yet available; waiting before starting CryptAuth managers.
[  FAILED  ] PaymentRequestShippingAddressEditorTest.EnterAcceleratorSyncData, where TypeParam =  and GetParam() =  (4991 ms)

Failures happen particularly often on MSAN bots but also on others. E.g. 
https://ci.chromium.org/buildbot/chromium.mac/Mac10.12%20Tests/8799
https://chromium-swarm.appspot.com/task?id=3ad72986b68f6e10&refresh=10&show_raw=1

Assigning to rouslan due to https://chromium-review.googlesource.com/c/chromium/src/+/814575
 Issue 799027  has been merged into this issue.
Project Member

Comment 3 by bugdroid1@chromium.org, Jan 4 2018

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

commit 3e06fd313e318894289ed26435379077de7f7754
Author: Dominic Battre <battre@chromium.org>
Date: Thu Jan 04 13:35:22 2018

Disabled flaky PaymentRequestShippingAddressEditorTest.EnterAcceleratorSyncData

TBR=rouslan@chromium.org

Bug: 799028
Change-Id: I12904a56860ce6e185f6f9c20d2ebb48b43ab54e
Reviewed-on: https://chromium-review.googlesource.com/850552
Reviewed-by: Dominic Battré <battre@chromium.org>
Commit-Queue: Dominic Battré <battre@chromium.org>
Cr-Commit-Position: refs/heads/master@{#526974}
[modify] https://crrev.com/3e06fd313e318894289ed26435379077de7f7754/chrome/browser/ui/views/payments/shipping_address_editor_view_controller_browsertest.cc

Cc: linds...@chromium.org rouslan@chromium.org
Owner: ma...@chromium.org
@mathp is there someone else that can review this since rouslan is ooto?

Comment 5 by ma...@chromium.org, Jan 19 2018

Owner: anthonyvd@chromium.org
Hoping Anthony can take a look
Components: Tests>Disabled
Labels: Test-Disabled
It looks like the test fails because we expect the spinner to be hidden but the dialog instead closes altogether. I don't know why this would be happening, and I can't repro at all so this will need more investigation.
Cc: anthonyvd@chromium.org
Owner: rouslan@chromium.org
Rouslan, can you PTAL?
Nothing in the code immediately explains why the dialog closes. Need more investigation.
Labels: Hotlist-DisableReview
I need this information  it was me that mashed that comment William  harper
thank u

Sign in to add a comment