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

Issue 809541 link

Starred by 1 user
Status: Fixed
Owner:
samans@chromium.org
Closed: Feb 2018
Cc:
samans@chromium.org
█ fsam...@chromium.org
tsepez@chromium.org
█ kraynov@chromium.org
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug-Regression



Sign in to add a comment

119.5%-129.2% regression in cc_perftests at 533806:533981

Project Member Reported by kraynov@chromium.org, Feb 6 2018 
See the link to graphs below.
Project Member

Comment 1 by 42576172...@developer.gserviceaccount.com, Feb 6 2018 

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

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


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

android-nexus5
android-nexus7v2
Project Member

Comment 3 by 42576172...@developer.gserviceaccount.com, Feb 7 2018

Cc: tsepez@chromium.org fsam...@chromium.org samans@chromium.org
Owner: samans@chromium.org
Status: Assigned (was: Untriaged)
📍 Found a significant difference after 1 commit.
https://pinpoint-dot-chromeperf.appspot.com/job/15d36981840000

viz: Disallow CompositorFrames with empty size by samans@chromium.org
chromium @ 99eea10d0d184ffb5367ff050b27b03b58d2cb4a

Understanding performance regressions:
  http://g.co/ChromePerformanceRegressions
Project Member

Comment 4 by bugdroid1@chromium.org, Feb 8 2018 

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

commit 516b9791ff497d54ad2d99d9628165a36ec77240
Author: Saman Sami <samans@chromium.org>
Date: Thu Feb 08 16:56:30 2018

Fix empty CompositorFrame in cc_serialization_perftest

Bug:  809541 
Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.android:android_optional_gpu_tests_rel
Change-Id: Ic74d0d1ae5eca2a5d7dd4be375d5aa4a2856f067
Reviewed-on: https://chromium-review.googlesource.com/908718
Reviewed-by: Fady Samuel <fsamuel@chromium.org>
Commit-Queue: Saman Sami <samans@chromium.org>
Cr-Commit-Position: refs/heads/master@{#535408}
[modify] https://crrev.com/516b9791ff497d54ad2d99d9628165a36ec77240/cc/ipc/cc_serialization_perftest.cc

Comment 5 by samans@chromium.org, Feb 9 2018

Status: Fixed (was: Assigned)
►

Sign in to add a comment