Issue metadata
Sign in to add a comment
|
28kb regression in resource_sizes (MonochromePublic.apk) at 453285:453287 |
||||||||||||||||||||
Issue descriptionLooks like the blamed CL is wrong here. My best guess is two before: http://test-results.appspot.com/revision_range?start=453285&end=453285 Requires further investigation.
,
Mar 2 2017
Started bisect job https://chromeperf.appspot.com/buildbucket_job_status/8986227542167123088
,
Mar 2 2017
=== BISECT JOB RESULTS === NO Perf regression found, tests failed to produce values Bisect Details Configuration: android_nexus7_perf_bisect Benchmark : (MonochromePublic.apk) Metric : MonochromePublic.apk_Specifics/normalized apk size To Run This Test src/tools/perf/run_benchmark -v --browser=android-chromium --output-format=chartjson --upload-results --pageset-repeat=1 --also-run-disabled-tests --story-filter=normalized.apk.size resource_sizes (MonochromePublic.apk) Debug Info https://chromeperf.appspot.com/buildbucket_job_status/8986227542167123088 Is this bisect wrong? https://chromeperf.appspot.com/bad_bisect?try_job_id=5869991741620224 | O O | Visit http://www.chromium.org/developers/speed-infra/perf-bug-faq | X | for more information addressing perf regression bugs. For feedback, | / \ | file a bug with component Speed>Bisection. Thank you!
,
Mar 2 2017
Confirmed, just zooming into the graph shows the the jump came from 453285 https://chromeperf.appspot.com/report?sid=5d55db44beb13bb1e813882381fe82b47aa2934496f2e59840f8ee908da122ca&rev=453285 +simon in case this is interesting (an alert blamed the wrong commit, even when the graph shows it correctly.
,
Mar 2 2017
Looks like the regression is strictly due to more strings being added. Closing.
,
Mar 2 2017
Whoops, didn't mean to change the title. commit link: https://codereview.chromium.org/2703733004
,
May 9 2017
|
|||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||
Comment 1 by agrieve@google.com
, Mar 2 2017