Bug 42218 - ES5 requires BOMs to be treated as whitespace
Summary: ES5 requires BOMs to be treated as whitespace
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: Other OS X 10.5
: P2 Normal
Assignee: Oliver Hunt
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-07-13 17:46 PDT by Oliver Hunt
Modified: 2010-07-13 18:35 PDT (History)
5 users (show)

See Also:


Attachments
Patch (20.74 KB, patch)
2010-07-13 17:50 PDT, Oliver Hunt
barraclough: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Oliver Hunt 2010-07-13 17:46:14 PDT
ES5 requires BOMs to be treated as whitespace
Comment 1 Oliver Hunt 2010-07-13 17:50:36 PDT
Created attachment 61440 [details]
Patch
Comment 2 Gavin Barraclough 2010-07-13 17:54:17 PDT
Comment on attachment 61440 [details]
Patch

it would be nice if we could unify our whitespace checking methods a little – isStrWhiteSpace in JSGlobalObjectFunctions duplicates some of this.
Comment 3 Oliver Hunt 2010-07-13 18:02:29 PDT
Committed r63273: <http://trac.webkit.org/changeset/63273>
Comment 4 WebKit Review Bot 2010-07-13 18:14:48 PDT
http://trac.webkit.org/changeset/63273 might have broken Chromium Linux Release
Comment 5 Tony Gentilcore 2010-07-13 18:26:22 PDT
(In reply to comment #4)
> http://trac.webkit.org/changeset/63273 might have broken Chromium Linux Release

If this broke Chromium, it might be because this should have been reverted along with this change:
http://trac.webkit.org/changeset/63162
Comment 6 Oliver Hunt 2010-07-13 18:34:30 PDT
(In reply to comment #5)
> (In reply to comment #4)
> > http://trac.webkit.org/changeset/63273 might have broken Chromium Linux Release
> 
> If this broke Chromium, it might be because this should have been reverted along with this change:
> http://trac.webkit.org/changeset/63162

Yup, bug 42224 is the rollout patch, i have a vague hope the commit bot will actually commit it at some point.
Comment 7 WebKit Review Bot 2010-07-13 18:35:05 PDT
Attachment 61440 [details] did not build on chromium:
Build output: http://webkit-commit-queue.appspot.com/results/3426289