HTMLElement.cpp does not compile with new clang

<http://webkit.org/b/136600>

Reviewed by Chris Fleizach.

Fixes the following static analyzer warning:

    Source/WebCore/html/HTMLElement.cpp:545:10: error: 'this' pointer cannot be null in well-defined C++ code; pointer may be assumed to always convert to true [-Werror,-Wundefined-bool-conversion]
        if (!this || !parentNode())
            ~^~~~

* html/HTMLElement.cpp:
(WebCore::HTMLElement::setOuterText): Remove "!this" check added
in r75738.  It would only cover up real bugs, and isn't even hit
in layout tests.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@173355 268f45cc-cd09-0410-ab3c-d52691b4dbfc

diff --git a/Source/WebCore/ChangeLog b/Source/WebCore/ChangeLog
index 4cd14cbc2ddf1f57cde06506c4c69ed79556b05f..9b0ea0abab0b9fac1b1eb75e94932e683e7e68e4 100644 (file)
--- a/Source/WebCore/ChangeLog
+++ b/Source/WebCore/ChangeLog
@@ -1,3 +1,21 @@
+2014-09-06  David Kilzer  <ddkilzer@apple.com>
+
+        HTMLElement.cpp does not compile with new clang
+        <http://webkit.org/b/136600>
+
+        Reviewed by Chris Fleizach.
+
+        Fixes the following static analyzer warning:
+
+            Source/WebCore/html/HTMLElement.cpp:545:10: error: 'this' pointer cannot be null in well-defined C++ code; pointer may be assumed to always convert to true [-Werror,-Wundefined-bool-conversion]
+                if (!this || !parentNode())
+                    ~^~~~
+
+        * html/HTMLElement.cpp:
+        (WebCore::HTMLElement::setOuterText): Remove "!this" check added
+        in r75738.  It would only cover up real bugs, and isn't even hit
+        in layout tests.
+
 2014-09-06  Ryuan Choi  <ryuan.choi@gmail.com>
 
         [EFL] Drop evas object cursor support

diff --git a/Source/WebCore/html/HTMLElement.cpp b/Source/WebCore/html/HTMLElement.cpp
index f5e52e9b17975298f8e0467a868695b07bad2bf1..a6d5758125aac5936d8bbc59e00cdb8da1fe67e7 100644 (file)
--- a/Source/WebCore/html/HTMLElement.cpp
+++ b/Source/WebCore/html/HTMLElement.cpp
@@ -542,7 +542,7 @@ void HTMLElement::setOuterText(const String& text, ExceptionCode& ec)
     else
         newChild = Text::create(document(), text);
 
-    if (!this || !parentNode())
+    if (!parentNode())
         ec = HIERARCHY_REQUEST_ERR;
     if (ec)
         return;