Created attachment 157524 [details]
Attachment 157524 [details] did not pass style-queue:
Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/WebKit/ChangeLog', u'Source/WebKit/..." exit_code: 1
Source/WebKit/blackberry/Api/WebPage.h:24: Alphabetical sorting problem. [build/include_order] 
Total errors found: 1 in 8 files
If any of these errors are false positives, please file a bug against check-webkit-style.
Created attachment 157529 [details]
Fixed style error
Comment on attachment 157529 [details]
View in context: https://bugs.webkit.org/attachment.cgi?id=157529&action=review
I think this looks good overall. Please fix the issues and then I'll look one last time.
> + // Output warning?
ASSERT_NOT_REACHED() I would say.
Remove the extra new line here.
Created attachment 157551 [details]
Corrections from Adam's review
Comment on attachment 157551 [details]
View in context: https://bugs.webkit.org/attachment.cgi?id=157551&action=review
nit: extra blank line.
> + Reviewed by Joe Mason
Should be "Internally reviewed by Joe"
Created attachment 157771 [details]
Changes from Antonio Gomes comments, (remove blank link and change review by link in the changelogs and commit message)
Comment on attachment 157771 [details]
Clearing flags on attachment: 157771
Committed r125324: <http://trac.webkit.org/changeset/125324>
All reviewed patches have been landed. Closing bug.