Title: [182024] trunk/Source/WebCore
- Revision
- 182024
- Author
- [email protected]
- Date
- 2015-03-26 14:02:29 -0700 (Thu, 26 Mar 2015)
Log Message
Fix state maching debugging after r181964
https://bugs.webkit.org/show_bug.cgi?id=143082
Patch by Benjamin Poulain <[email protected]> on 2015-03-26
Reviewed by Alex Christensen.
* contentextensions/ContentExtensionCompiler.cpp:
(WebCore::ContentExtensions::compileRuleList):
Modified Paths
Diff
Modified: trunk/Source/WebCore/ChangeLog (182023 => 182024)
--- trunk/Source/WebCore/ChangeLog 2015-03-26 20:48:06 UTC (rev 182023)
+++ trunk/Source/WebCore/ChangeLog 2015-03-26 21:02:29 UTC (rev 182024)
@@ -1,3 +1,13 @@
+2015-03-26 Benjamin Poulain <[email protected]>
+
+ Fix state maching debugging after r181964
+ https://bugs.webkit.org/show_bug.cgi?id=143082
+
+ Reviewed by Alex Christensen.
+
+ * contentextensions/ContentExtensionCompiler.cpp:
+ (WebCore::ContentExtensions::compileRuleList):
+
2015-03-25 David Hyatt <[email protected]>
Improve the offsetWidth/Height layout optimization
Modified: trunk/Source/WebCore/contentextensions/ContentExtensionCompiler.cpp (182023 => 182024)
--- trunk/Source/WebCore/contentextensions/ContentExtensionCompiler.cpp 2015-03-26 20:48:06 UTC (rev 182023)
+++ trunk/Source/WebCore/contentextensions/ContentExtensionCompiler.cpp 2015-03-26 21:02:29 UTC (rev 182024)
@@ -153,7 +153,11 @@
#endif
#if CONTENT_EXTENSIONS_STATE_MACHINE_DEBUGGING
- nfa.debugPrintDot();
+ for (size_t i = 0; i < nfas.size(); ++i) {
+ WTFLogAlways("NFA %zu", i);
+ const NFA& nfa = nfas[i];
+ nfa.debugPrintDot();
+ }
#endif
#if CONTENT_EXTENSIONS_PERFORMANCE_REPORTING
@@ -162,6 +166,12 @@
Vector<DFABytecode> bytecode;
for (size_t i = 0; i < nfas.size(); ++i) {
DFA dfa = NFAToDFA::convert(nfas[i]);
+
+#if CONTENT_EXTENSIONS_STATE_MACHINE_DEBUGGING
+ WTFLogAlways("DFA %zu", i);
+ dfa.debugPrintDot();
+#endif
+
if (!i) {
// Put all the universal actions on the first DFA.
for (uint64_t actionLocation : universalActionLocations)
@@ -171,13 +181,6 @@
compiler.compile();
}
-#if CONTENT_EXTENSIONS_PERFORMANCE_REPORTING
- double dfaBuildTimeEnd = monotonicallyIncreasingTime();
- dataLogF(" Time spent building the DFA: %f\n", (dfaBuildTimeEnd - dfaBuildTimeStart));
-#endif
-
- // FIXME: never add a DFA that only matches the empty set.
-
#if CONTENT_EXTENSIONS_STATE_MACHINE_DEBUGGING
dfa.debugPrintDot();
#endif
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes
