[webkit-changes] cvs commit: WebKit/Plugins.subproj WebBaseNetscapePluginView.m

[Vicki]

vicki       05/07/12 11:23:37

  Modified:    .        Tag: Safari-2-0-branch ChangeLog
               Plugins.subproj Tag: Safari-2-0-branch
                        WebBaseNetscapePluginView.m
  Log:
  	- merged this fix from HEAD
  
      2005-07-11  Kevin Decker  <kdecker at apple.com>
  
          Reviewed by cblu and mjs.
  
  	Fixed: <rdar://problem/4099552> REGRESSION: Safari 1.3 Netscape API NPN_PostURL[Notify]
  	no longer allows manual headers
  
  	Most plugins (flash) send 2 CRFL's between the header and body of their POST requests, while
  	the Adboe plugin sends two LF's. This caused us to send custom headers as part of the actual
  	POST data itself, and correspondently, would skew Content-Length.
  
          * Plugins.subproj/WebBaseNetscapePluginView.m:
          (-[NSData _web_locationAfterFirstBlankLine]): Look for two LF's as well
  
  Revision  Changes    Path
  No                   revision
  
  
  No                   revision
  
  
  1.3118.4.16 +18 -0     WebKit/ChangeLog
  
  Index: ChangeLog
  ===================================================================
  RCS file: /cvs/root/WebKit/ChangeLog,v
  retrieving revision 1.3118.4.15
  retrieving revision 1.3118.4.16
  diff -u -r1.3118.4.15 -r1.3118.4.16
  --- ChangeLog	2 Jul 2005 01:32:41 -0000	1.3118.4.15
  +++ ChangeLog	12 Jul 2005 18:23:34 -0000	1.3118.4.16
  @@ -1,3 +1,21 @@
  +2005-07-12  Vicki Murley  <vicki at apple.com>
  +
  +	- merged this fix from HEAD
  +
  +    2005-07-11  Kevin Decker  <kdecker at apple.com>
  +
  +        Reviewed by cblu and mjs.
  +
  +	Fixed: <rdar://problem/4099552> REGRESSION: Safari 1.3 Netscape API NPN_PostURL[Notify] 
  +	no longer allows manual headers
  +	
  +	Most plugins (flash) send 2 CRFL's between the header and body of their POST requests, while 
  +	the Adboe plugin sends two LF's. This caused us to send custom headers as part of the actual 
  +	POST data itself, and correspondently, would skew Content-Length.
  +
  +        * Plugins.subproj/WebBaseNetscapePluginView.m:
  +        (-[NSData _web_locationAfterFirstBlankLine]): Look for two LF's as well
  +
   2005-07-01  Adele Peterson  <adele at apple.com>
   
           Merged fix from TOT to Safari-2-0-branch
  
  
  
  No                   revision
  
  
  No                   revision
  
  
  1.127.8.1 +8 -1      WebKit/Plugins.subproj/WebBaseNetscapePluginView.m
  
  Index: WebBaseNetscapePluginView.m
  ===================================================================
  RCS file: /cvs/root/WebKit/Plugins.subproj/WebBaseNetscapePluginView.m,v
  retrieving revision 1.127
  retrieving revision 1.127.8.1
  diff -u -r1.127 -r1.127.8.1
  --- WebBaseNetscapePluginView.m	10 Mar 2005 01:00:27 -0000	1.127
  +++ WebBaseNetscapePluginView.m	12 Jul 2005 18:23:37 -0000	1.127.8.1
  @@ -1786,7 +1786,7 @@
       return [self length] > 0 && ((const char *)[self bytes])[0] == '\n';
   }
   
  -// Returns the position after 2 CRLF's or 1 CRLF if it is the first line.
  +
   - (unsigned)_web_locationAfterFirstBlankLine
   {
       const char *bytes = (const char *)[self bytes];
  @@ -1794,6 +1794,13 @@
       
       unsigned i;
       for (i = 0; i < length - 4; i++) {
  +        
  +        //most plugins (Flash) send 2 CRFL's between the header and body of their POST requests, while the adboe plugin sends two LF's.
  +        if (bytes[i] == '\n' && bytes[i+1] == '\n') {
  +            return i+2;
  +        }
  +        
  +        // Returns the position after 2 CRLF's or 1 CRLF if it is the first line.
           if (bytes[i] == '\r' && bytes[i+1] == '\n') {
               i += 2;
               if (i == 2) {