RESOLVED FIXED 137372
Use version number 101000 instead of 10100 for OS X Yosemite 
https://bugs.webkit.org/show_bug.cgi?id=137372
Summary Use version number 101000 instead of 10100 for OS X Yosemite
Daniel Bates
Reported 2014-10-02 17:18:19 PDT
We use 10100 to refer to Yosemite in at least two files: In -[WebView _preferencesChanged:] in file Source/WebKit/mac/WebView/WebView.mm, <http://trac.webkit.org/browser/trunk/Source/WebKit/mac/WebView/WebView.mm?rev=173804&format=txt> On line 41 (as of r172542) in file Tools/DumpRenderTree/mac/TextInputController.m, <http://trac.webkit.org/browser/trunk/Tools/DumpRenderTree/mac/TextInputController.m?rev=172542#L41> The version number for Yosemite is 101000 according to the value of MAC_OS_X_VERSION_10_10 in AvailabilityMacros.h in the OS X Yosemite SDK.
Attachments
Patch (2.73 KB, patch)
2014-10-02 17:19 PDT, Daniel Bates 		mrowe: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Daniel Bates
Comment 1 2014-10-02 17:19:53 PDT
Created attachment 239160 [details] Patch
Mark Rowe (bdash)
Comment 2 2014-10-02 17:22:24 PDT
Comment on attachment 239160 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=239160&action=review > Source/WebKit/mac/ChangeLog:3 > + Clean up: Use version number 101000 instead of 10100 for OS X Yosemite This isn't really cleanup. It's a correctness issue.
Daniel Bates
Comment 3 2014-10-02 17:31:06 PDT
Committed r174251: <http://trac.webkit.org/changeset/174251>
Note You need to log in before you can comment on or make changes to this bug.