|Summary:||Simplify speech input plumbing in webcore and webkit|
|Product:||WebKit||Reporter:||Satish Sampath <satish>|
|Version:||528+ (Nightly build)|
|Bug Depends on:|
Description Satish Sampath 2010-07-26 15:01:15 PDT
A few related changes to make things simpler with the speech input plumbing between WebCore <> WebKit. This will help in subsequent patches for adding testing/mocking code and the mac/linux port implementation. - Each WebCore::InputFieldSpeechButtonElement object was creating one WebCore::SpeechInput object for accessing the WebKit speech client, and there were 2 listener interfaces involved. Instead, now WebCore::Page creates only 1 instance of WebCore::SpeechInput and WebCore::InputFieldSpeechButtonElement calls the webkit APIs through this object. - WebKit::SpeechInputClientImpl was having some intelligence to deal with starting speech recognition when another instance was already running. This has now been moved to the above mentioned WebCore::SpeechInput object which WebCore::Page maintains. With this change, WebKit::SpeechInputClientImpl becomes a dumb proxy between WebCore and the WebKit. - Added necessary methods to WebCore::SpeechInput and WebCore::SpeechInputClient so that the above mentioned functionality can be moved from WebKit::SpeechInputClientImpl to WebCore::SpeechInput.
Comment 2 Kent Tamura 2010-07-26 22:25:35 PDT
Comment on attachment 62618 [details] Patch The code change looks OK. But please re-make the patch with the latest WebKit. Our commit-queue has a bug for ChangeLog diff not starting with the first line like your patch. > diff --git a/WebCore/ChangeLog b/WebCore/ChangeLog > index e26d1854a9a330a50728633b8e55d07feb945f6d..7226ef6e566e8a6849f4f2cc3ccd614f118f3963 100644 > --- a/WebCore/ChangeLog > +++ b/WebCore/ChangeLog > @@ -1,5 +1,39 @@ > 2010-07-26 Satish Sampath <firstname.lastname@example.org> > > + Reviewed by NOBODY (OOPS!). > + > + Simplify speech input plumbing in webcore and webkit > + https://bugs.webkit.org/show_bug.cgi?id=43008 > diff --git a/WebKit/chromium/ChangeLog b/WebKit/chromium/ChangeLog > index 0309952965ca68664268ba361e5c63f80f6c7612..3dddb2d6bc562091b15d992050558259a1c7d6dd 100644 > --- a/WebKit/chromium/ChangeLog > +++ b/WebKit/chromium/ChangeLog > @@ -1,5 +1,20 @@ > 2010-07-26 Satish Sampath <email@example.com> > > + Reviewed by NOBODY (OOPS!). > + > + Simplify speech input plumbing in webcore and webkit > + https://bugs.webkit.org/show_bug.cgi?id=43008
Comment 3 Satish Sampath 2010-07-26 22:55:59 PDT
Created attachment 62646 [details] Patch Merged with latest.
Comment 5 WebKit Commit Bot 2010-07-27 00:51:09 PDT
Comment on attachment 62646 [details] Patch Clearing flags on attachment: 62646 Committed r64111: <http://trac.webkit.org/changeset/64111>
Comment 6 WebKit Commit Bot 2010-07-27 00:51:14 PDT
All reviewed patches have been landed. Closing bug.