[WK2] Start a prototype for declarative site specific extensions
Created attachment 244116 [details] Patch
Created attachment 244123 [details] Patch
Created attachment 244136 [details] Patch
Comment on attachment 244136 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=244136&action=review > Source/WebKit2/UIProcess/API/Cocoa/WKProcessPoolPrivate.h:46 > +- (void)_loadContentExtensionWithIdentifier:(NSString *)identifier serializedRules:(NSString *)serializedRules successCompletionHandler:(void (^)(NSDictionary *info))successCompletionHandler errorCompletionHandler:(void (^)(NSError *error))errorCompletionHandler; Add FIXME: move the API to a better place.
Comment on attachment 244136 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=244136&action=review r=me. This seems like a good-enough foundation to build on. > Source/WebCore/contentextensions/DFA.cpp:43 > + : m_nodes(nodes) I think this needs to explicitly move or it will be a copy: : m_nodes(WTF::move(nodes)) > Source/WebCore/contentextensions/DFA.h:51 > + Vector<uint64_t> actions(unsigned currentState) const; This is returning a copy of the Vector, which is probably not what you want.
Committed r178151: <http://trac.webkit.org/changeset/178151>