Bug 97014

Summary: [WK2][WTR] InjectedBundle::booleanForKey() should handle literals effectively
Product: WebKit Reporter: Mikhail Pozdnyakov <mikhail.pozdnyakov>
Component: WebKit2Assignee: Mikhail Pozdnyakov <mikhail.pozdnyakov>
Status: RESOLVED FIXED    
Severity: Normal CC: cdumez, gyuyoung.kim, kenneth, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
patch none

Description Mikhail Pozdnyakov 2012-09-18 06:38:33 PDT
According to http://trac.webkit.org/wiki/EfficientStrings WTF::StringBuilder::appendLiteral() shall be used for literals rather than WTF::StringBuilder::append().
Comment 1 Mikhail Pozdnyakov 2012-09-18 06:41:08 PDT
Created attachment 164547 [details]
patch
Comment 2 Chris Dumez 2012-09-18 06:41:49 PDT
Comment on attachment 164547 [details]
patch

LGTM.
Comment 3 WebKit Review Bot 2012-09-18 08:00:22 PDT
Comment on attachment 164547 [details]
patch

Clearing flags on attachment: 164547

Committed r128895: <http://trac.webkit.org/changeset/128895>
Comment 4 WebKit Review Bot 2012-09-18 08:00:26 PDT
All reviewed patches have been landed.  Closing bug.