Bug 1338086 - Remove useless else blocks in order to reduce complexity in toolkit/components/telemetry/ r=Dexter

MozReview-Commit-ID: EjMufQ1PReI

--HG--
extra : rebase_source : 0ff54a384e3f9e39eec74c894f705ee51412d88c

toolkit/components/telemetry/TelemetryHistogram.cpp

@@ -851,9 +851,8 @@ internal_ShouldReflectHistogram(Histogram *h)
       return false;
     }
     return true;
-  } else {
-    return !gCorruptHistograms[id];
   }
+  return !gCorruptHistograms[id];
 }
 
 } // namespace

toolkit/components/telemetry/TelemetryScalar.cpp

@@ -1103,7 +1103,8 @@ internal_UpdateScalar(const nsACString& aName, ScalarActionType aType,
 
   if (aType == ScalarActionType::eAdd) {
     return scalar->AddValue(aValue);
-  } else if (aType == ScalarActionType::eSet) {
+  }
+  if (aType == ScalarActionType::eSet) {
     return scalar->SetValue(aValue);
   }
 
@@ -1244,7 +1245,8 @@ internal_UpdateKeyedScalar(const nsACString& aName, const nsAString& aKey,
 
   if (aType == ScalarActionType::eAdd) {
     return scalar->AddValue(aKey, aValue);
-  } else if (aType == ScalarActionType::eSet) {
+  }
+  if (aType == ScalarActionType::eSet) {
     return scalar->SetValue(aKey, aValue);
   }