CTTE some of the BackForwardList member functions
Created attachment 291842 [details] Patch
Comment on attachment 291842 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=291842&action=review > Source/WebKit/mac/History/WebBackForwardList.mm:208 > - return core(self)->containsItem(core(item)); > + if (!item) > + return NO; > + > + return core(self)->containsItem(*core(item)); I like to write it like this: return item && return core(self)->containsItem(*core(item));
Committed r207419: <http://trac.webkit.org/changeset/207419>
I get a build error on iOS: /Volumes/Data/cdumez/WebKit/OpenSource/Source/WebKit/mac/History/WebBackForwardList.mm:170:34: error: no matching function for call to 'kit' [entriesArray addObject:[kit(historyItems[i].ptr()) dictionaryRepresentationIncludingChildren:NO]];