Skip to content

Commit 07391f0

Browse files
authored
Simplify the wifi vs. mobile data connection code. (#3469)
1 parent e520b78 commit 07391f0

File tree

1 file changed

+7
-23
lines changed

1 file changed

+7
-23
lines changed

GoogleDataTransport/GDTLibrary/GDTUploadCoordinator.m

Lines changed: 7 additions & 23 deletions
Original file line numberDiff line numberDiff line change
@@ -133,38 +133,22 @@ - (void)uploadTargets:(NSArray<NSNumber *> *)targets conditions:(GDTUploadCondit
133133
*/
134134
- (GDTUploadConditions)uploadConditions {
135135
SCNetworkReachabilityFlags currentFlags = [GDTReachability currentFlags];
136-
137136
BOOL reachable =
138137
(currentFlags & kSCNetworkReachabilityFlagsReachable) == kSCNetworkReachabilityFlagsReachable;
139138
BOOL connectionRequired = (currentFlags & kSCNetworkReachabilityFlagsConnectionRequired) ==
140139
kSCNetworkReachabilityFlagsConnectionRequired;
141-
BOOL interventionRequired = (currentFlags & kSCNetworkReachabilityFlagsInterventionRequired) ==
142-
kSCNetworkReachabilityFlagsInterventionRequired;
143-
BOOL connectionOnDemand = (currentFlags & kSCNetworkReachabilityFlagsConnectionOnDemand) ==
144-
kSCNetworkReachabilityFlagsConnectionOnDemand;
145-
BOOL connectionOnTraffic = (currentFlags & kSCNetworkReachabilityFlagsConnectionOnTraffic) ==
146-
kSCNetworkReachabilityFlagsConnectionOnTraffic;
147-
BOOL isWWAN = GDTReachabilityFlagsContainWWAN(currentFlags);
140+
BOOL networkConnected = reachable && !connectionRequired;
148141

149-
if (!reachable) {
142+
if (!networkConnected) {
150143
return GDTUploadConditionNoNetwork;
151144
}
152145

153-
GDTUploadConditions conditions = 0;
154-
conditions |= !connectionRequired ? GDTUploadConditionWifiData : conditions;
155-
conditions |= isWWAN ? GDTUploadConditionMobileData : conditions;
156-
if ((connectionOnTraffic || connectionOnDemand) && !interventionRequired) {
157-
conditions = GDTUploadConditionWifiData;
158-
}
159-
160-
BOOL wifi = (conditions & GDTUploadConditionWifiData) == GDTUploadConditionWifiData;
161-
BOOL cell = (conditions & GDTUploadConditionMobileData) == GDTUploadConditionMobileData;
162-
163-
if (!(wifi || cell)) {
164-
conditions = GDTUploadConditionUnclearConnection;
146+
BOOL isWWAN = GDTReachabilityFlagsContainWWAN(currentFlags);
147+
if (isWWAN) {
148+
return GDTUploadConditionMobileData;
149+
} else {
150+
return GDTUploadConditionWifiData;
165151
}
166-
167-
return conditions;
168152
}
169153

170154
#pragma mark - NSSecureCoding support

0 commit comments

Comments
 (0)