Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(96)

Issue 899853002: Merge 188868 "Revert of Make SpeechRecognitionController per fra..." (Closed)

Created:
5 years, 10 months ago by mlamouri (slow - plz ping)
Modified:
5 years, 10 months ago
CC:
blink-reviews, dglazkov+blink
Project:
blink
Visibility:
Public.

Description

Merge 188868 "Revert of Make SpeechRecognitionController per fra..." > Revert of Make SpeechRecognitionController per frame instead of per page. (patchset #3 id:40001 of https://codereview.chromium.org/636863002/) > > Reason for revert: > https://crbug.com/449739 > > Original issue's description: > > Make SpeechRecognitionController per frame instead of per page. > > > > This is still using the WebViewClient::speechRecognizer() if > > WebFrameClient::speechRecognizer() returns null. After part 2 > > lands, part 3 will remove it. > > > > Part 1: <this> > > Part 2: https://codereview.chromium.org/636863003/ > > Part 3: https://codereview.chromium.org/752303003/ > > > > BUG=390749 > > > > Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=186174 > > TBR=dcheng@chromium.org,mkwst@chromium.org > NOPRESUBMIT=true > NOTREECHECKS=true > NOTRY=true > BUG=390749, 449739 > > Review URL: https://codereview.chromium.org/863213002 TBR=mlamouri@chromium.org Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=189483

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+16 lines, -37 lines) Patch
M Source/modules/speech/SpeechRecognition.cpp View 1 chunk +3 lines, -3 lines 0 comments Download
M Source/modules/speech/SpeechRecognitionController.h View 2 chunks +4 lines, -6 lines 0 comments Download
M Source/modules/speech/SpeechRecognitionController.cpp View 2 chunks +7 lines, -13 lines 0 comments Download
M Source/web/WebLocalFrameImpl.cpp View 4 chunks +0 lines, -8 lines 0 comments Download
M Source/web/WebViewImpl.cpp View 2 chunks +2 lines, -0 lines 0 comments Download
M public/web/WebFrameClient.h View 2 chunks +0 lines, -7 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
mlamouri (slow - plz ping)
5 years, 10 months ago (2015-02-04 11:14:32 UTC) #1
mlamouri (slow - plz ping)
5 years, 10 months ago (2015-02-04 11:14:50 UTC) #2
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as r189483.

Powered by Google App Engine
This is Rietveld 408576698