RESOLVED FIXED 73524
upstream BlackBerry porting of KURL/Logging 
https://bugs.webkit.org/show_bug.cgi?id=73524
Summary upstream BlackBerry porting of KURL/Logging
Mary Wu
Reported 2011-11-30 22:45:21 PST
Add two files KURLBlackBerry.cpp and LoggingBlackBerry.cpp to WebCore/platform/blackberry
Attachments
Patch (5.25 KB, patch)
2011-11-30 22:54 PST, Mary Wu 		no flags
DetailsFormatted DiffDiff
Patch (5.25 KB, patch)
2011-12-01 00:12 PST, Mary Wu 		no flags
DetailsFormatted DiffDiff
Patch (5.26 KB, patch)
2011-12-01 01:33 PST, Mary Wu 		no flags
DetailsFormatted DiffDiff
Show Obsolete (2) View All Add attachment proposed patch, testcase, etc.
Mary Wu
Comment 1 2011-11-30 22:54:04 PST
Created attachment 117346 [details] Patch
Daniel Bates
Comment 2 2011-11-30 23:29:17 PST
Comment on attachment 117346 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=117346&action=review > Source/WebCore/platform/blackberry/KURLBlackBerry.cpp:33 > + return String(path()); KURL::path() returns a string. Therefore, it's sufficient to do: return path(); > Source/WebCore/platform/blackberry/LoggingBlackBerry.cpp:32 > +void InitializeLoggingChannelsIfNecessary() The name of this function doesn't conform to the WebKit Code Style Guidelines. Function names should be in camel case starting with a lowercase letter.
Mary Wu
Comment 3 2011-11-30 23:49:46 PST
(In reply to comment #2) > (From update of attachment 117346 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=117346&action=review > > > Source/WebCore/platform/blackberry/KURLBlackBerry.cpp:33 > > + return String(path()); > > KURL::path() returns a string. Therefore, it's sufficient to do: > > return path(); OK > > Source/WebCore/platform/blackberry/LoggingBlackBerry.cpp:32 > > +void InitializeLoggingChannelsIfNecessary() > > The name of this function doesn't conform to the WebKit Code Style Guidelines. Function names should be in camel case starting with a lowercase letter. The function name was defined in header file platform/Logging.h, qt/mac/efl/win/wx/gtk all follows. If we fix this style issue, should we do that in a separate PR with all portings changed?
Mary Wu
Comment 4 2011-12-01 00:12:09 PST
Created attachment 117359 [details] Patch
Mary Wu
Comment 5 2011-12-01 01:33:57 PST
Created attachment 117379 [details] Patch
WebKit Review Bot
Comment 6 2011-12-06 22:26:51 PST
Comment on attachment 117379 [details] Patch Clearing flags on attachment: 117379 Committed r102223: <http://trac.webkit.org/changeset/102223>
WebKit Review Bot
Comment 7 2011-12-06 22:26:55 PST
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.