Bug 64857 - fast/speech/input-text-speechbutton.html is timing out after r91249
Summary: fast/speech/input-text-speechbutton.html is timing out after r91249
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit Misc. (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Jeremy Moskovich
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-07-20 03:33 PDT by Jeremy Moskovich
Modified: 2011-07-21 03:18 PDT (History)
5 users (show)

See Also:


Attachments
Patch (2.72 KB, patch)
2011-07-20 03:40 PDT, Jeremy Moskovich
no flags Details | Formatted Diff | Diff
Patch for landing (3.10 KB, patch)
2011-07-21 02:19 PDT, Jeremy Moskovich
no flags Details | Formatted Diff | Diff
Patch for landing (3.10 KB, patch)
2011-07-21 02:40 PDT, Jeremy Moskovich
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Jeremy Moskovich 2011-07-20 03:33:45 PDT
Patch is in the works.
Comment 1 Jeremy Moskovich 2011-07-20 03:34:35 PDT
Chromium bug: crbug.com/89717
Comment 2 Jeremy Moskovich 2011-07-20 03:40:08 PDT
Created attachment 101447 [details]
Patch
Comment 3 Satish Sampath 2011-07-20 03:42:41 PDT
Comment on attachment 101447 [details]
Patch

Looks good.
Comment 4 Kent Tamura 2011-07-20 03:43:28 PDT
Comment on attachment 101447 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=101447&action=review

ok!

> LayoutTests/fast/speech/input-text-speechbutton.html:16
> +    var isRTL = input.dir == 'rtl';

This doesn't work for CSS direction:rtl, but it's ok for testing.
Comment 5 WebKit Review Bot 2011-07-20 03:43:51 PDT
Comment on attachment 101447 [details]
Patch

Rejecting attachment 101447 [details] from review queue.

satish@chromium.org does not have reviewer permissions according to http://trac.webkit.org/browser/trunk/Tools/Scripts/webkitpy/common/config/committers.py.

- If you do not have reviewer rights please read http://webkit.org/coding/contributing.html for instructions on how to use bugzilla flags.

- If you have reviewer rights please correct the error in Tools/Scripts/webkitpy/common/config/committers.py by adding yourself to the file (no review needed).  The commit-queue restarts itself every 2 hours.  After restart the commit-queue will correctly respect your reviewer rights.
Comment 6 WebKit Review Bot 2011-07-20 04:03:35 PDT
Comment on attachment 101447 [details]
Patch

Attachment 101447 [details] did not pass chromium-ews (chromium-xvfb):
Output: http://queues.webkit.org/results/9191221

New failing tests:
fast/speech/input-text-speechbutton.html
Comment 7 Ryosuke Niwa 2011-07-20 08:54:09 PDT
(In reply to comment #6)
> (From update of attachment 101447 [details])
> Attachment 101447 [details] did not pass chromium-ews (chromium-xvfb):
> Output: http://queues.webkit.org/results/9191221
> 
> New failing tests:
> fast/speech/input-text-speechbutton.html

Odd. http://queues.webkit.org/results/9191221 doesn't says nothing of the kind :(
Comment 8 WebKit Review Bot 2011-07-20 12:53:32 PDT
Comment on attachment 101447 [details]
Patch

Rejecting attachment 101447 [details] from commit-queue.

Failed to run "['./Tools/Scripts/webkit-patch', '--status-host=queues.webkit.org', '--bot-id=ec2-cq-01', '--port..." exit_code: 2

Last 500 characters of output:
 fast/forms/implicit-submission.html = TEXT PASS


Regressions: Unexpected text diff mismatch : (1)
  fast/speech/input-text-speechbutton.html = TEXT

Regressions: Unexpected image mismatch : (5)
  fast/text/atsui-multiple-renderers.html = IMAGE
  fast/text/international/danda-space.html = IMAGE
  fast/text/international/thai-baht-space.html = IMAGE
  fast/text/international/thai-line-breaks.html = IMAGE
  platform/chromium-linux/fast/text/international/complex-joining-using-gpos.html = IMAGE



Full output: http://queues.webkit.org/results/9200374
Comment 9 Jeremy Moskovich 2011-07-21 02:19:12 PDT
Created attachment 101563 [details]
Patch for landing
Comment 10 WebKit Review Bot 2011-07-21 02:38:27 PDT
Comment on attachment 101563 [details]
Patch for landing

Rejecting attachment 101563 [details] from commit-queue.

Failed to run "['./Tools/Scripts/webkit-patch', '--status-host=queues.webkit.org', '--bot-id=ec2-cq-02', '--port..." exit_code: 1

Last 500 characters of output:
s/ChangeLog does not appear to be a valid reviewer according to committers.py.
ERROR: /mnt/git/webkit-commit-queue/LayoutTests/ChangeLog neither lists a valid reviewer nor contains the string "Unreviewed" or "Rubber stamp" (case insensitive).
Updating OpenSource
Current branch master is up to date.
Updating chromium port dependencies using gclient...

________ running '/usr/bin/python gyp_webkit' in '/mnt/git/webkit-commit-queue/Source/WebKit/chromium'
Updating webkit projects from gyp files...

Full output: http://queues.webkit.org/results/9192581
Comment 11 Jeremy Moskovich 2011-07-21 02:40:08 PDT
Created attachment 101565 [details]
Patch for landing
Comment 12 WebKit Review Bot 2011-07-21 03:18:44 PDT
Comment on attachment 101565 [details]
Patch for landing

Clearing flags on attachment: 101565

Committed r91458: <http://trac.webkit.org/changeset/91458>
Comment 13 WebKit Review Bot 2011-07-21 03:18:48 PDT
All reviewed patches have been landed.  Closing bug.