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

Issue 872677 link

Starred by 1 user

Issue metadata

Status: Duplicate
Owner:
Closed: Aug 10
Cc:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug-Regression



Sign in to add a comment

25.4% regression in blink_perf.parser at 580842:580893

Project Member Reported by toyoshim@chromium.org, Aug 9

Issue description

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

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


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

Win 7 Perf
Cc: mlippautz@chromium.org cheyuw@google.com tedchoc@google.com
Owner: tedchoc@google.com
Status: Assigned (was: Untriaged)
📍 Found significant differences after each of 3 commits.
https://pinpoint-dot-chromeperf.appspot.com/job/168ce008640000

Enable traditional PKWARE decryption in zlib/contrib/minizip. by cheyuw@google.com
https://chromium.googlesource.com/chromium/src/+/f3ace98803035b8425d127fb3d874dafe0b9475a
2.388e+04 → 2.421e+04 (+326.5)

[bindings,heap] Prepare PerIsolateData for holding different heap tracer by mlippautz@chromium.org
https://chromium.googlesource.com/chromium/src/+/1186c664d583e43a45f365d4a5291bed032542bd
2.415e+04 → 1.846e+04 (-5688)

Small bug fixes for multi-display support for Android. by tedchoc@google.com
https://chromium.googlesource.com/chromium/src/+/0dd4703c5a3b2efb18cbef5a53d2ffe52ae07fd1
1.846e+04 → 1.682e+04 (-1641)

Understanding performance regressions:
  http://g.co/ChromePerformanceRegressions
Owner: mlippautz@chromium.org
Reassigning to the previous change in the list mlippautz@chromium.org.

My change was targeted to multi-display android device and the code is currently disabled in the browser by:
https://chromium.googlesource.com/chromium/src/+/580bc96a5dcb4b1c307304e2bbd32ffa9a777d49/chrome/android/java/src/org/chromium/chrome/browser/multiwindow/MultiWindowUtils.java#50

One of the few cases where I can definitely say it wasn't that change.
Mergedinto: 872581
Status: Duplicate (was: Assigned)

Sign in to add a comment