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

Side by Side Diff: content/renderer/render_frame_impl.cc

Issue 636863003: Make SpeechRecognition per RenderFrame instead of per RenderView. Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: review comments Created 6 years 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
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "content/renderer/render_frame_impl.h" 5 #include "content/renderer/render_frame_impl.h"
6 6
7 #include <map> 7 #include <map>
8 #include <string> 8 #include <string>
9 9
10 #include "base/auto_reset.h" 10 #include "base/auto_reset.h"
(...skipping 73 matching lines...) Expand 10 before | Expand all | Expand 10 after
84 #include "content/renderer/push_messaging_dispatcher.h" 84 #include "content/renderer/push_messaging_dispatcher.h"
85 #include "content/renderer/render_frame_proxy.h" 85 #include "content/renderer/render_frame_proxy.h"
86 #include "content/renderer/render_process.h" 86 #include "content/renderer/render_process.h"
87 #include "content/renderer/render_thread_impl.h" 87 #include "content/renderer/render_thread_impl.h"
88 #include "content/renderer/render_view_impl.h" 88 #include "content/renderer/render_view_impl.h"
89 #include "content/renderer/render_widget_fullscreen_pepper.h" 89 #include "content/renderer/render_widget_fullscreen_pepper.h"
90 #include "content/renderer/renderer_webapplicationcachehost_impl.h" 90 #include "content/renderer/renderer_webapplicationcachehost_impl.h"
91 #include "content/renderer/renderer_webcolorchooser_impl.h" 91 #include "content/renderer/renderer_webcolorchooser_impl.h"
92 #include "content/renderer/screen_orientation/screen_orientation_dispatcher.h" 92 #include "content/renderer/screen_orientation/screen_orientation_dispatcher.h"
93 #include "content/renderer/shared_worker_repository.h" 93 #include "content/renderer/shared_worker_repository.h"
94 #include "content/renderer/speech_recognition_dispatcher.h"
94 #include "content/renderer/v8_value_converter_impl.h" 95 #include "content/renderer/v8_value_converter_impl.h"
95 #include "content/renderer/websharedworker_proxy.h" 96 #include "content/renderer/websharedworker_proxy.h"
96 #include "gin/modules/module_registry.h" 97 #include "gin/modules/module_registry.h"
97 #include "media/base/audio_renderer_mixer_input.h" 98 #include "media/base/audio_renderer_mixer_input.h"
98 #include "media/base/renderer.h" 99 #include "media/base/renderer.h"
99 #include "media/blink/encrypted_media_player_support.h" 100 #include "media/blink/encrypted_media_player_support.h"
100 #include "media/blink/webcontentdecryptionmodule_impl.h" 101 #include "media/blink/webcontentdecryptionmodule_impl.h"
101 #include "media/blink/webmediaplayer_impl.h" 102 #include "media/blink/webmediaplayer_impl.h"
102 #include "media/blink/webmediaplayer_params.h" 103 #include "media/blink/webmediaplayer_params.h"
103 #include "media/filters/gpu_video_accelerator_factories.h" 104 #include "media/filters/gpu_video_accelerator_factories.h"
(...skipping 475 matching lines...) Expand 10 before | Expand all | Expand 10 after
579 #if defined(ENABLE_BROWSER_CDMS) 580 #if defined(ENABLE_BROWSER_CDMS)
580 cdm_manager_(NULL), 581 cdm_manager_(NULL),
581 #endif 582 #endif
582 #if defined(VIDEO_HOLE) 583 #if defined(VIDEO_HOLE)
583 contains_media_player_(false), 584 contains_media_player_(false),
584 #endif 585 #endif
585 geolocation_dispatcher_(NULL), 586 geolocation_dispatcher_(NULL),
586 push_messaging_dispatcher_(NULL), 587 push_messaging_dispatcher_(NULL),
587 screen_orientation_dispatcher_(NULL), 588 screen_orientation_dispatcher_(NULL),
588 manifest_manager_(NULL), 589 manifest_manager_(NULL),
590 speech_recognition_dispatcher_(NULL),
589 accessibility_mode_(AccessibilityModeOff), 591 accessibility_mode_(AccessibilityModeOff),
590 renderer_accessibility_(NULL), 592 renderer_accessibility_(NULL),
591 weak_factory_(this) { 593 weak_factory_(this) {
592 std::pair<RoutingIDFrameMap::iterator, bool> result = 594 std::pair<RoutingIDFrameMap::iterator, bool> result =
593 g_routing_id_frame_map.Get().insert(std::make_pair(routing_id_, this)); 595 g_routing_id_frame_map.Get().insert(std::make_pair(routing_id_, this));
594 CHECK(result.second) << "Inserting a duplicate item."; 596 CHECK(result.second) << "Inserting a duplicate item.";
595 597
596 RenderThread::Get()->AddRoute(routing_id_, this); 598 RenderThread::Get()->AddRoute(routing_id_, this);
597 599
598 render_view_->RegisterRenderFrame(this); 600 render_view_->RegisterRenderFrame(this);
(...skipping 2808 matching lines...) Expand 10 before | Expand all | Expand 10 after
3407 } 3409 }
3408 } 3410 }
3409 3411
3410 void RenderFrameImpl::didChangeManifest(blink::WebLocalFrame* frame) 3412 void RenderFrameImpl::didChangeManifest(blink::WebLocalFrame* frame)
3411 { 3413 {
3412 DCHECK(!frame_ || frame_ == frame); 3414 DCHECK(!frame_ || frame_ == frame);
3413 3415
3414 FOR_EACH_OBSERVER(RenderFrameObserver, observers_, DidChangeManifest()); 3416 FOR_EACH_OBSERVER(RenderFrameObserver, observers_, DidChangeManifest());
3415 } 3417 }
3416 3418
3419 blink::WebSpeechRecognizer* RenderFrameImpl::speechRecognizer() {
3420 if (!speech_recognition_dispatcher_)
3421 speech_recognition_dispatcher_ = new SpeechRecognitionDispatcher(this);
3422 return speech_recognition_dispatcher_;
3423 }
3424
3417 void RenderFrameImpl::DidPlay(blink::WebMediaPlayer* player) { 3425 void RenderFrameImpl::DidPlay(blink::WebMediaPlayer* player) {
3418 Send(new FrameHostMsg_MediaPlayingNotification( 3426 Send(new FrameHostMsg_MediaPlayingNotification(
3419 routing_id_, reinterpret_cast<int64>(player), player->hasVideo(), 3427 routing_id_, reinterpret_cast<int64>(player), player->hasVideo(),
3420 player->hasAudio(), player->isRemote())); 3428 player->hasAudio(), player->isRemote()));
3421 } 3429 }
3422 3430
3423 void RenderFrameImpl::DidPause(blink::WebMediaPlayer* player) { 3431 void RenderFrameImpl::DidPause(blink::WebMediaPlayer* player) {
3424 Send(new FrameHostMsg_MediaPausedNotification( 3432 Send(new FrameHostMsg_MediaPausedNotification(
3425 routing_id_, reinterpret_cast<int64>(player))); 3433 routing_id_, reinterpret_cast<int64>(player)));
3426 } 3434 }
(...skipping 47 matching lines...) Expand 10 before | Expand all | Expand 10 after
3474 InternalDocumentStateData::FromDocumentState(document_state); 3482 InternalDocumentStateData::FromDocumentState(document_state);
3475 3483
3476 FrameHostMsg_DidCommitProvisionalLoad_Params params; 3484 FrameHostMsg_DidCommitProvisionalLoad_Params params;
3477 params.http_status_code = response.httpStatusCode(); 3485 params.http_status_code = response.httpStatusCode();
3478 params.url_is_unreachable = ds->hasUnreachableURL(); 3486 params.url_is_unreachable = ds->hasUnreachableURL();
3479 params.is_post = false; 3487 params.is_post = false;
3480 params.post_id = -1; 3488 params.post_id = -1;
3481 params.page_id = render_view_->page_id_; 3489 params.page_id = render_view_->page_id_;
3482 // We need to track the RenderViewHost routing_id because of downstream 3490 // We need to track the RenderViewHost routing_id because of downstream
3483 // dependencies (crbug.com/392171 DownloadRequestHandle, SaveFileManager, 3491 // dependencies (crbug.com/392171 DownloadRequestHandle, SaveFileManager,
3484 // ResourceDispatcherHostImpl, MediaStreamUIProxy, 3492 // ResourceDispatcherHostImpl, MediaStreamUIProxy, and possibly others). They
3485 // SpeechRecognitionDispatcherHost and possibly others). They look up the view 3493 // look up the view based on the ID stored in the resource requests. Once
3486 // based on the ID stored in the resource requests. Once those dependencies 3494 // those dependencies are unwound or moved to RenderFrameHost
3487 // are unwound or moved to RenderFrameHost (crbug.com/304341) we can move the 3495 // (crbug.com/304341) we can move the client to be based on the routing_id of
3488 // client to be based on the routing_id of the RenderFrameHost. 3496 // the RenderFrameHost.
3489 params.render_view_routing_id = render_view_->routing_id(); 3497 params.render_view_routing_id = render_view_->routing_id();
3490 params.socket_address.set_host(response.remoteIPAddress().utf8()); 3498 params.socket_address.set_host(response.remoteIPAddress().utf8());
3491 params.socket_address.set_port(response.remotePort()); 3499 params.socket_address.set_port(response.remotePort());
3492 WebURLResponseExtraDataImpl* extra_data = GetExtraDataFromResponse(response); 3500 WebURLResponseExtraDataImpl* extra_data = GetExtraDataFromResponse(response);
3493 if (extra_data) 3501 if (extra_data)
3494 params.was_fetched_via_proxy = extra_data->was_fetched_via_proxy(); 3502 params.was_fetched_via_proxy = extra_data->was_fetched_via_proxy();
3495 params.was_within_same_page = navigation_state->was_within_same_page(); 3503 params.was_within_same_page = navigation_state->was_within_same_page();
3496 params.security_info = response.securityInfo(); 3504 params.security_info = response.securityInfo();
3497 3505
3498 // Set the URL to be displayed in the browser UI to the user. 3506 // Set the URL to be displayed in the browser UI to the user.
(...skipping 767 matching lines...) Expand 10 before | Expand all | Expand 10 after
4266 4274
4267 #if defined(ENABLE_BROWSER_CDMS) 4275 #if defined(ENABLE_BROWSER_CDMS)
4268 RendererCdmManager* RenderFrameImpl::GetCdmManager() { 4276 RendererCdmManager* RenderFrameImpl::GetCdmManager() {
4269 if (!cdm_manager_) 4277 if (!cdm_manager_)
4270 cdm_manager_ = new RendererCdmManager(this); 4278 cdm_manager_ = new RendererCdmManager(this);
4271 return cdm_manager_; 4279 return cdm_manager_;
4272 } 4280 }
4273 #endif // defined(ENABLE_BROWSER_CDMS) 4281 #endif // defined(ENABLE_BROWSER_CDMS)
4274 4282
4275 } // namespace content 4283 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698