Adding ASSERT for RuntimeEnabled feature XSLT

Source/core/fetch/XSLStyleSheetResource.cpp

 /*
     Copyright (C) 1998 Lars Knoll (knoll@mpi-hd.mpg.de)
     Copyright (C) 2001 Dirk Mueller (mueller@kde.org)
     Copyright (C) 2002 Waldo Bastian (bastian@kde.org)
     Copyright (C) 2006 Samuel Weinig (sam.weinig@gmail.com)
     Copyright (C) 2004, 2005, 2006, 2007, 2008 Apple Inc. All rights reserved.
 
     This library is free software; you can redistribute it and/or
     modify it under the terms of the GNU Library General Public
     License as published by the Free Software Foundation; either
     version 2 of the License, or (at your option) any later version.
 
     This library is distributed in the hope that it will be useful,
     but WITHOUT ANY WARRANTY; without even the implied warranty of
     MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
     Library General Public License for more details.
 
     You should have received a copy of the GNU Library General Public License
     along with this library; see the file COPYING.LIB.  If not, write to
     the Free Software Foundation, Inc., 51 Franklin Street, Fifth Floor,
     Boston, MA 02110-1301, USA.
 
     This class provides all functionality needed for loading images, style sheets and html
     pages from the web. It has a memory cache for these objects.
 */
 
 #include "config.h"
 #include "core/fetch/XSLStyleSheetResource.h"
 
+#include "RuntimeEnabledFeatures.h"
 #include "core/fetch/ResourceClientWalker.h"
 #include "core/fetch/StyleSheetResourceClient.h"
 #include "core/fetch/TextResourceDecoder.h"
 #include "platform/SharedBuffer.h"
 #include "wtf/Vector.h"
 
 namespace WebCore {
 
 XSLStyleSheetResource::XSLStyleSheetResource(const ResourceRequest& resourceRequest)
     : Resource(resourceRequest, XSLStyleSheet)
     , m_decoder(TextResourceDecoder::create("text/xsl"))
 {
+    ASSERT(RuntimeEnabledFeatures::xsltEnabled());

[abarth-chromium, 2013/10/01 15:06:09]

All these ASSERTs are super redundant.  This is the only one that matters for
the whole fetch directory.  Maybe remove the others and keep this one?

[vivekg__, 2013/10/02 14:18:23]

Done. Also removed one ASSERT check before the new XSLStyleSheetResource
creation in ResourceFetcher.cpp. Should we also remove all the other ASSERTs
which are there in ResourceFetcher.cpp?

     DEFINE_STATIC_LOCAL(const AtomicString, acceptXSLT, ("text/xml, application/xml, application/xhtml+xml, text/xsl, application/rss+xml, application/atom+xml", AtomicString::ConstructFromLiteral));
 
     // It's XML we want.
     // FIXME: This should accept more general xml formats */*+xml, image/svg+xml for example.
     setAccept(acceptXSLT);
 }
 
 void XSLStyleSheetResource::didAddClient(ResourceClient* c)
 {
     ASSERT(c->resourceClientType() == StyleSheetResourceClient::expectedType());
@@ skipping 18 matching lines @@
         m_sheet = m_decoder->decode(m_data->data(), encodedSize());
         m_sheet.append(m_decoder->flush());
     }
 
     ResourceClientWalker<StyleSheetResourceClient> w(m_clients);
     while (StyleSheetResourceClient* c = w.next())
         c->setXSLStyleSheet(m_resourceRequest.url(), m_response.url(), m_sheet);
 }
 
 } // namespace WebCore