Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(827)

Issue 166260043: Remove permessage-compress extension support (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
11 years ago by tyoshino (chromium.org)
Modified:
10 years, 12 months ago
Reviewers:
yhirano, Yutaka Hirano
CC:
pywebsocket-reviews_googlegroups.com
Base URL:
http://pywebsocket.googlecode.com/svn/trunk/
Visibility:
Public.

Description

Remove permessage-compress extension support Release note: Remove permessage-compress extension support R=yhirano@chromium.org Committed: https://code.google.com/p/pywebsocket/source/detail?r=860

Patch Set 1 #

Patch Set 2 : Add missing implementation change #

Patch Set 3 : #

Patch Set 4 : Addressed #3 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+66 lines, -290 lines) Patch
M src/mod_pywebsocket/common.py View 1 1 chunk +0 lines, -2 lines 0 comments Download
M src/mod_pywebsocket/extensions.py View 1 2 4 chunks +6 lines, -127 lines 0 comments Download
M src/test/test_extensions.py View 2 chunks +0 lines, -81 lines 0 comments Download
M src/test/test_handshake_hybi.py View 1 8 chunks +22 lines, -35 lines 0 comments Download
M src/test/test_msgutil.py View 5 chunks +27 lines, -33 lines 0 comments Download
M src/test/test_mux.py View 6 chunks +11 lines, -12 lines 0 comments Download

Messages

Total messages: 6
tyoshino (chromium.org)
No hurry
11 years ago (2014-11-06 09:05:24 UTC) #1
tyoshino (chromium.org)
Implementation changes were missing. Added.
11 years ago (2014-11-18 06:23:36 UTC) #2
Yutaka Hirano
Sorry for the late reply. comment modification for PerMessageDeflateExtensionProcessor in extensions.py is needed?
11 years ago (2014-11-18 06:56:17 UTC) #3
tyoshino (chromium.org)
On 2014/11/18 06:56:17, Yutaka Hirano wrote: > Sorry for the late reply. > comment modification ...
11 years ago (2014-11-18 07:27:08 UTC) #4
Yutaka Hirano
lgtm
11 years ago (2014-11-18 09:31:52 UTC) #5
tyoshino (chromium.org)
10 years, 12 months ago (2014-11-25 03:46:19 UTC) #6
Message was sent while issue was closed.
Committed patchset #4 (id:60001) manually as r860 (presubmit successful).
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b