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

Issue 874921 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Sep 4
Cc:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug-Regression



Sign in to add a comment

1.8%-4% regression in rendering.mobile at 581613:581665

Project Member Reported by tdres...@chromium.org, Aug 16

Issue description

See the link to graphs below.
 
All graphs for this bug:
  https://chromeperf.appspot.com/group_report?bug_id=874921

(For debugging:) Original alerts at time of bug-filing:
  https://chromeperf.appspot.com/group_report?sid=b16f0283dc5f74307345caaed7614978c12994f9d6f5df6de857b58eca6794a0


Bot(s) for this bug's original alert(s):

Android Nexus6 WebView Perf
Cc: digit@google.com mcasas@chromium.org davidben@google.com mtklein@chromium.org svaldez@chromium.org
Owner: mcasas@chromium.org
Status: Assigned (was: Untriaged)
📍 Found significant differences after each of 6 commits.
https://pinpoint-dot-chromeperf.appspot.com/job/14c27c0c640000

android: Disable thinLTO for 32-bit ARM builds. by digit@google.com
https://chromium.googlesource.com/chromium/src/+/069a1be166b09bd5e640d9eebda50bb4a9b4337e
157.2 → 161.2 (+3.918)

Give SSL and SSL_CTX dummy constructor and destructor. by davidben@google.com
https://boringssl.googlesource.com/boringssl/+/c1389f2ce8ed7493c8f1f6144023f5fc233898ae
162.2 → 163.7 (+1.559)

Remove custom extensions support. by davidben@google.com
https://boringssl.googlesource.com/boringssl/+/0a3e07ac1d139eb92b775190d0d1bc88b9da6167
164.1 → No values

Roll src/third_party/boringssl/src c7db3232c..6410e18e9 by svaldez@chromium.org
https://chromium.googlesource.com/chromium/src/+/6537605c43d4c4751c409c65e4ce24e9be5d9d1d
No values → 163.2

rebaseline SK_LEGACY_COLORSPACE_XFORMER_IMPL by mtklein@chromium.org
https://chromium.googlesource.com/chromium/src/+/f591fcf3632ac7706fe483b419f559921d848df8
163.2 → 161.3 (-1.879)

OffscreenCanvas: Remove unused method and member variable by mcasas@chromium.org
https://chromium.googlesource.com/chromium/src/+/2a7b492663d25f505f4d02db2dbcafe5df512e69
160.7 → 162.4 (+1.714)

Understanding performance regressions:
  http://g.co/ChromePerformanceRegressions
Owner: digit@google.com
Don't think my CL is the culprit, it was just a removal of some dead code.

Tentatively assigning to digit@ since the first CL in the list touches 
Android code and the regression was (first) found on a Nexus6.
Cc: -mtklein@chromium.org
ThinLTO was re-enabled after a new clang DEPS roll (see https://bugs.chromium.org/p/chromium/issues/detail?id=872611)

I.e. this should be fixed now.
Status: Fixed (was: Assigned)

Sign in to add a comment