src/java.net.http/share/classes/jdk/internal/net/http/frame/SettingsFrame.java
changeset 51231 1d8b1d4eae6a
parent 50985 cd41f34e548c
child 56842 43c3b82728eb
--- a/src/java.net.http/share/classes/jdk/internal/net/http/frame/SettingsFrame.java	Tue Jul 24 13:55:57 2018 +0800
+++ b/src/java.net.http/share/classes/jdk/internal/net/http/frame/SettingsFrame.java	Mon Jul 23 11:47:03 2018 +0100
@@ -161,15 +161,20 @@
         }
     }
 
-    public static final int DEFAULT_INITIAL_WINDOW_SIZE = 64 * K -1;
+    // The initial value is 4,096 octets.
     public static final int DEFAULT_HEADER_TABLE_SIZE = 4 * K;
-    public static final int DEFAULT_MAX_CONCURRENT_STREAMS = 100;
+    // The initial value is 1, which indicates that server push is permitted.
+    public static final int DEFAULT_ENABLE_PUSH = 1;
+    // Initially, there is no limit to this value. This limit is directional.
+    public static final int DEFAULT_MAX_CONCURRENT_STREAMS = Integer.MAX_VALUE;
+    // The initial value is 2^16-1 (65,535) octets.
+    public static final int DEFAULT_INITIAL_WINDOW_SIZE = 64 * K -1;
+    // The initial value is 2^14 (16,384) octets.
     public static final int DEFAULT_MAX_FRAME_SIZE = 16 * K;
 
-    public static SettingsFrame getDefaultSettings() {
+    public static SettingsFrame defaultRFCSettings() {
         SettingsFrame f = new SettingsFrame();
-        // TODO: check these values
-        f.setParameter(ENABLE_PUSH, 1);
+        f.setParameter(ENABLE_PUSH, DEFAULT_ENABLE_PUSH);
         f.setParameter(HEADER_TABLE_SIZE, DEFAULT_HEADER_TABLE_SIZE);
         f.setParameter(MAX_CONCURRENT_STREAMS, DEFAULT_MAX_CONCURRENT_STREAMS);
         f.setParameter(INITIAL_WINDOW_SIZE, DEFAULT_INITIAL_WINDOW_SIZE);