Bug 30730

Summary: [Qt] Enable YARR when YARR_JIT is enabled
Product: WebKit Reporter: Afonso Costa <arabelo>
Component: JavaScriptCoreAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue, loki, tonikitoo, vestbo
Priority: P2 Keywords: Qt
Version: 528+ (Nightly build)   
Hardware: Other   
OS: Linux   
Attachments:
Description Flags
Proposed patch
none
Proposed patch - v2 none

Afonso Costa
Reported 2009-10-23 15:30:26 PDT
Created attachment 41752 [details] Proposed patch When enabling or disabling JIT using JAVASCRIPTCORE_JIT, the ENABLE_YARR should be toggled also.
Attachments
Proposed patch (524 bytes, patch)
2009-10-23 15:30 PDT, Afonso Costa 		no flags
DetailsFormatted DiffDiff
Proposed patch - v2 (1.80 KB, patch)
2009-10-26 09:15 PDT, Afonso Costa 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Gabor Loki
Comment 1 2009-10-26 07:29:53 PDT
> Created an attachment (id=41752) [details] Please, add your ChangeLog entry as well. See at http://webkit.org/coding/contributing.html .
Afonso Costa
Comment 2 2009-10-26 09:15:29 PDT
Created attachment 41870 [details] Proposed patch - v2 Added changelog
Tor Arne Vestbø
Comment 3 2009-10-27 03:40:59 PDT
Thanks for your bug report. In the future, please follow the QtWebKit bug reporting guidelines: http://trac.webkit.org/wiki/QtWebKitContrib#ReportingBugs In particular: - All bugs related to the Qt port of WebKit should have the keyword 'Qt' - The 'WebKit Qt' component should only be used for API layer bugs You can use the following URL as a template: http://tinyurl.com/NewQtWebKitBug
Oliver Hunt
Comment 4 2009-10-28 12:57:21 PDT
Comment on attachment 41870 [details] Proposed patch - v2 r=me
WebKit Commit Bot
Comment 5 2009-10-28 14:44:16 PDT
Comment on attachment 41870 [details] Proposed patch - v2 Clearing flags on attachment: 41870 Committed r50239: <http://trac.webkit.org/changeset/50239>
WebKit Commit Bot
Comment 6 2009-10-28 14:44:20 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.