Don't convert entities in script/style elements

In HTML the text of a script/style tag should be left untouched, no
entities must be converted. Doing so would break Javascript such as the
following:

    foo&&bar;

Such code is often the result of minifiers doing their dirty business.

lib/oga/xml/text.rb

@@ -26,7 +26,7 @@ module Oga
       # @return [String]
       #
       def text
-        unless @decoded
+        if decode_entities?
           @text    = EntityDecoder.try_decode(@text, html?)
           @decoded = true
         end
@@ -38,15 +38,29 @@ module Oga
       # @see [Oga::XML::CharacterNode#to_xml]
       #
       def to_xml
-        node = parent
+        return super if inside_literal_html?
-
-        if node.is_a?(Element) and html? \
-        and Lexer::LITERAL_HTML_ELEMENTS.include?(node.name)
-          return super
-        end
 
         return Entities.encode(super)
       end
+
+      private
+
+      ##
+      # @return [TrueClass|FalseClass]
+      #
+      def decode_entities?
+        return !@decoded && !inside_literal_html?
+      end
+
+      ##
+      # @return [TrueClass|FalseClass]
+      #
+      def inside_literal_html?
+        node = parent
+
+        return node.is_a?(Element) && html? &&
+          Lexer::LITERAL_HTML_ELEMENTS.include?(node.name)
+      end
     end # Text
   end # XML
 end # Oga

spec/oga/xml/text_spec.rb

@@ -85,6 +85,38 @@ describe Oga::XML::Text do
         node.text.should == [160].pack('U')
       end
     end
+
+    describe 'inside an HTML script element' do
+      before do
+        @element  = Oga::XML::Element.new(:name => 'script')
+        @document = Oga::XML::Document.new(
+          :type     => :html,
+          :children => [@element]
+        )
+      end
+
+      it 'does not decode any entities' do
+        @element.inner_text = '&foo;'
+
+        @element.inner_text.should == '&foo;'
+      end
+    end
+
+    describe 'inside an HTML style element' do
+      before do
+        @element  = Oga::XML::Element.new(:name => 'style')
+        @document = Oga::XML::Document.new(
+          :type     => :html,
+          :children => [@element]
+        )
+      end
+
+      it 'does not decode any entities' do
+        @element.inner_text = '&foo;'
+
+        @element.inner_text.should == '&foo;'
+      end
+    end
   end
 
   describe '#to_xml' do