chrome/common/extensions/docs/examples/api/speechInput/basic/background.html - Issue 8536022: Adding static documentation to the speech input extension API.

Side by Side Diff

Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.

Keyboard Shortcuts

	File
u :	up to issue
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line

	Issue
u :	up to list of issues
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue

Side by Side Diff: chrome/common/extensions/docs/examples/api/speechInput/basic/background.html

Issue 8536022: Adding static documentation to the speech input extension API. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src

Patch Set: fixing nits and adding a example. Created 9 years, 1 month ago

Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.

Jump to:

View unified diff | Download patch | Annotate | Revision Log

« no previous file with comments | « chrome/common/extensions/docs/examples/api/speechInput/basic.zip ('k') | chrome/common/extensions/docs/examples/api/speechInput/basic/icon.png » ('j') | chrome/common/extensions/docs/examples/api/speechInput/basic/manifest.json » ('J')
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

OLD	NEW
(Empty)
	1 <html>

	2 <script>

	3 function checkStart() {
	Satish 2011/11/17 16:29:23 since this is a small enough function and used onl since this is a small enough function and used only once, may be define it as a closure in the .start() call for simplicity? Leandro Graciá Gil 2011/11/17 22:02:45 Done. Show quoted text On 2011/11/17 16:29:23, Satish wrote: > since this is a small enough function and used only once, may be define it as a > closure in the .start() call for simplicity? Done.
	4 if (chrome.extension.lastError) {

	5 alert("Couldn't start speech input: " + chrome.extension.lastError.message);

	6 }

	7 }

	8

	9 chrome.browserAction.onClicked.addListener(function(tab) {

	10 chrome.experimental.speechInput.isRecording(function (recording) {

	11 if (!recording)
	Satish 2011/11/17 16:29:23 use braces here? use braces here? Leandro Graciá Gil 2011/11/17 22:02:45 Done. Show quoted text On 2011/11/17 16:29:23, Satish wrote: > use braces here? Done.
	12 chrome.experimental.speechInput.start({}, checkStart);
	Satish 2011/11/17 16:29:23 it may be useful if we change the browser action i it may be useful if we change the browser action icon to a 'recording' state after this call and switch back to normal state after calling .stop below Satish 2011/11/17 16:29:23 is that empty dictionary {} required or can a null is that empty dictionary {} required or can a null be passed? Leandro Graciá Gil 2011/11/17 22:02:45 Done. Show quoted text On 2011/11/17 16:29:23, Satish wrote: > it may be useful if we change the browser action icon to a 'recording' state > after this call and switch back to normal state after calling .stop below Done. Leandro Graciá Gil 2011/11/17 22:02:45 It is required. Show quoted text On 2011/11/17 16:29:23, Satish wrote: > is that empty dictionary {} required or can a null be passed? It is required.
	13 else

	14 chrome.experimental.speechInput.stop();

	15 });

	16 });

	17

	18 chrome.experimental.speechInput.onError.addListener(function(error) {

	19 alert("Speech input failed: " + error.code);

	20 });

	21

	22 chrome.experimental.speechInput.onResult.addListener(function(result) {

	23 alert(result.hypotheses[0].utterance);

	24 });

	25 </script>

	26 </html>

OLD	NEW