Bug 89531

Summary: Pass ScriptValue by const ref where possible
Product: WebKit Reporter: Adam Klein <adamk>
Component: New BugsAssignee: Adam Klein <adamk>
Status: RESOLVED FIXED    
Severity: Normal CC: abarth, cmarcelo, eric.carlson, feature-media-reviews, gustavo, haraken, japhet, jochen, macpherson, menard, philn, webkit.review.bot, xan.lopez
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Tentative patch for landing - fix JSC build none

Description Adam Klein 2012-06-19 16:53:30 PDT
Pass ScriptValue by const ref where possible
Comment 1 Adam Klein 2012-06-19 16:57:48 PDT
Created attachment 148461 [details]
Patch
Comment 2 Kentaro Hara 2012-06-19 17:10:24 PDT
Comment on attachment 148461 [details]
Patch

Looks OK. This is one of the things I wanted to fix:)
Comment 3 Adam Klein 2012-06-19 17:31:16 PDT
Created attachment 148470 [details]
Tentative patch for landing - fix JSC build
Comment 4 WebKit Review Bot 2012-06-19 20:27:45 PDT
Comment on attachment 148470 [details]
Tentative patch for landing - fix JSC build

Clearing flags on attachment: 148470

Committed r120791: <http://trac.webkit.org/changeset/120791>
Comment 5 WebKit Review Bot 2012-06-19 20:27:51 PDT
All reviewed patches have been landed.  Closing bug.