Bug 37481 – Minor fix to pass RenderProgress instead of RenderObject

• Component: Accessibility
• Status: RESOLVED FIXED

[Yael]

Per https://bugs.webkit.org/show_bug.cgi?id=37275#c4, should pass RenderProgress* instead of RenderObject* to AccessibilityProgressIndicator::create().

[Yael]

Created attachment 53215 [details]
Patch, minor.

[WebKit Commit Bot]

Comment on attachment 53215 [details]
Patch, minor.

Rejecting patch 53215 from commit-queue.

Unexpected failure when landing patch!  Please file a bug against webkit-patch.
Failed to run "['WebKitTools/Scripts/webkit-patch', '--status-host=webkit-commit-queue.appspot.com', 'land-attachment', '--force-clean', '--build', '--test', '--non-interactive', '--ignore-builders', '--build-style=both', '--quiet', '53215', '--parent-command=commit-queue', '--no-update']" exit_code: 1
Last 500 characters of output:
mon/config/committers.py:275: UnicodeWarning: Unicode equal comparison failed to convert both arguments to Unicode - interpreting them as being unequal
  if committer.full_name == name:
Tor Arne Vestbø found in /Users/eseidel/Projects/CommitQueue/WebCore/ChangeLog does not appear to be a valid reviewer according to committers.py.
ERROR: /Users/eseidel/Projects/CommitQueue/WebCore/ChangeLog neither lists a valid reviewer nor contains the string "Unreviewed" or "Rubber stamp" (case insensitive).

[Eric Seidel (no email)]

Tor strikes again!  We need to fix committers.py to like him.  Setting cq+ again is not the solution, I have a bug to fix here first.

[Eric Seidel (no email)]

The reviewer_text string parsed with ChangeLogEntry._parse_entry() does not seem to be unicode.  When I attempt to convert it to unicode I get:

Parsing ChangeLog: /Projects/WebKit/WebCore/ChangeLog
Traceback (most recent call last):
  File "/Projects/WebKit/WebKitTools/Scripts/webkit-patch", line 56, in <module>
    main()
  File "/Projects/WebKit/WebKitTools/Scripts/webkit-patch", line 51, in main
    WebKitPatch(__file__).main()
  File "/Projects/WebKit/WebKitTools/Scripts/webkitpy/tool/multicommandtool.py", line 302, in main
    result = command.check_arguments_and_execute(options, args, self)
  File "/Projects/WebKit/WebKitTools/Scripts/webkitpy/tool/multicommandtool.py", line 113, in check_arguments_and_execute
    return self.execute(options, args, tool) or 0
  File "/Projects/WebKit/WebKitTools/Scripts/webkitpy/tool/commands/upload.py", line 57, in execute
    print "%s" % tool.checkout().commit_message_for_this_commit().message()
  File "/Projects/WebKit/WebKitTools/Scripts/webkitpy/common/checkout/api.py", line 99, in commit_message_for_this_commit
    changelog_entry = ChangeLog(changelog_path).latest_entry()
  File "/Projects/WebKit/WebKitTools/Scripts/webkitpy/common/checkout/changelog.py", line 121, in latest_entry
    return self.parse_latest_entry_from_file(changelog_file)
  File "/Projects/WebKit/WebKitTools/Scripts/webkitpy/common/checkout/changelog.py", line 114, in parse_latest_entry_from_file
    return ChangeLogEntry(''.join(entry_lines[:-1]))
  File "/Projects/WebKit/WebKitTools/Scripts/webkitpy/common/checkout/changelog.py", line 53, in __init__
    self._parse_entry()
  File "/Projects/WebKit/WebKitTools/Scripts/webkitpy/common/checkout/changelog.py", line 66, in _parse_entry
    print unicode(self._reviewer_text)
UnicodeDecodeError: 'ascii' codec can't decode byte 0xc3 in position 14: ordinal not in range(128)

[WebKit Commit Bot]

Comment on attachment 53215 [details]
Patch, minor.

Rejecting patch 53215 from commit-queue.

Unexpected failure when landing patch!  Please file a bug against webkit-patch.
Failed to run "['WebKitTools/Scripts/webkit-patch', '--status-host=webkit-commit-queue.appspot.com', 'land-attachment', '--force-clean', '--build', '--test', '--non-interactive', '--ignore-builders', '--build-style=both', '--quiet', '53215', '--parent-command=commit-queue', '--no-update']" exit_code: 1
Last 500 characters of output:
mon/config/committers.py:275: UnicodeWarning: Unicode equal comparison failed to convert both arguments to Unicode - interpreting them as being unequal
  if committer.full_name == name:
Tor Arne Vestbø found in /Users/eseidel/Projects/CommitQueue/WebCore/ChangeLog does not appear to be a valid reviewer according to committers.py.
ERROR: /Users/eseidel/Projects/CommitQueue/WebCore/ChangeLog neither lists a valid reviewer nor contains the string "Unreviewed" or "Rubber stamp" (case insensitive).

[Yael]

Committed in r57527.

[Eric Seidel (no email)]

I have a fix, will post.

[Eric Seidel (no email)]

Fixed in bug 37511.  Sorry for the trouble.