New issue
Advanced search Search tips

Issue 788651 link

Starred by 1 user
Status: Fixed
Owner:
csharrison@chromium.org
Closed: Mar 2018
Cc:
gov...@chromium.org
nyerramilli@chromium.org
pbomm...@chromium.org
style-bugs@google.com
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows
Pri: 2
Type: Bug



Sign in to add a comment

PageLoadMetricsBrowserTest.CSSTiming flaky on Linux

Project Member Reported by grt@chromium.org, Nov 27 2017 
PageLoad.CSSTiming.Parse.BeforeFirstContentfulPaint isn't logged sometimes:

[ RUN      ] PageLoadMetricsBrowserTest.CSSTiming
Xlib:  extension "RANDR" missing on display ":99".
[24649:24749:1125/144050.641181:ERROR:bus.cc(395)] Failed to connect to the bus: Could not parse server address: Unknown address type (examples of valid types are "tcp" and on UNIX "unix")
[24649:24649:1125/144050.644575:WARNING:password_store_factory.cc(241)] Using basic (unencrypted) store for password storage. See https://chromium.googlesource.com/chromium/src/+/master/docs/linux_password_storage.md for more information about password storage options.
[24649:24777:1125/144050.861963:WARNING:embedded_test_server.cc(228)] Request not handled. Returning 404: /favicon.ico
../../base/test/histogram_tester.cc:72: Failure
      Expected: count
      Which is: 1
To be equal to: 0
Histogram "PageLoad.CSSTiming.Parse.BeforeFirstContentfulPaint" does not exist.
[24649:24649:1125/144052.861868:INFO:chrome_cryptauth_service.cc(222)] Profile is not authenticated yet; waiting before starting CryptAuth managers.
[  FAILED  ] PageLoadMetricsBrowserTest.CSSTiming, where TypeParam =  and GetParam() =  (2827 ms)
Project Member

Comment 1 by bugdroid1@chromium.org, Nov 27 2017 

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

commit fde50d4484f7f30899e8a3426a98c0782281b07d
Author: Greg Thompson <grt@chromium.org>
Date: Mon Nov 27 09:38:25 2017

Disable PageLoadMetricsBrowserTest.CSSTiming on Linux due to flakes.

BUG= 788651 
TBR=grt@chromium.org
NOTRY=true

Change-Id: Iaf789d0140c2c7045a20c113f939e3c298899653
Reviewed-on: https://chromium-review.googlesource.com/789879
Reviewed-by: Greg Thompson <grt@chromium.org>
Commit-Queue: Greg Thompson <grt@chromium.org>
Cr-Commit-Position: refs/heads/master@{#519254}
[modify] https://crrev.com/fde50d4484f7f30899e8a3426a98c0782281b07d/chrome/browser/page_load_metrics/page_load_metrics_browsertest.cc

Comment 2 by dstockwell@google.com, Nov 27 2017

Labels: Test-Functional

Comment 3 by dpranke@chromium.org, Jan 24 2018

Components: Tests>Disabled

Comment 4 by dpranke@chromium.org, Jan 24 2018

Labels: Test-Disabled

Comment 5 by jmukthavaram@chromium.org, Mar 28 2018

Cc: pbomm...@chromium.org gov...@chromium.org nyerramilli@chromium.org
Labels: Build OS-Windows
Just to update:

PageLoadMetricsBrowserTest.CSSTiming -Win clang Flaky issue on official desktop builder.

Link to the builder:
------------------
https://uberchromegw.corp.google.com/i/official.desktop/builders/win-clang/builds/852


Link to Error log:
----------------
https://logs.chromium.org/v/?s=chrome%2Fbb%2Fofficial.desktop%2Fwin-clang%2F852%2F%2B%2Frecipes%2Fsteps%2Fbrowser_tests%2F0%2Fstdout

csharrison@,Could you please take a look into this issue.

Thanks..!

Comment 6 by csharrison@chromium.org, Mar 28 2018 

This test is flaky by design, and isn't critical. Let's just delete it.

Comment 8 by csharrison@chromium.org, Mar 28 2018

Status: Fixed (was: Assigned)
►

Sign in to add a comment