New issue
Advanced search Search tips

Issue 602757 link

Starred by 1 user
Status: Fixed
Owner:
e...@chromium.org
Closed: Apr 2016
EstimatedDays: ----
NextAction: ----
OS: Mac
Pri: 2
Type: Bug-Regression



Sign in to add a comment

20.8% regression in blink_perf.layout at 386017:386041

Project Member Reported by m...@chromium.org, Apr 12 2016

Comment 1 by m...@chromium.org, Apr 12 2016 

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

Original alerts at time of bug-filing:
  https://chromeperf.appspot.com/group_report?keys=agxzfmNocm9tZXBlcmZyFAsSB0Fub21hbHkYgICg-NSptAoM


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

chromium-rel-mac11

Comment 2 by m...@chromium.org, Apr 12 2016

Labels: OS-Mac
Owner: e...@chromium.org
Based on looking at the commits in the possible rev-range, this seems to most likely be the culprit (the only WebKit code change related to text layout performance):

commit	63f4a420e42667f2167b7fcf4a9ac8e10f66e792	
author	eae <eae@chromium.org>	Fri Apr 08 09:02:53 2016
committer	Commit bot <commit-bot@chromium.org>	Fri Apr 08 09:04:32 2016
Segment strings on dot and comma in addition to space and tab

Comment 3 by e...@chromium.org, Apr 12 2016

Status: Started (was: Assigned)
Confirmed. It didn't have the intended effect, will revert after hours as it'll require a lot of rebaselines.

Comment 4 by e...@chromium.org, Apr 19 2016

Status: Fixed (was: Started)

Comment 6 by tin...@google.com, Apr 20 2016

Labels: -Merge-Request-51 Merge-Approved-51 Hotlist-Merge-Approved
Your change meets the bar and is auto-approved for M51 (branch: 2704)

Comment 7 by e...@chromium.org, Apr 21 2016

Labels: -Merge-Approved-51 Merge-merged-51
►

Sign in to add a comment