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

Unified Diff: content/child/blink_platform_impl.cc

Issue 1036823003: Fix to respect --explicitly-allowed-ports command line option-Chromium Side (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 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
Index: content/child/blink_platform_impl.cc
diff --git a/content/child/blink_platform_impl.cc b/content/child/blink_platform_impl.cc
index 92bef2ed5d1b440709cebd98cb6e4899cf1da7c1..62302409e3a0ba2c3caa93cb891dca990ab13a0a 100644
--- a/content/child/blink_platform_impl.cc
+++ b/content/child/blink_platform_impl.cc
@@ -504,6 +504,13 @@ bool BlinkPlatformImpl::isReservedIPAddress(
return net::IsIPAddressReserved(address);
}
+bool BlinkPlatformImpl::portAllowed(const unsigned short port, bool isFtp) const
eroman 2015/03/30 19:40:02 is_ftp. also not typical to mark plain old data li
Paritosh Kumar 2015/04/01 13:38:05 Done.
+{
+ if (isFtp)
+ return net::IsPortAllowedByFtp(port) || net::IsPortAllowedByOverride(port);
+ return net::IsPortAllowedByDefault(port) || net::IsPortAllowedByOverride(port);
eroman 2015/03/30 19:40:02 > 80 characters
Paritosh Kumar 2015/04/01 13:38:05 Done.
+}
+
blink::WebThread* BlinkPlatformImpl::createThread(const char* name) {
WebThreadImpl* thread = new WebThreadImpl(name);
thread->TaskRunner()->PostTask(

Powered by Google App Engine
This is Rietveld 408576698