Summary: | No need to declare JSC::LLInt::Data as friend class of WTF::StringImpl | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Xan Lopez <xan.lopez> | ||||
Component: | JavaScriptCore | Assignee: | Nobody <webkit-unassigned> | ||||
Status: | REOPENED --- | ||||||
Severity: | Normal | CC: | benjamin, cmarcelo, commit-queue, ggaren, kling, xan.lopez | ||||
Priority: | P2 | ||||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
Attachments: |
|
Description
Xan Lopez
2013-05-04 08:14:47 PDT
Created attachment 200528 [details]
stringimplllintdata.diff
The commit-queue encountered the following flaky tests while processing attachment 200528 [details]: fast/frames/crash-remove-iframe-during-object-beforeload.html bug 115322 (author: zalan@apple.com) The commit-queue is continuing to process your patch. Comment on attachment 200528 [details] stringimplllintdata.diff Clearing flags on attachment: 200528 Committed r149563: <http://trac.webkit.org/changeset/149563> All reviewed patches have been landed. Closing bug. Reverted r149563 for reason: Broke LLInt build. Committed r149568: <http://trac.webkit.org/changeset/149568> (In reply to comment #5) > Reverted r149563 for reason: > > Broke LLInt build. > > Committed r149568: <http://trac.webkit.org/changeset/149568> Right, there was an ASSERT accessing a private flag, missed that (like EWS). We can change it to use the public getter, will upload a new patch later when I can verify with a debug build. Can someone svn blame to find out when this interface was added, and when its use was removed? If the use was truly removed, we can remove the ASSERT. Otherwise, we probably can't remove the interface. |