Avoid silly crash in ContextFeatures::urlDidChange

Source/core/dom/ContextFeatures.h

 /*
  * Copyright (C) 2012 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Neither the name of Google Inc. nor the names of its
@@ skipping 84 matching lines @@
 
 inline bool ContextFeatures::isEnabled(Document* document, FeatureType type, bool defaultValue) const
 {
     if (!m_client)
         return defaultValue;
     return m_client->isEnabled(document, type, defaultValue);
 }
 
 inline void ContextFeatures::urlDidChange(Document* document)
 {
-    if (m_client)
-        return;
-    m_client->urlDidChange(document);
+    // FIXME: The original code, commented out below, is obviously
+    // wrong, but the seemingly correct fix of negating the test to
+    // the more logical 'if (!m_client)' crashes the renderer.
+    // See issue 294180
+    //
+    // if (m_client)
+    //     return;
+    // m_client->urlDidChange(document);
 }
 
 } // namespace WebCore
 
 #endif // ContextFeatures_h