[WebSocket] bufferedAmount should not decrease inside a task.

Source/modules/websockets/WebSocketChannelClient.h

 /*
  * Copyright (C) 2011 Google Inc.  All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 19 matching lines @@
 
 #ifndef WebSocketChannelClient_h
 #define WebSocketChannelClient_h
 
 #include "wtf/Forward.h"
 #include "wtf/PassOwnPtr.h"
 #include "wtf/Vector.h"
 
 namespace WebCore {
 
-    class WebSocketChannelClient {
-    public:
-        virtual ~WebSocketChannelClient() { }
-        virtual void didConnect() { }
-        virtual void didReceiveMessage(const String&) { }
-        virtual void didReceiveBinaryData(PassOwnPtr<Vector<char> >) { }
-        virtual void didReceiveMessageError() { }
-        virtual void didUpdateBufferedAmount(unsigned long /* bufferedAmount */) { }
-        virtual void didStartClosingHandshake() { }
-        enum ClosingHandshakeCompletionStatus {
-            ClosingHandshakeIncomplete,
-            ClosingHandshakeComplete
-        };
-        virtual void didClose(unsigned long /* unhandledBufferedAmount */, ClosingHandshakeCompletionStatus, unsigned short /* code */, const String& /* reason */) { }
+class WebSocketChannelClient {
+public:
+    virtual ~WebSocketChannelClient() { }
+    virtual void didConnect() { }
+    virtual void didReceiveMessage(const String&) { }
+    virtual void didReceiveBinaryData(PassOwnPtr<Vector<char> >) { }
+    virtual void didReceiveMessageError() { }
+    // These two functions are devided in order to keep the WebSocket API:

[tyoshino (SeeGerritForStatus), 2014/06/09 09:24:26]

keep -> follow?

and how about s/API/API spec/

[yhirano, 2014/06/09 11:59:51]

Done.

+    // The amount should be exposed immediately and the amount should
+    // be exposed behind.

[tyoshino (SeeGerritForStatus), 2014/06/09 09:24:26]

couldn't get the meaning of this.

[yhirano, 2014/06/09 11:59:51]

Done.

+    virtual void didIncreaseBufferedAmount(unsigned long amountOfIncrease) { }
+    virtual void didDecreaseBufferedAmount(unsigned long amountOfDecrease) { }
+    virtual void didStartClosingHandshake() { }
+    enum ClosingHandshakeCompletionStatus {
+        ClosingHandshakeIncomplete,
+        ClosingHandshakeComplete
+    };
+    virtual void didClose(unsigned long /* unhandledBufferedAmount */, ClosingHandshakeCompletionStatus, unsigned short /* code */, const String& /* reason */) { }
 
-    protected:
-        WebSocketChannelClient() { }
-    };
+protected:
+    WebSocketChannelClient() { }
+};
 
 } // namespace WebCore
 
 #endif // WebSocketChannelClient_h