[Webkit-unassigned] [Bug 34539] Implement NPN_GetValueForURL and NPN_SetValueForURL and provide a stub for NPN_GetAuthenticationInfo

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Mon Jun 21 19:20:48 PDT 2010


Adam Barth <abarth at webkit.org> changed:

           What    |Removed                     |Added
  Attachment #55729|review?                     |review-
               Flag|                            |

--- Comment #29 from Adam Barth <abarth at webkit.org>  2010-06-21 19:20:48 PST ---
(From update of attachment 55729)
This patch needs a lot of love.

 +          NPN_GetAuthenticationInfo,
This ChangeLog doesn't tell me any of the "why" behind this patch.  I can see the "what" by looking at the code, but I don't understand what problem we're solving with this patch.

 +          KURL URL = KURL(ParsedURLString, url); // NSURL *URL = [self URLWithCString:url];
Why would |url| be a ParsedURLString?  That doesn't seem right.

 +                  strcpy(*value, cookieStringUTF8.data());
please use strncpy !

 +  #if defined(BUILDING_QT__)
Surely this is the wrong preprocessor check.  Also, why do we have Qt-specific code in a non-Qt-specific file?  I think we need some better kind of abstraction here.

 +          memcpy(*value, proxyUTF8.data(), proxyUTF8.length());
Why do you use memcpy here but strcpy above?  One of the two must be wrong.

 +      }
You should have an explicit default case in to handle garbage from our caller.

 +      }
You should have an explicit default case in to handle garbage from our caller.

 +              KURL URL = KURL(ParsedURLString, url); // NSURL *URL = [self URLWithCString:url];
same ParsedURLString comment here.

 +                  String cookieString = String::fromUTF8(value, len);
Are you sure this is supposed to be UTF8?

 +                      Frame* frame = pluginViewForInstance(instance)->parentFrame();
parentFrame?  Could be, but looks scary.

 +                          setCookies(frame->document(), URL, cookieString);
URL should be in lower case.

Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

More information about the webkit-unassigned mailing list