RESOLVED FIXED 140160
[WK2] Start a prototype for declarative site specific extensions 
https://bugs.webkit.org/show_bug.cgi?id=140160
Summary [WK2] Start a prototype for declarative site specific extensions
Benjamin Poulain
Reported 2015-01-06 17:57:03 PST
[WK2] Start a prototype for declarative site specific extensions
Attachments
Patch (94.37 KB, patch)
2015-01-06 18:38 PST, Benjamin Poulain 		no flags
DetailsFormatted DiffDiff
Patch (94.32 KB, patch)
2015-01-06 19:13 PST, Benjamin Poulain 		no flags
DetailsFormatted DiffDiff
Patch (94.29 KB, patch)
2015-01-06 21:01 PST, Benjamin Poulain 		kling: review+
DetailsFormatted DiffDiff
Show Obsolete (2) View All Add attachment proposed patch, testcase, etc.
Benjamin Poulain
Comment 1 2015-01-06 18:38:42 PST
Created attachment 244116 [details] Patch
Benjamin Poulain
Comment 2 2015-01-06 19:13:36 PST
Created attachment 244123 [details] Patch
Benjamin Poulain
Comment 3 2015-01-06 21:01:37 PST
Created attachment 244136 [details] Patch
Benjamin Poulain
Comment 4 2015-01-07 12:47:51 PST
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.
Andreas Kling
Comment 5 2015-01-08 17:01:30 PST
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.
Benjamin Poulain
Comment 6 2015-01-08 17:55:57 PST
Committed r178151: <http://trac.webkit.org/changeset/178151>
Note You need to log in before you can comment on or make changes to this bug.