Bug 40878 - Speech Input Patch 0: Added compilation argument to conditionally compile pending patches.
Summary: Speech Input Patch 0: Added compilation argument to conditionally compile pen...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Forms (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks: 39485
  Show dependency treegraph
 
Reported: 2010-06-19 04:25 PDT by Satish Sampath
Modified: 2010-06-21 09:02 PDT (History)
7 users (show)

See Also:


Attachments
Patch (22.25 KB, patch)
2010-06-19 05:23 PDT, Satish Sampath
no flags Details | Formatted Diff | Diff
Patch (25.32 KB, patch)
2010-06-19 11:05 PDT, Satish Sampath
no flags Details | Formatted Diff | Diff
Patch (25.37 KB, patch)
2010-06-21 08:39 PDT, Satish Sampath
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Satish Sampath 2010-06-19 04:25:01 PDT
This is the first in a series of patches to enable speech in HTML input elements. I used http://trac.webkit.org/changeset/60543 as template for adding new conditionally compiled functionality and tested on my local machine to work.

More information about the speech input API proposal can be found in the following links:
 Parent bug: https://bugs.webkit.org/show_bug.cgi?id=39485
 Full API proposal: https://docs.google.com/View?id=dcfg79pz_5dhnp23f5
 Relevant discussions in the WHATWG list:
  - http://lists.whatwg.org/htdig.cgi/whatwg-whatwg.org/2010-May/026338.html
  - http://lists.whatwg.org/pipermail/whatwg-whatwg.org/2010-June/026747.html
Comment 1 Satish Sampath 2010-06-19 05:23:54 PDT
Created attachment 59185 [details]
Patch
Comment 2 Kent Tamura 2010-06-19 05:49:31 PDT
Comment on attachment 59185 [details]
Patch

* You need to update WebKit2/Configurations/FeatureDefine.xcconfig.

WebKit/chromium/features.gypi:56
 +          'ENABLE_INPUT_SPEECH=0',
Please enable by default in order that buildbots can check compilation.
Enabling INPUT_SPEECH here doesn't mean Chromium/Chrome browser have INPUT_SPEECH.
Comment 3 Satish Sampath 2010-06-19 11:05:01 PDT
Created attachment 59189 [details]
Patch

Added to Webkit2 config & enabled flag in chromium port
Comment 4 Kent Tamura 2010-06-20 21:27:26 PDT
Comment on attachment 59189 [details]
Patch

LGTM.  Other reviewers, any comments?
Comment 5 Jeremy Orlow 2010-06-21 07:31:46 PDT
Comment on attachment 59189 [details]
Patch

Seems fine to me.  r+ing
Comment 6 WebKit Commit Bot 2010-06-21 08:28:28 PDT
Comment on attachment 59189 [details]
Patch

Rejecting patch 59189 from commit-queue.

Failed to run "[u'git', u'svn', u'dcommit']" exit_code: 1
Last 500 characters of output:
es/win/tools/vsprops/FeatureDefinesCairo.vsprops
	M	WebKitTools/ChangeLog
	M	WebKitTools/Scripts/build-webkit
	M	configure.ac
A repository hook failed: MERGE request failed on '/repository/webkit/trunk': Commit blocked by pre-commit hook (exit code 1) with output:
svnlook: Can't write to stream: Broken pipe

    The following ChangeLog files contain OOPS:

        trunk/WebCore/ChangeLog

    Please don't ever say "OOPS" in a ChangeLog file.
 at /usr/local/git/libexec/git-core/git-svn line 570


Full output: http://webkit-commit-queue.appspot.com/results/3284500
Comment 7 Satish Sampath 2010-06-21 08:39:12 PDT
Created attachment 59254 [details]
Patch

Removed duplicate OOPS in changelog
Comment 8 Steve Block 2010-06-21 08:44:13 PDT
Comment on attachment 59254 [details]
Patch

r=me
Comment 9 WebKit Commit Bot 2010-06-21 09:02:22 PDT
Comment on attachment 59254 [details]
Patch

Clearing flags on attachment: 59254

Committed r61545: <http://trac.webkit.org/changeset/61545>
Comment 10 WebKit Commit Bot 2010-06-21 09:02:27 PDT
All reviewed patches have been landed.  Closing bug.