Changeset 180687 in webkit

Timestamp:

Feb 26, 2015, 11:29:48 AM (12 years ago)

Author:

beidson@apple.com

Message:

Make WKPageGroupRemoveAllUserContentFilters actually remove all user content filters
​https://bugs.webkit.org/show_bug.cgi?id=142047

Reviewed by Geoff Garen.

Source/WebKit2:

• UIProcess/API/C/WKPageGroup.cpp:

(WKPageGroupRemoveAllUserContentFilters): Remove all content filters instead of removing all user scripts.

LayoutTests:

• platform/mac-wk2/TestExpectations: Renable content filter tests, as they should no longer interfere with others.

Location:

trunk

Files:

• LayoutTests/ChangeLog (modified) (1 diff)
• LayoutTests/platform/mac-wk2/TestExpectations (modified) (1 diff)
• Source/WebKit2/ChangeLog (modified) (1 diff)
• Source/WebKit2/UIProcess/API/C/WKPageGroup.cpp (modified) (1 diff)

trunk/LayoutTests/ChangeLog

@@ +1 @@
+2015-02-26  Brady Eidson  <beidson@apple.com>
+
+        Make WKPageGroupRemoveAllUserContentFilters actually remove all user content filters
+        https://bugs.webkit.org/show_bug.cgi?id=142047
+
+        Reviewed by Geoff Garen.
+
+        * platform/mac-wk2/TestExpectations: Renable content filter tests, as they should no longer interfere with others.
+
 2015-02-26  Said Abou-Hallawa  <sabouhallawa@apple.com>
 

trunk/LayoutTests/platform/mac-wk2/TestExpectations

@@ -517 +517 @@
 ### END OF (7) New WebKit2-only failures in Yosemite
 ########################################
+
+http/tests/usercontentfilter [ Pass ]

trunk/Source/WebKit2/ChangeLog

@@ +1 @@
+2015-02-26  Brady Eidson  <beidson@apple.com>
+
+        Make WKPageGroupRemoveAllUserContentFilters actually remove all user content filters
+        https://bugs.webkit.org/show_bug.cgi?id=142047
+
+        Reviewed by Geoff Garen.
+
+        * UIProcess/API/C/WKPageGroup.cpp:
+        (WKPageGroupRemoveAllUserContentFilters): Remove all content filters instead of removing all user scripts.
+
 2015-02-25  Anders Carlsson  <andersca@apple.com>
 

trunk/Source/WebKit2/UIProcess/API/C/WKPageGroup.cpp

@@ -93 +93 @@
 {
 #if ENABLE(CONTENT_EXTENSIONS)
-    toImpl(pageGroupRef)->removeAllUserScripts();
+    toImpl(pageGroupRef)->removeAllUserContentFilters();
 #else
     UNUSED_PARAM(pageGroupRef);