commit 9a423ff19e3a7503cd8128998b3a10d65c499fdc
author Gilad Arnold <garnold@chromium.org> Thu Mar 27 15:27:35 2014 -0700
committer chrome-internal-fetch <chrome-internal-fetch@google.com> Fri Mar 28 17:05:29 2014 +0000
tree da3cf4e00b876ee90267b1dfc380d484fff1d447
parent b6a039fd3f601db455e2990e157eefb10f5c724e

Removed redundant clause in conditional.

BUG=None
TEST=None

Change-Id: I9f61e480c09b10e7ff4c7bf5d0f3630e91b678ec
Reviewed-on: https://chromium-review.googlesource.com/192011
Tested-by: Gilad Arnold <garnold@chromium.org>
Reviewed-by: Alex Deymo <deymo@chromium.org>
Commit-Queue: Gilad Arnold <garnold@chromium.org>

connection_manager.cc

diff --git a/connection_manager.cc b/connection_manager.cc
index 88cbe57..49d19d4 100644
--- a/connection_manager.cc
+++ b/connection_manager.cc
@@ -202,8 +202,7 @@
       if (device_policy->GetAllowedConnectionTypesForUpdate(&allowed_types)) {
         // The update setting is enforced by the device policy.
 
-        if ((type == kNetCellular &&
-            !ContainsKey(allowed_types, shill::kTypeCellular))) {
+        if (!ContainsKey(allowed_types, shill::kTypeCellular)) {
           LOG(INFO) << "Disabling updates over cellular connection as it's not "
                        "allowed in the device policy.";
           return false;