svn commit: r393832 - /spamassassin/trunk/lib/Mail/SpamAssassin/HTML.pm

[fe...@apache.org]

Author: felicity
Date: Thu Apr 13 08:26:19 2006
New Revision: 393832

URL: http://svn.apache.org/viewcvs?rev=393832&view=rev
Log:
style="visibility: hidden" ought to be caught as making text invisible...

Modified:
    spamassassin/trunk/lib/Mail/SpamAssassin/HTML.pm

Modified: spamassassin/trunk/lib/Mail/SpamAssassin/HTML.pm
URL: http://svn.apache.org/viewcvs/spamassassin/trunk/lib/Mail/SpamAssassin/HTML.pm?rev=393832&r1=393831&r2=393832&view=diff
==============================================================================
--- spamassassin/trunk/lib/Mail/SpamAssassin/HTML.pm (original)
+++ spamassassin/trunk/lib/Mail/SpamAssassin/HTML.pm Thu Apr 13 08:26:19 2006
@@ -519,6 +519,10 @@
 	    $new{display} = 'none';
 	    $self->put_results(span_invisible => 1);
 	  }
+	  elsif (/\s*visibility:\s*hidden\b/i) {
+	    $new{visibility} = 'hidden';
+	    $self->put_results(span_invisible => 1);
+	  }
 	}
       }
       else {
@@ -556,6 +560,7 @@
   my $bg = $self->{text_style}[-1]->{bgcolor};
   my $size = $self->{text_style}[-1]->{size};
   my $display = $self->{text_style}[-1]->{display};
+  my $visibility = $self->{text_style}[-1]->{visibility};
 
   # invisibility
   if (substr($fg,-6) eq substr($bg,-6)) {
@@ -595,6 +600,10 @@
 
   # <span style="display: none">
   if ($display && lc $display eq 'none') {
+    return 1;
+  }
+
+  if ($visibility && lc $visibility eq 'hidden') {
     return 1;
   }