Issue 1183 - Fix Acceptable Ads hiding

src/engine/Main.cpp

Index: src/engine/Main.cpp
===================================================================
--- a/src/engine/Main.cpp
+++ b/src/engine/Main.cpp
@@ -96,6 +96,15 @@
         response << (filter && filter->GetType() != AdblockPlus::Filter::TYPE_EXCEPTION);
         break;
       }
+      case Communication::PROC_HIDING_ENABLED_ON_DOMAIN:

[Felix Dahlke, 2014/08/08 16:11:35]

Since this is conceptually very similar to PROC_IS_WHITELISTED_URL, I propose
this:

1. Rename it to "PROC_IS_ELEMHIDE_WHITELISTED_ON_URL" and move it below that
one.
2. Rename "documentUrl" to "url".

+      {
+        std::string type;

[Felix Dahlke, 2014/08/08 16:11:35]

Unused, isn't it?

+        std::string documentUrl;
+        request >> documentUrl;
+        AdblockPlus::FilterPtr filter = filterEngine->Matches(documentUrl, "ELEMHIDE", documentUrl);
+        response << (!filter);

[Felix Dahlke, 2014/08/08 16:11:35]

Superfluous parentheses.

More importantly, this only checks if there was a filter, not that it was
actually an element hiding filter. Should basically work the same way as
PROC_IS_WHITELISTED_URL (logic negated of course):

response << (match && match->GetType() == AdblockPlus::Filter::TYPE_EXCEPTION);

+        break;
+      }
       case Communication::PROC_GET_ELEMHIDE_SELECTORS:
       {
         std::string domain;