Bug 53630

Created attachment 80976 [details] Fix v1 Chrome try runs in progress: http://build.chromium.org/p/tryserver.chromium/builders/mac/builds/11309 http://build.chromium.org/p/tryserver.chromium/builders/linux/builds/10901 http://build.chromium.org/p/tryserver.chromium/builders/win/builds/13703 Note for checkin: WebCore/WebCore.gyp/mac/check_objc_rename.sh needs to have the executable bit set. bash$ svn propset svn:executable '*' WebCore/WebCore.gyp/mac/check_objc_rename.sh

Attachment 80976 [details] did not pass style-queue: Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/WebCore/ChangeLog', u'Source/WebCor..." exit_code: 1 Source/WebCore/WebCore.gyp/WebCore.gyp:1025: Line contains tab character. [whitespace/tab] [5] Total errors found: 1 in 3 files If any of these errors are false positives, please file a bug against check-webkit-style.

Created attachment 80977 [details] Fix v2 (no tabs)

Created attachment 81026 [details] Fix v3 (no nm -U) Apparently, nm -U (don’t display undefined symbols) is new in cctools 782 (Xcode 3.2.3). Chrome still needs to support 3.2.2 (and, for some real stragglers, 3.1.5), so the -U needs to go. This has no impact on the script, it’ll work just as well without -U.

Comment on attachment 81026 [details] Fix v3 (no nm -U) ok.

Comment on attachment 81026 [details] Fix v3 (no nm -U) Clearing flags on attachment: 81026 Committed r77562: <http://trac.webkit.org/changeset/77562>

All reviewed patches have been landed. Closing bug.

The commit-queue encountered the following flaky tests while processing attachment 81026 [details]: http/tests/websocket/tests/handshake-challenge-randomness.html bug 53738 (author: abarth@webkit.org) The commit-queue is continuing to process your patch.

I think this may have broken the mac build ...

Committed r77589: <http://trac.webkit.org/changeset/77589>

Dirk, can you please show me the Mac build break you encountered?

OK, I found http://build.webkit.org/builders/Chromium%20Mac%20Release/builds/18787. The errors in the build log: PhaseScriptExecution "Postbuild \"Check Objective-C Rename\"" ../../WebKit/chromium/xcodebuild/WebCore.build/Release/webcore_html.build/Script-C80125C94888F34350BF52AA.sh cd /Users/chrome-bot/Desktop/WebKit-BuildSlave/chromium-mac-release/build/Source/WebCore/WebCore.gyp /bin/sh -c /Users/chrome-bot/Desktop/WebKit-BuildSlave/chromium-mac-release/build/Source/WebCore/WebCore.gyp/../../WebKit/chromium/xcodebuild/WebCore.build/Release/webcore_html.build/Script-C80125C94888F34350BF52AA.sh /Users/chrome-bot/Desktop/WebKit-BuildSlave/chromium-mac-release/build/Source/WebCore/WebCore.gyp/../../WebKit/chromium/xcodebuild/WebCore.build/Release/webcore_html.build/Script-C80125C94888F34350BF52AA.sh: line 2: /Users/chrome-bot/Desktop/WebKit-BuildSlave/chromium-mac-release/build/Source/WebCore/WebCore.gyp/mac/check_objc_rename.sh: Permission denied /Users/chrome-bot/Desktop/WebKit-BuildSlave/chromium-mac-release/build/Source/WebCore/WebCore.gyp/../../WebKit/chromium/xcodebuild/WebCore.build/Release/webcore_html.build/Script-C80125C94888F34350BF52AA.sh: line 2: exec: /Users/chrome-bot/Desktop/WebKit-BuildSlave/chromium-mac-release/build/Source/WebCore/WebCore.gyp/mac/check_objc_rename.sh: cannot execute: Unknown error: 0 and then mark@rj bash$ svn proplist -v http://svn.webkit.org/repository/webkit/trunk/Source/WebCore/WebCore.gyp/mac/check_objc_rename.sh@77562 mark@rj bash$ (as in “no properties set.”) This looks like check_objc_rename.sh was checked in without the svn:executable property set as I recommended in comment 1. When I asked Dimitri to commit-queue this, I also asked whether the commit-queue supported setting svn:executable, and he said he didn’t know but there was only one way to find out. :) The v3 patch from this patch set should be checked in manually, with the svn:executable property set properly on check_objc_rename.sh.

Yup, that sounds like the problem. Thanks Mark! I imagine either Dimitri or I can land it manually in the morning.

http://trac.webkit.org/changeset/77562 might have broken GTK Linux 32-bit Debug and GTK Linux 64-bit Debug The following tests are not passing: fast/events/pagehide-timeout.html

(In reply to comment #13) > Yup, that sounds like the problem. Thanks Mark! I imagine either Dimitri or I can land it manually in the morning. I don't have an svn checkout.

It ought to be easy to pull an svn tree to check this in. I guess there’s no way to set svn properties from git? Dirk, do you have a WebKit svn checkout handy?

Yes, I do. I'll land it when I get into work in a little while.

Committed r77663. <http://trac.webkit.org/changeset/77663>

http://trac.webkit.org/changeset/77663 might have broken GTK Linux 32-bit Debug and GTK Linux 64-bit Debug The following tests are not passing: fast/blockflow/Kusa-Makura-background-canvas.html fast/blockflow/japanese-ruby-horizontal-bt.html fast/blockflow/japanese-ruby-vertical-lr.html fast/blockflow/japanese-ruby-vertical-rl.html fast/ruby/nested-ruby.html fast/text/emphasis-avoid-ruby.html