Here is the webkit-dev thread about it without any objection: https://lists.webkit.org/pipermail/webkit-dev/2014-August/026829.html
Created attachment 238372 [details] WIP patch for EWS
Created attachment 238373 [details] WIP patch for EWS
Created attachment 238374 [details] WIP patch for EWS
Created attachment 238435 [details] WIP patch for EWS - with win fixes
I know this is WIP, but it looks good to me so far.
Comment on attachment 238435 [details] WIP patch for EWS - with win fixes View in context: https://bugs.webkit.org/attachment.cgi?id=238435&action=review > Source/WTF/wtf/OSAllocator.h:82 > -inline void OSAllocator::decommitAndRelease(void* releaseBase, size_t releaseSize, void* decommitBase, size_t decommitSize) > +inline void OSAllocator::decommitAndRelease(void* releaseBase, size_t releaseSize, void*, size_t) Maybe we can get rid of these parameters from call sides too. > Source/WebCore/platform/graphics/win/GDIExtras.h:-59 > -#endif We can remove the whole header, because it was included only from Source/WebCore/platform/graphics/win/GDIExtras.cpp. > Source/WebCore/platform/win/COMPtr.h:-40 > -#if !OS(WINCE) > -#include <guiddef.h> > -#endif > - It shouldn't be removed because of the guard, but is seems Windows EWS is still happy.
Created attachment 238592 [details] proposed patch addressed my comments in Comment #6 , except the Source/WebCore/platform/win/COMPtr.h change
Comment on attachment 238592 [details] proposed patch I think this patch is ready for review now.
Comment on attachment 238592 [details] proposed patch This looks good, passes the Win EWS, and I just checked that it doesn't break WinCairo. I am disappointed that nobody thinks WinCE is worth maintaining, but I see no reason that this shouldn't go in based on the discussion on webkit-dev. This can always be used as reference later.
Comment on attachment 238592 [details] proposed patch Rejecting attachment 238592 [details] from commit-queue. Failed to run "['/Volumes/Data/EWS/WebKit/Tools/Scripts/webkit-patch', '--status-host=webkit-queues.appspot.com', '--bot-id=webkit-cq-01', 'validate-changelog', '--check-oops', '--non-interactive', 238592, '--port=mac']" exit_code: 1 cwd: /Volumes/Data/EWS/WebKit Last 500 characters of output: self._filesystem.open_text_file_for_reading(self.path) File "/Volumes/Data/EWS/WebKit/Tools/Scripts/webkitpy/common/system/filesystem.py", line 205, in open_text_file_for_reading return codecs.open(path, 'r', 'utf8', errors) File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/codecs.py", line 881, in open file = __builtin__.open(filename, mode, buffering) IOError: [Errno 2] No such file or directory: u'/Volumes/Data/EWS/WebKit/Source/WebKit/wince/ChangeLog' Full output: http://webkit-queues.appspot.com/results/6567887099658240
Landed manually in https://trac.webkit.org/changeset/173949 and https://trac.webkit.org/changeset/173950 , because commit-log-editor can't handle the removed Source/WebKit/wince/ChangeLog file.