Summary: | [BlackBerry] Upstream BlackBerry WebCoreSupport FrameNetworkingContextBlackBerry and IconDatabaseClientBlackBerry classes | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Jacky Jiang <jkjiang> | ||||||||
Component: | New Bugs | Assignee: | Jacky Jiang <jkjiang> | ||||||||
Status: | RESOLVED FIXED | ||||||||||
Severity: | Normal | CC: | dbates, rwlbuis, tonikitoo, webkit.review.bot, zimmermann | ||||||||
Priority: | P2 | ||||||||||
Version: | 528+ (Nightly build) | ||||||||||
Hardware: | Unspecified | ||||||||||
OS: | Unspecified | ||||||||||
Bug Depends on: | |||||||||||
Bug Blocks: | 73144 | ||||||||||
Attachments: |
|
Description
Jacky Jiang
2012-02-07 15:10:38 PST
Created attachment 125946 [details]
Patch
Comment on attachment 125946 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=125946&action=review Some small nits. > Source/WebKit/blackberry/WebCoreSupport/IconDatabaseClientBlackBerry.cpp:32 > + instance = new IconDatabaseClientBlackBerry(); Can you use DEFINE_STATIC_LOCAL? > Source/WebKit/blackberry/WebCoreSupport/IconDatabaseClientBlackBerry.cpp:60 > + return (m_initState == InitializeSucceeded); No need for parentheses. > Source/WebKit/blackberry/WebCoreSupport/IconDatabaseClientBlackBerry.h:40 > +public: Better move the public section up above the private, this is the standard. Created attachment 126178 [details]
Patch
Update the patch except the DEFINE_STATIC_LOCAL which doesn't seem to work well.
Comment on attachment 126178 [details]
Patch
Typo in commit log, will update.
Created attachment 126180 [details]
Patch
Comment on attachment 126180 [details]
Patch
Looks good.
Comment on attachment 126180 [details] Patch Clearing flags on attachment: 126180 Committed r107183: <http://trac.webkit.org/changeset/107183> All reviewed patches have been landed. Closing bug. |