Perform some spring cleaning to WKContentView and WKView
Created attachment 217983 [details] Patch
Comment on attachment 217983 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=217983&action=review > Source/WebKit2/UIProcess/API/ios/WKContentView.mm:28 > #import "WKContentViewInternal.h" > +#import "WKContentViewPrivate.h" This shouldn’t be needed. Internal should import Private. > Source/WebKit2/UIProcess/API/ios/WKContentView.mm:39 > +#import "WKProcessGroupInternal.h" > #import "WKProcessGroupPrivate.h" Private should be imported by Internal. > Source/WebKit2/UIProcess/API/ios/WKContentView.mm:66 > + // FIXME: Implement. > + return nil; Should release self before returning nil from an initializer. > Source/WebKit2/UIProcess/API/ios/WKContentView.mm:72 > - if (!(self = [super initWithFrame:frame])) > + self = [super initWithFrame:frame]; > + if (!self) Why?
Comment on attachment 217983 [details] Patch Attachment 217983 [details] did not pass efl-wk2-ews (efl-wk2): Output: http://webkit-queues.appspot.com/results/39388002
Comment on attachment 217983 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=217983&action=review > Source/WebKit2/UIProcess/API/ios/WKView.mm:28 > #import "WKView.h" > +#import "WKViewPrivate.h" No need for #import "WKView.h"
Committed r159859: <http://trac.webkit.org/changeset/159859>
Bug to fix gtk build: https://bugs.webkit.org/show_bug.cgi?id=124992