New issue
Advanced search Search tips

Issue 889164 link

Starred by 1 user

Issue metadata

Status: WontFix
Owner: ----
Closed: Sep 28
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug-Regression



Sign in to add a comment

12.4% regression in blink_perf.shadow_dom at 592367:592385

Project Member Reported by lanwei@chromium.org, Sep 25

Issue description

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

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


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

linux-perf

blink_perf.shadow_dom - Benchmark documentation link:
  https://bit.ly/blink-perf-benchmarks
Cc: cbiesin...@chromium.org f...@opera.com
Owner: cbiesin...@chromium.org
Status: Assigned (was: Untriaged)
📍 Found significant differences after each of 2 commits.
https://pinpoint-dot-chromeperf.appspot.com/job/1549b18f640000

Use DEFINE_STATIC_LOCAL in CSSProperty::GetPropertyNameAtomicString by fs@opera.com
https://chromium.googlesource.com/chromium/src/+/f7ef1c942101a6560f1d5487a3688d13eaa15be4
9.25 → 9.61 (+0.36)

[layoutng] Store the constraint space as a pointer on LayoutBlock by cbiesinger@chromium.org
https://chromium.googlesource.com/chromium/src/+/f0252116eab5af1a2fba125b1aebbfed08de96b7
9.61 → 10.37 (+0.76)

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

Benchmark documentation link:
  https://bit.ly/blink-perf-benchmarks
Owner: ----
Status: Available (was: Assigned)
That doesn't make sense... this code doesn't run during that test...
This benchmark seems to alternate between fast and slow and in this case it got fast for a day and went back to being slow. I can't see anything in the CLs in either range that looks suspicious. This doesn't seem actionable.
Status: WontFix (was: Available)

Sign in to add a comment