Bug 37481

Summary: Minor fix to pass RenderProgress instead of RenderObject
Product: WebKit Reporter: Yael <yael>
Component: AccessibilityAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue, eric
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: PC   
OS: OS X 10.5   
Attachments:
Description Flags
Patch, minor. vestbo: review+, commit-queue: commit-queue-

Description Yael 2010-04-12 20:15:02 PDT
Per https://bugs.webkit.org/show_bug.cgi?id=37275#c4, should pass RenderProgress* instead of RenderObject* to AccessibilityProgressIndicator::create().
Comment 1 Yael 2010-04-12 20:19:58 PDT
Created attachment 53215 [details]
Patch, minor.
Comment 2 WebKit Commit Bot 2010-04-12 21:43:19 PDT
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).
Comment 3 Eric Seidel (no email) 2010-04-13 08:44:56 PDT
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.
Comment 4 Eric Seidel (no email) 2010-04-13 09:40:52 PDT
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)
Comment 5 WebKit Commit Bot 2010-04-13 11:07:10 PDT
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).
Comment 6 Yael 2010-04-13 11:18:44 PDT
Committed in r57527.
Comment 7 Eric Seidel (no email) 2010-04-13 11:22:54 PDT
I have a fix, will post.
Comment 8 Eric Seidel (no email) 2010-04-13 11:33:35 PDT
Fixed in bug 37511.  Sorry for the trouble.