Title: [235701] branches/safari-606-branch/Source/WebCore
- Revision
- 235701
- Author
- [email protected]
- Date
- 2018-09-05 14:52:34 -0700 (Wed, 05 Sep 2018)
Log Message
Cherry-pick r234890. rdar://problem/44144063
Remove failing assertion introduced in r234873
https://bugs.webkit.org/show_bug.cgi?id=188581
* contentextensions/ContentExtensionCompiler.cpp:
(WebCore::ContentExtensions::compileRuleList):
The assertion is correct but failing because VectorTraits<String> is incorrect.
I'll re-add it and fix VectorTraits<String> in a separate patch.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@234890 268f45cc-cd09-0410-ab3c-d52691b4dbfc
Modified Paths
Diff
Modified: branches/safari-606-branch/Source/WebCore/ChangeLog (235700 => 235701)
--- branches/safari-606-branch/Source/WebCore/ChangeLog 2018-09-05 21:52:32 UTC (rev 235700)
+++ branches/safari-606-branch/Source/WebCore/ChangeLog 2018-09-05 21:52:34 UTC (rev 235701)
@@ -1,5 +1,31 @@
2018-09-05 Babak Shafiei <[email protected]>
+ Cherry-pick r234890. rdar://problem/44144063
+
+ Remove failing assertion introduced in r234873
+ https://bugs.webkit.org/show_bug.cgi?id=188581
+
+ * contentextensions/ContentExtensionCompiler.cpp:
+ (WebCore::ContentExtensions::compileRuleList):
+ The assertion is correct but failing because VectorTraits<String> is incorrect.
+ I'll re-add it and fix VectorTraits<String> in a separate patch.
+
+
+
+ git-svn-id: https://svn.webkit.org/repository/webkit/trunk@234890 268f45cc-cd09-0410-ab3c-d52691b4dbfc
+
+ 2018-08-15 Alex Christensen <[email protected]>
+
+ Remove failing assertion introduced in r234873
+ https://bugs.webkit.org/show_bug.cgi?id=188581
+
+ * contentextensions/ContentExtensionCompiler.cpp:
+ (WebCore::ContentExtensions::compileRuleList):
+ The assertion is correct but failing because VectorTraits<String> is incorrect.
+ I'll re-add it and fix VectorTraits<String> in a separate patch.
+
+2018-09-05 Babak Shafiei <[email protected]>
+
Cherry-pick r234873. rdar://problem/44144063
isValidCSSSelector is unsafe to be called from a non-main thread
Modified: branches/safari-606-branch/Source/WebCore/contentextensions/ContentExtensionCompiler.cpp (235700 => 235701)
--- branches/safari-606-branch/Source/WebCore/contentextensions/ContentExtensionCompiler.cpp 2018-09-05 21:52:32 UTC (rev 235700)
+++ branches/safari-606-branch/Source/WebCore/contentextensions/ContentExtensionCompiler.cpp 2018-09-05 21:52:34 UTC (rev 235701)
@@ -285,12 +285,6 @@
std::error_code compileRuleList(ContentExtensionCompilationClient& client, String&& ruleJSON, Vector<ContentExtensionRule>&& parsedRuleList)
{
-#if !ASSERT_DISABLED
- callOnMainThread([ruleJSON = ruleJSON.isolatedCopy(), parsedRuleList = parsedRuleList.isolatedCopy()] {
- ASSERT(parseRuleList(ruleJSON) == parsedRuleList);
- });
-#endif
-
bool domainConditionSeen = false;
bool topURLConditionSeen = false;
for (const auto& rule : parsedRuleList) {
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes
