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

Issue 4452066: Add support for long messages in IE 6/7 to IFPC (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
14 years, 11 months ago by Javier Pedemonte
Modified:
14 years, 10 months ago
Reviewers:
johnfargo, dev-remailer, plindner1
CC:
fargo
Base URL:
http://svn.apache.org/repos/asf/shindig/trunk/
Visibility:
Public.

Description

For IFPC transport, split long messages (> 2000 characters) into chunks. Necessary in order to avoid URL character limit on IE 6 & 7. https://issues.apache.org/jira/browse/SHINDIG-1530

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+59 lines, -6 lines) Patch
M features/src/main/javascript/features/rpc/ifpc.transport.js View 4 chunks +58 lines, -4 lines 0 comments Download
M features/src/main/javascript/features/rpc/rpc.js View 1 chunk +1 line, -2 lines 0 comments Download

Messages

Total messages: 6
plindner1
John, is this relevant anymore?
14 years, 11 months ago (2011-05-09 04:44:50 UTC) #1
fargo
Not for any use cases I'm aware. @Javier, who on your side still makes use ...
14 years, 11 months ago (2011-05-09 19:58:23 UTC) #2
plindner1
Safe to apply then? Also, safe to get rid of legacy mode? On Mon, May ...
14 years, 11 months ago (2011-05-09 20:11:15 UTC) #3
Javier Pedemonte
On 2011/05/09 19:58:23, fargo wrote: > Not for any use cases I'm aware. @Javier, who ...
14 years, 11 months ago (2011-05-09 20:25:12 UTC) #4
Javier Pedemonte
On 2011/05/09 19:58:23, fargo wrote: > Not for any use cases I'm aware. @Javier, who ...
14 years, 11 months ago (2011-05-09 20:32:30 UTC) #5
johnfargo
14 years, 11 months ago (2011-05-09 20:40:59 UTC) #6
Yes, and no. :(

On Monday, May 9, 2011, Paul Lindner <plindner@google.com> wrote:
> Safe to apply then?
> Also, safe to get rid of legacy mode?
>
> On Mon, May 9, 2011 at 12:58 PM, John Hjelmstad <fargo@google.com> wrote:
> Not for any use cases I'm aware. @Javier, who on your side still makes use of
IFPC?
>
> On Sun, May 8, 2011 at 9:44 PM,  <plindner@google.com> wrote:
> John, is this relevant anymore?
>
> http://codereview.appspot.com/4452066/
>
>
>
>
>
Sign in to reply to this message.

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