Bug 582174: IsDead() was intended to check for null. r=dwitte
authorChris Jones <jones.chris.g@gmail.com>
Tue, 27 Jul 2010 01:26:41 -0500
changeset 48426 b400cce8d06fe114eccaa6b46dd902951f447323
parent 48425 f1fa3a94646653e951d4d242f617e7bb1beaa2e2
child 48427 a4494a32a9110a6d4c3dc30aea2b3419744ab6e6
push id14728
push userdougt@mozilla.com
push dateFri, 30 Jul 2010 19:28:54 +0000
treeherderautoland@a4494a32a911 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersdwitte
bugs582174
milestone2.0b3pre
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 582174: IsDead() was intended to check for null. r=dwitte
dom/ipc/ContentChild.cpp
--- a/dom/ipc/ContentChild.cpp
+++ b/dom/ipc/ContentChild.cpp
@@ -99,17 +99,17 @@ public:
 
     /**
      * Return true if this observer can no longer receive
      * notifications.
      */
     bool IsDead() const
     {
         nsCOMPtr<nsIObserver> observer = GetObserver();
-        return !!observer;
+        return !observer;
     }
 
     /**
      * Return true iff a request to remove observers matching
      * <aObserver, aDomain, aPrefRoot> entails removal of this.
      */
     bool ShouldRemoveFrom(nsIObserver* aObserver,
                           const nsCString& aPrefRoot,