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

Issue 847402 link

Starred by 1 user

Issue metadata

Status: WontFix
Owner:
Closed: Aug 6
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug-Regression



Sign in to add a comment

28% regression in blink_perf.bindings at 561543:561709

Project Member Reported by kraynov@chromium.org, May 29 2018

Issue description

See the link to graphs below.
 
Project Member

Comment 1 by 42576172...@developer.gserviceaccount.com, May 29 2018

All graphs for this bug:
  https://chromeperf.appspot.com/group_report?bug_id=847402

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


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

android-one
Cc: b...@chromium.org bajones@chromium.org ikilpatrick@chromium.org ksakamoto@chromium.org danakj@chromium.org qin...@chromium.org petewil@chromium.org pdr@chromium.org mlippautz@chromium.org
Owner: ksakamoto@chromium.org
Status: Assigned (was: Untriaged)
📍 Found significant differences after each of 9 commits.
https://pinpoint-dot-chromeperf.appspot.com/job/149074c6240000

Use the selected download directory for Offline Pages. by petewil@chromium.org
https://chromium.googlesource.com/chromium/src/+/34277063f94f50b86e89f8b17ab0fc345a919f1f
2.004e+04 → 2.011e+04 (+66.98)

[RLS] Remove !RLS code in CLM and background_image_geometry by pdr@chromium.org
https://chromium.googlesource.com/chromium/src/+/ec4da9b334be9e53cd2b1cf32b16af284da63b8f
2.011e+04 → 2.02e+04 (+96.83)

Allow proto database to filter returned entries on key by qinmin@chromium.org
https://chromium.googlesource.com/chromium/src/+/7f0cff2a447c312a5e29925fa88d323732a2c806
2.014e+04 → 1.999e+04 (-155.2)

[css-layout-api] Adds ability to set available sizes on children. by ikilpatrick@chromium.org
https://chromium.googlesource.com/chromium/src/+/9858d76f45443c63645b4beec46882ca18b318fe
2.005e+04 → 2.021e+04 (+155.2)

Do not expand UMA_HISTOGRAM_ENUMERATION in 19 places. by bnc@chromium.org
https://chromium.googlesource.com/chromium/src/+/40e9c2237ae1a0d338ba596bf4c661f07bbc8080
2.026e+04 → 1.994e+04 (-320.8)

Adjust VRShell floor grid to align with users physical floor. by bajones@chromium.org
https://chromium.googlesource.com/chromium/src/+/e44b8a89d4d887f6503ade46df6312526fca4ecc
1.998e+04 → 1.979e+04 (-185.8)

Move LayerTreeHostImpl-dependent code in viz/client/ to cc/. by danakj@chromium.org
https://chromium.googlesource.com/chromium/src/+/54af81af521e95e5b51174c87060fa17e3776415
1.978e+04 → 2.003e+04 (+253.3)

[wrapper-tracing] Allow tracing wrappers from ScriptWrappable by mlippautz@chromium.org
https://chromium.googlesource.com/chromium/src/+/68d0a5f03bfddab66aebe21dfe088cdc855c5e5d
2.003e+04 → 2.508e+04 (+5051)

Add OCSP check for Signed Exchange by ksakamoto@chromium.org
https://chromium.googlesource.com/chromium/src/+/081d150e19d10f6f4bcd83f432ee73f8869f17d6
2.518e+04 → 2.48e+04 (-376.7)

Understanding performance regressions:
  http://g.co/ChromePerformanceRegressions
Owner: mlippautz@chromium.org
My change is for a behind-flag feature and must not affect blink_perf.binding.

Looks like the wrapper-tracing change has most significant change (+5051)?

Cc: -danakj@chromium.org -mlippautz@chromium.org -pdr@chromium.org -qin...@chromium.org -ikilpatrick@chromium.org -bajones@chromium.org -b...@chromium.org -ksakamoto@chromium.org -petewil@chromium.org haraken@chromium.org
Components: Blink>MemoryAllocator>GarbageCollection
Yep, -others
Status: WontFix (was: Assigned)
We decided on purpose to merge the Trace and TraceWrappers methods. This means that for Oilpan we have to do more dispatching without getting any benefits. 

Sign in to add a comment