[Webkit-unassigned] [Bug 40204] New: Need to recognize "; " as separator for arguments of viewport tag

[bugzilla-daemon at webkit.org]

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

           Summary: Need to recognize ";" as separator for arguments of
                    viewport tag
           Product: WebKit
           Version: 528+ (Nightly build)
          Platform: All
        OS/Version: All
            Status: UNCONFIRMED
          Severity: Normal
          Priority: P2
         Component: WebCore Misc.
        AssignedTo: webkit-unassigned at lists.webkit.org
        ReportedBy: gyuyoung.kim at samsung.com


When arguments of viewport's tag are processed, ";" is not recognized as separator as below,

static bool isSeparator(UChar c)
{
    return c == ' ' || c == '\t' || c == '\n' || c == '\r' || c == '=' || c == ',' || c == '\0';
}

However, some website is using ";" as separator.
For example, in http://m.flickr.com/#/home, it is using ";" for arguments of viewport as below,
<meta name="viewport" content="width=device-width; initial-scale=1.0; maximum-scale=1.0; user-scalable=0;">

So, the "device-width;" argument is not processed by processArguments() function because the "deveice-width;" contains ";".
In my opinion, we need to add ";" to isSeparator() as below,

static bool isSeparator(UChar c)
{
    return c == ' ' || c == '\t' || c == '\n' || c == '\r' || c == '=' || c == ',' || c == '\0' || c == ';';
}

If there are better solutions, please let me know.

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