[Webkit-unassigned] [Bug 26154] SecurityOrigin::createFromDatabaseIdentifier should handle _'s in the hostname

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=26154


darin at apple.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #30891|review?                     |review+
               Flag|                            |




------- Comment #2 from darin at apple.com  2009-06-03 09:05 PDT -------
(From update of attachment 30891)
> +        WARNING: NO TEST CASES ADDED OR CHANGED

This is something you're supposed to delete if you either add a test case or
decide you can't add one. You shouldn't submit patches with this in them. We
don't want to check this warning into the change log!

> +    // Ensure there were at least 2 seperator characters.  Some hostnames on intranets have
> +    // underscores in them, so we'll assume that any additional underscores are part of the host.

Typo in this comment ("seperator"). Also, we use only one space after periods
in our comments.

Seems OK to land this without a test. To make a test we'd have to change
run-webkit-tests to set up the Apache server to run with a hostname that had an
underscore in it. Not impossible, but perhaps difficult.

It would also be good to have tests for the existing behavior, for example,
illegal database identifiers that fall into the failure case.

r=me


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