Title: [183882] trunk/Tools
Revision
183882
Author
[email protected]
Date
2015-05-06 12:26:15 -0700 (Wed, 06 May 2015)

Log Message

Fix api tests after r183877.

* TestWebKitAPI/Tests/WebCore/ContentExtensions.cpp:
(TestWebKitAPI::TEST_F):
Generate all DFAs before compiling them.

Modified Paths

Diff

Modified: trunk/Tools/ChangeLog (183881 => 183882)


--- trunk/Tools/ChangeLog	2015-05-06 19:02:02 UTC (rev 183881)
+++ trunk/Tools/ChangeLog	2015-05-06 19:26:15 UTC (rev 183882)
@@ -1,3 +1,11 @@
+2015-05-06  Alex Christensen  <[email protected]>
+
+        Fix api tests after r183877.
+
+        * TestWebKitAPI/Tests/WebCore/ContentExtensions.cpp:
+        (TestWebKitAPI::TEST_F):
+        Generate all DFAs before compiling them.
+
 2015-05-06  Alexey Proskuryakov  <[email protected]>
 
         Do not link tools to WebKit2.framework

Modified: trunk/Tools/TestWebKitAPI/Tests/WebCore/ContentExtensions.cpp (183881 => 183882)


--- trunk/Tools/TestWebKitAPI/Tests/WebCore/ContentExtensions.cpp	2015-05-06 19:02:02 UTC (rev 183881)
+++ trunk/Tools/TestWebKitAPI/Tests/WebCore/ContentExtensions.cpp	2015-05-06 19:26:15 UTC (rev 183882)
@@ -738,11 +738,15 @@
             nfas.append(WTF::move(nfa));
         });
         EXPECT_EQ(nfas.size(), expectedNFACounts[i]);
-
+        
+        Vector<ContentExtensions::DFA> dfas;
+        for (auto& nfa : nfas)
+            dfas.append(ContentExtensions::NFAToDFA::convert(nfa));
+        
         Vector<ContentExtensions::DFABytecode> combinedBytecode;
-        for (auto& nfa : nfas) {
+        for (const auto& dfa : dfas) {
             Vector<ContentExtensions::DFABytecode> bytecode;
-            ContentExtensions::DFABytecodeCompiler compiler(ContentExtensions::NFAToDFA::convert(nfa), bytecode);
+            ContentExtensions::DFABytecodeCompiler compiler(dfa, bytecode);
             compiler.compile();
             combinedBytecode.appendVector(bytecode);
         }
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to