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

Issue 747046 link

Starred by 1 user

Issue metadata

Status: WontFix
Owner:
Closed: Jul 2017
Cc:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug-Regression



Sign in to add a comment

12.1% regression in media.tough_video_cases_tbmv2 at 487396:487494

Project Member Reported by hubbe@google.com, Jul 20 2017

Issue description

See the link to graphs below.
 
Project Member

Comment 1 by 42576172...@developer.gserviceaccount.com, Jul 20 2017

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

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


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

chromium-rel-win7-dual
Project Member

Comment 3 by 42576172...@developer.gserviceaccount.com, Jul 21 2017

Cc: yangguo@chromium.org
Owner: yangguo@chromium.org

=== Auto-CCing suspected CL author yangguo@chromium.org ===

Hi yangguo@chromium.org, the bisect results pointed to your CL, please take a look at the
results.


=== BISECT JOB RESULTS ===
Perf regression found with culprit

Suspected Commit
  Author : Yang Guo
  Commit : a2ab1353f6708b44d305fdd9fe65a6d29b95c6d6
  Date   : Tue Jul 18 10:48:43 2017
  Subject: [snapshot] Rehash strings after deserialization.

Bisect Details
  Configuration: win_perf_bisect
  Benchmark    : media.tough_video_cases_tbmv2
  Metric       : memory:chrome:all_processes:reported_by_chrome:v8:effective_size_avg/video.html?src_tulip2.wav_type_audio
  Change       : 11.90% | 4407296.0 -> 4931584.0

Revision                           Result              N
chromium@487395                    4407296 +- 0.0      6      good
chromium@487445                    4407296 +- 0.0      6      good
chromium@487470                    4407296 +- 0.0      6      good
chromium@487476                    4407296 +- 0.0      6      good
chromium@487479                    4407296 +- 0.0      6      good
chromium@487479,v8@a2ab1353f6      4931584 +- 0.0      6      bad       <--
chromium@487479,v8@afcc8bb200      4931584 +- 0.0      6      bad
chromium@487479,v8@10c6fb566f      4931584 +- 0.0      6      bad
chromium@487480                    4931584 +- 0.0      6      bad
chromium@487481                    4931584 +- 0.0      6      bad
chromium@487482                    4931584 +- 0.0      6      bad
chromium@487494                    4931584 +- 0.0      6      bad

To Run This Test
  src/tools/perf/run_benchmark -v --browser=release --output-format=chartjson --upload-results --pageset-repeat=1 --also-run-disabled-tests --story-filter=video.html.src.tulip2.wav.type.audio media.tough_video_cases_tbmv2

More information on addressing performance regressions:
  http://g.co/ChromePerformanceRegressions

Debug information about this bisect:
  https://chromeperf.appspot.com/buildbucket_job_status/8973529277876875168


For feedback, file a bug with component Speed>Bisection
Status: WontFix (was: Assigned)
This is a fix for a security issue, and I'm working on improving it. So for the time being I don't see any options.
Project Member

Comment 5 by 42576172...@developer.gserviceaccount.com, Jul 21 2017

Cc: hubbe@chromium.org
 Issue 747049  has been merged into this issue.

Sign in to add a comment