[Webkit-unassigned] [Bug 42093] Change handling of -khtml- and -apple- vendor-prefixes

Wed Jul 14 01:57:50 PDT 2010

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

Eric Seidel <eric at webkit.org> changed:

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

--- Comment #5 from Eric Seidel <eric at webkit.org>  2010-07-14 01:57:50 PST ---
(From update of attachment 61342)
WebCore/css/CSSParser.cpp:5583
 +          if (!strcmp(buffer, "-apple-dashboard-region") || !strcmp(buffer, "-apple-line-clamp")) {
This looks wrong.  Previously the variable "length" was updated by "hasPrefix".  That's no longer the case, so the memmove will be wrong.  Do we not have tests for these?  We should add them.

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