WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
128956
[Cocoa] Implement -description in some API objects
https://bugs.webkit.org/show_bug.cgi?id=128956
Summary
[Cocoa] Implement -description in some API objects
mitz
Reported
2014-02-17 23:35:33 PST
[Cocoa] Implement -description in some API objects
Attachments
Implement -description in UIProcess/API/Cocoa and Shared/API/Cocoa
(6.53 KB, patch)
2014-02-17 23:37 PST
,
mitz
andersca
: review+
Details
Formatted Diff
Diff
View All
Add attachment
proposed patch, testcase, etc.
mitz
Comment 1
2014-02-17 23:37:42 PST
Created
attachment 224475
[details]
Implement -description in UIProcess/API/Cocoa and Shared/API/Cocoa
Anders Carlsson
Comment 2
2014-02-18 11:27:13 PST
Comment on
attachment 224475
[details]
Implement -description in UIProcess/API/Cocoa and Shared/API/Cocoa View in context:
https://bugs.webkit.org/attachment.cgi?id=224475&action=review
> Source/WebKit2/Shared/API/Cocoa/WKRemoteObjectInterface.mm:146 > + return [NSString stringWithFormat:@"<%@: %p; protocol = \"%@\"; identifier = \"%@\">", NSStringFromClass([self class]), self, _identifier, NSStringFromProtocol(_protocol)];
Please use self.class instead of [self class].
> Source/WebKit2/UIProcess/API/Cocoa/WKFrameInfo.mm:53 > + return [NSString stringWithFormat:@"<%@: %p; isMainFrame = %d; request = %@>", NSStringFromClass([self class]), self, _mainFrame, _request.get()];
Please use self.class instead of [self class]. I think you should return either YES or NO for the main frame bit.
> Source/WebKit2/UIProcess/API/Cocoa/WKNavigationAction.mm:41 > + return [NSString stringWithFormat:@"<%@: %p; navigationType = %ld; request = %@; sourceFrame = %@; destinationFrame = %@>", NSStringFromClass([self class]), self,
Please use self.class instead of [self class].
> Source/WebKit2/UIProcess/API/Cocoa/WKNavigationResponse.mm:40 > + return [NSString stringWithFormat:@"<%@: %p; frame = %@; response = %@>", NSStringFromClass([self class]), self, _frame.get(), _response.get()];
Please use self.class instead of [self class].
> Source/WebKit2/UIProcess/API/Cocoa/WKProcessClass.mm:83 > + return [NSString stringWithFormat:@"<%@: %p; configuration = %@>", NSStringFromClass([self class]), self, _configuration.get()];
Please use self.class instead of [self class].
> Source/WebKit2/UIProcess/API/Cocoa/WKProcessClassConfiguration.mm:49 > + NSString *description = [NSString stringWithFormat:@"<%@: %p; maximumProcessCount = %ld", NSStringFromClass([self class]), self, _maximumProcessCount];
Please use self.class instead of [self class].
> Source/WebKit2/UIProcess/API/Cocoa/WKWebViewConfiguration.mm:42 > + return [NSString stringWithFormat:@"<%@: %p; processClass = %@; preferences = %@>", NSStringFromClass([self class]), self, _processClass.get(), _preferences.get()];
Please use self.class instead of [self class].
mitz
Comment 3
2014-02-18 11:31:39 PST
Committed <
http://trac.webkit.org/r164300
>.
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug