Issue 3237 - Whitelisted AFS ad is not shown on netzwelt.de

src/plugin/PluginWbPassThrough.cpp

Index: src/plugin/PluginWbPassThrough.cpp
===================================================================
--- a/src/plugin/PluginWbPassThrough.cpp
+++ b/src/plugin/PluginWbPassThrough.cpp
@@ -146,6 +146,13 @@
 {
   if (mimeType.find(L"image/") != std::wstring::npos)
   {
+	// Ambivalent mime type suggests both image and other types. 
+	// Heuristically this is more likely a SUBDOCUMENT request.
+	if ((mimeType.find(L"text/") != std::wstring::npos) ||

[sergei, 2015/12/01 10:53:59]

In IE11 on my machine Accept header for "subdocument" seems "text/html,
application/xhtml+xml, image/jxr, */*" pretty stable. Just wonder, when and what
are there some different "text/???" and "application/???"? May be it  makes
sense to restrict the rule here.

What are the values on IE{8-10}?

[Eric, 2015/12/01 20:32:57]

We don't have anything like a crawler for IE, but it doesn't seem that it would
be all that difficult to log the media type list here, crawl manually, and
gather a collection of the media type lists that IE actually generates. 

[Oleksandr, 2015/12/15 02:10:16]

I have only been able to log 2 types here:
text/html, application/xhtml+xml, image/jxr, */*
image/png, image/svg+xml, image/jxr, image/*;q=0.8, */*;q=0.5

In IE9 the image/jxr part is missing, but other then that the header looks very
similar. So I agree, we can be precise here.

[Eric, 2015/12/15 03:18:18]

These are the strings I'd like to see in a comment.

+		(mimeType.find(L"application/") != std::wstring::npos))
+	{
+		return ContentType::CONTENT_TYPE_SUBDOCUMENT;
+	}
     return ContentType::CONTENT_TYPE_IMAGE;
   }
   if (mimeType.find(L"text/css") != std::wstring::npos)