Changeset 202751 in webkit

Timestamp:

Jul 1, 2016 1:46:14 PM (8 years ago)

Author:

akling@apple.com

Message:

Add early return when processing content extensions if there aren't any.
<​https://webkit.org/b/159363>

Reviewed by Antti Koivisto.

Short-circuit outta there if there aren't any extensions to query.

• contentextensions/ContentExtensionsBackend.cpp:

(WebCore::ContentExtensions::ContentExtensionsBackend::processContentExtensionRulesForLoad):

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• contentextensions/ContentExtensionsBackend.cpp (modified) (1 diff)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2016-07-01  Andreas Kling  <akling@apple.com>
+
+        Add early return when processing content extensions if there aren't any.
+        <https://webkit.org/b/159363>
+
+        Reviewed by Antti Koivisto.
+
+        Short-circuit outta there if there aren't any extensions to query.
+
+        * contentextensions/ContentExtensionsBackend.cpp:
+        (WebCore::ContentExtensions::ContentExtensionsBackend::processContentExtensionRulesForLoad):
+
 2016-07-01  Eric Carlson  <eric.carlson@apple.com>
 

trunk/Source/WebCore/contentextensions/ContentExtensionsBackend.cpp

@@ -149 +149 @@
 BlockedStatus ContentExtensionsBackend::processContentExtensionRulesForLoad(ResourceRequest& request, ResourceType resourceType, DocumentLoader& initiatingDocumentLoader)
 {
+    if (m_contentExtensions.isEmpty())
+        return BlockedStatus::NotBlocked;
+
     Document* currentDocument = nullptr;
     URL mainDocumentURL;