103791 – [v8?] DOM traversal test became bimodal

Bug 103791 - [v8?] DOM traversal test became bimodal

Summary: [v8?] DOM traversal test became bimodal

Status:	RESOLVED WONTFIX

Alias:	None

Product:	WebKit
Classification:	Unclassified
Component:	DOM (show other bugs)
Version:	528+ (Nightly build)
Hardware:	Unspecified Unspecified

Importance:	P2 Normal
Assignee:	Nobody

URL:
Keywords:

Depends on:
Blocks:

Reported:	2012-11-30 17:02 PST by Ojan Vafai
Modified:	2013-09-12 22:37 PDT (History)
CC List:	3 users (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description Ojan Vafai 2012-11-30 17:02:10 PST

Range: http://trac.webkit.org/log/?verbose=on&rev=136017&stop_rev=136010

Could it possibly be http://trac.webkit.org/changeset/136015/ ?

http://build.chromium.org/f/chromium/perf/mac-release-10.6-webkit-latest/dom_perf/report.html?rev=170457&graph=DOMDivWalk&history=50

http://build.chromium.org/f/chromium/perf/mac-release-10.6-webkit-latest/dom_perf/report.html?rev=170457&graph=DOMDivWalk&history=50

http://build.chromium.org/f/chromium/perf/mac-release-10.6-webkit-latest/dromaeo_domcoretraverse/report.html?rev=170457&graph=dom_traverse_nextSibling&history=50

Comment 1 Ojan Vafai 2012-11-30 17:07:28 PST

Actually, it looks like it might be an actual regression.

http://build.chromium.org/f/chromium/perf/mac-release-10.6/dromaeo_domcoretraverse/report.html?history=90&rev=170138&graph=dom_traverse_previousSibling

Comment 2 Adam Barth 2012-12-01 16:15:12 PST

Ok.  Let's roll http://trac.webkit.org/changeset/136015/ out.  It adds an instruction into a host code path, which is why I made the original change.

Comment 3 Adam Barth 2012-12-01 16:35:31 PST

Rolled out in http://trac.webkit.org/changeset/136313.  Let's see if that heals the bots.

Comment 4 Anders Carlsson 2013-09-12 22:37:32 PDT

V8 is gone.