RESOLVED FIXED 64221
update the boilerplate documentation in the test_expectations.txt files 
https://bugs.webkit.org/show_bug.cgi?id=64221
Summary update the boilerplate documentation in the test_expectations.txt files
Dirk Pranke
Reported 2011-07-08 17:43:47 PDT
update the boilerplate documentation in the test_expectations.txt files
Attachments
Patch (10.78 KB, patch)
2011-07-08 17:46 PDT, Dirk Pranke 		eric: review+
eric: commit-queue-
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Dirk Pranke
Comment 1 2011-07-08 17:46:00 PDT
Created attachment 100191 [details] Patch
Eric Seidel (no email)
Comment 2 2011-07-09 03:41:14 PDT
Comment on attachment 100191 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=100191&action=review > LayoutTests/platform/mac/test_expectations.txt:1 > +// These are the layout test expectations for the Apple Macintosh port of WebKit. I don't think "macintosh" exists anymore. :) Just "Mac". Also it's "Apple's" Mac port, since Chromium owns a differnet Mac port, as does Qt.
Dirk Pranke
Comment 3 2011-07-09 10:19:31 PDT
Comment on attachment 100191 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=100191&action=review >> LayoutTests/platform/mac/test_expectations.txt:1 >> +// These are the layout test expectations for the Apple Macintosh port of WebKit. > > I don't think "macintosh" exists anymore. :) Just "Mac". Also it's "Apple's" Mac port, since Chromium owns a differnet Mac port, as does Qt. Really? Heh. Anyway, good point about "Apple's" vs. "Apple". It felt weird to me (even weirder in the "Apple Win port", below), but I don't know why I didn't think to use the possessive.
Dirk Pranke
Comment 4 2011-07-11 16:33:42 PDT
Committed r90792: <http://trac.webkit.org/changeset/90792>
Note You need to log in before you can comment on or make changes to this bug.