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

Issue 676612 link

Starred by 3 users

Issue metadata

Status: Assigned
Owner:
Cc:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: Bug-Regression



Sign in to add a comment

loading.mobile failure on chromium.perf at 2e3d43f9e0aa8fb1446f907a5f93d2489bf671b3 on Android Nexus6 WebView Perf (2)

Project Member Reported by picksi@chromium.org, Dec 22 2016

Issue description

loading.mobile started failing on build #1130:
https://build.chromium.org/p/chromium.perf/builders/Android%20Nexus6%20WebView%20Perf%20%282%29/builds/1130

It is not clear which CL is to blame, but CC'ing haraken@ on this bug as he has checked in several large CLs with this build. Apologies if I'm falsely accusing you :)

 
Cc: ksakamoto@chromium.org eyaich@chromium.org kouhei@chromium.org
This is really consistently failing on webview, should we disable it for webview?
Also, eyaich: why wasn't this addressed in the fixit?
Cc: -ksakamoto@chromium.org
Owner: ksakamoto@chromium.org
Status: Assigned (was: Untriaged)
Obviously my https://codereview.chromium.org/2564773002 should be blamed :)
That CL enabled loading.mobile on WebView bots. Nexus5X WebView looks OK, but it seems N6 still have some problem.

Let me disable it for now.

Project Member

Comment 4 by bugdroid1@chromium.org, Jan 6 2017

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

commit ddcf54461471ed503e3bd02f0d9563108ef1913e
Author: ksakamoto <ksakamoto@chromium.org>
Date: Fri Jan 06 04:05:54 2017

Disable loading.mobile on Nexus6 WebView

https://crrev.com/2558743002 fixed loading.mobile on Nexus5 WebView and
page_cycler_v2 on Nexus5/Nexus6 WebView, but loading.mobile on Nexus6
WebView is still failing.

BUG=676612
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.perf:linux_perf_cq;master.tryserver.chromium.perf:mac_retina_perf_cq;master.tryserver.chromium.perf:winx64_10_perf_cq

Review-Url: https://codereview.chromium.org/2613153005
Cr-Commit-Position: refs/heads/master@{#441868}

[modify] https://crrev.com/ddcf54461471ed503e3bd02f0d9563108ef1913e/tools/perf/benchmarks/loading.py

Cc: mikec...@chromium.org
Not sure why this wasn't addressed in fixit, cc'ing mikecase@ who was looking at all android shard 2.
I mentioned this failure yesterday during one of the meetings. Dont think loading.mobile ever passed on the WebView bot. Someone mentioned that we probably just need to increase the timeout for that test.
Project Member

Comment 7 by bugdroid1@chromium.org, Aug 10

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

commit 9a1e1f58d1be00e6f1c17f48a5d3d332c725d68e
Author: Ned Nguyen <nednguyen@google.com>
Date: Fri Aug 10 03:50:03 2018

Reenable loading.mobile on Nexus6 WebView

Capacity issue on Nexus6 Webview has been resolved, so this may be ok to reenable
the benchmarks on this platform now

TBR=ksakamoto@chromium.org
NOTRY=true

Bug: 676612
Change-Id: I8f6a93767d47f94ba66a8a9df43b8d3a0b03b951
Reviewed-on: https://chromium-review.googlesource.com/1170422
Reviewed-by: Ned Nguyen <nednguyen@google.com>
Commit-Queue: Ned Nguyen <nednguyen@google.com>
Cr-Commit-Position: refs/heads/master@{#582039}
[modify] https://crrev.com/9a1e1f58d1be00e6f1c17f48a5d3d332c725d68e/tools/perf/expectations.config

Project Member

Comment 8 by bugdroid1@chromium.org, Aug 10

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

commit e2692a46a03038298948882020ad3bb3548e99ca
Author: Ned Nguyen <nednguyen@google.com>
Date: Fri Aug 10 15:20:33 2018

Disable flaky mobile.loading test on Nexus 6 Webview

TBR=ksakamoto@chromium.org
NOTRY=true

Bug: 676612
Change-Id: Ia70720a2d00c4c7560736db33d8bbe7b418a87e5
Reviewed-on: https://chromium-review.googlesource.com/1170593
Reviewed-by: Ned Nguyen <nednguyen@google.com>
Commit-Queue: Ned Nguyen <nednguyen@google.com>
Cr-Commit-Position: refs/heads/master@{#582170}
[modify] https://crrev.com/e2692a46a03038298948882020ad3bb3548e99ca/tools/perf/expectations.config

Project Member

Comment 9 by bugdroid1@chromium.org, Aug 19

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

commit 71983b8b0bf9ada8cfef156305139bc0d5197c65
Author: Ned Nguyen <nednguyen@google.com>
Date: Sun Aug 19 10:04:32 2018

Disable more loading mobile tests flaky on Nexus6_Webview

Bug:676612
Change-Id: Ia884ea757f60c625442a7564955f4f0eea8a1164
TBR=ksakamoto@chromium.org, kouhei@chromium.org

NOTRY=true

Change-Id: Ia884ea757f60c625442a7564955f4f0eea8a1164
Reviewed-on: https://chromium-review.googlesource.com/1180213
Commit-Queue: Ned Nguyen <nednguyen@google.com>
Reviewed-by: Ned Nguyen <nednguyen@google.com>
Cr-Commit-Position: refs/heads/master@{#584332}
[modify] https://crrev.com/71983b8b0bf9ada8cfef156305139bc0d5197c65/tools/perf/expectations.config

Sign in to add a comment