Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(260)

Unified Diff: components/cronet/ios/Cronet.mm

Issue 2733253002: [Cronet] Don't claim support of UrlRequests with data URL scheme. (Closed)
Patch Set: Created 3 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | components/cronet/ios/test/cronet_http_test.mm » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/cronet/ios/Cronet.mm
diff --git a/components/cronet/ios/Cronet.mm b/components/cronet/ios/Cronet.mm
index d4217ab217f41b0c46e797cbacdac92a0c19dae3..b6aaeb13eab15f2d59f7f9b3bfd0db6a5aafc4f4 100644
--- a/components/cronet/ios/Cronet.mm
+++ b/components/cronet/ios/Cronet.mm
@@ -91,8 +91,7 @@ class CronetHttpProtocolHandlerDelegate
NSString* scheme = [[request URL] scheme];
if (!scheme)
return false;
- return [scheme caseInsensitiveCompare:@"data"] == NSOrderedSame ||
- [scheme caseInsensitiveCompare:@"http"] == NSOrderedSame ||
+ return [scheme caseInsensitiveCompare:@"http"] == NSOrderedSame ||
[scheme caseInsensitiveCompare:@"https"] == NSOrderedSame;
}
« no previous file with comments | « no previous file | components/cronet/ios/test/cronet_http_test.mm » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698