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

Side by Side Diff: content/browser/renderer_host/render_widget_host_view_base.cc

Issue 1948343002: [reland] Browser Side Text Input State Tracking for OOPIF (Aura Only) (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Addressing kenrb@ comments (Removing TextInputManager::Observer::OnTextInputManagerDestroyed) Created 4 years, 7 months 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 (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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/browser/renderer_host/render_widget_host_view_base.h" 5 #include "content/browser/renderer_host/render_widget_host_view_base.h"
6 6
7 #include "base/logging.h" 7 #include "base/logging.h"
8 #include "build/build_config.h" 8 #include "build/build_config.h"
9 #include "content/browser/accessibility/browser_accessibility_manager.h" 9 #include "content/browser/accessibility/browser_accessibility_manager.h"
10 #include "content/browser/gpu/gpu_data_manager_impl.h" 10 #include "content/browser/gpu/gpu_data_manager_impl.h"
11 #include "content/browser/renderer_host/input/synthetic_gesture_target_base.h" 11 #include "content/browser/renderer_host/input/synthetic_gesture_target_base.h"
12 #include "content/browser/renderer_host/render_process_host_impl.h" 12 #include "content/browser/renderer_host/render_process_host_impl.h"
13 #include "content/browser/renderer_host/render_widget_host_delegate.h" 13 #include "content/browser/renderer_host/render_widget_host_delegate.h"
14 #include "content/browser/renderer_host/render_widget_host_impl.h" 14 #include "content/browser/renderer_host/render_widget_host_impl.h"
15 #include "content/browser/renderer_host/render_widget_host_view_base_observer.h" 15 #include "content/browser/renderer_host/render_widget_host_view_base_observer.h"
16 #include "content/browser/renderer_host/text_input_manager.h"
16 #include "content/common/content_switches_internal.h" 17 #include "content/common/content_switches_internal.h"
17 #include "content/public/browser/render_widget_host_view_frame_subscriber.h" 18 #include "content/public/browser/render_widget_host_view_frame_subscriber.h"
18 #include "ui/display/display.h" 19 #include "ui/display/display.h"
19 #include "ui/display/screen.h" 20 #include "ui/display/screen.h"
20 #include "ui/gfx/geometry/point_conversions.h" 21 #include "ui/gfx/geometry/point_conversions.h"
21 #include "ui/gfx/geometry/size_conversions.h" 22 #include "ui/gfx/geometry/size_conversions.h"
22 #include "ui/gfx/geometry/size_f.h" 23 #include "ui/gfx/geometry/size_f.h"
23 24
24 namespace content { 25 namespace content {
25 26
(...skipping 349 matching lines...) Expand 10 before | Expand all | Expand 10 after
375 gfx::ToRoundedPoint(point))); 376 gfx::ToRoundedPoint(point)));
376 } 377 }
377 378
378 void RenderWidgetHostViewBase::TransformPointToLocalCoordSpace( 379 void RenderWidgetHostViewBase::TransformPointToLocalCoordSpace(
379 const gfx::Point& point, 380 const gfx::Point& point,
380 cc::SurfaceId original_surface, 381 cc::SurfaceId original_surface,
381 gfx::Point* transformed_point) { 382 gfx::Point* transformed_point) {
382 *transformed_point = point; 383 *transformed_point = point;
383 } 384 }
384 385
386 void RenderWidgetHostViewBase::TextInputStateChanged(
387 const TextInputState& text_input_state) {
388 if (GetTextInputManager())
389 GetTextInputManager()->UpdateTextInputState(this, text_input_state);
390 }
391
392 TextInputManager* RenderWidgetHostViewBase::GetTextInputManager() const {
393 RenderWidgetHostImpl* host =
394 RenderWidgetHostImpl::From(GetRenderWidgetHost());
395
396 if (!host || !host->delegate())
397 return nullptr;
398
399 TextInputManager* manager = host->delegate()->GetTextInputManager();
400
401 // If this is the first time getting a reference to this TextInputManager, we
402 // should get it to register this view.
403 if (!manager->IsRegisteredView(this))
404 manager->Register(const_cast<RenderWidgetHostViewBase*>(this));
EhsanK 2016/05/06 18:23:43 I am not sure how OK this is, but the reason for t
405
406 return manager;
407 }
408
385 void RenderWidgetHostViewBase::AddObserver( 409 void RenderWidgetHostViewBase::AddObserver(
386 RenderWidgetHostViewBaseObserver* observer) { 410 RenderWidgetHostViewBaseObserver* observer) {
387 observers_.AddObserver(observer); 411 observers_.AddObserver(observer);
388 } 412 }
389 413
390 void RenderWidgetHostViewBase::RemoveObserver( 414 void RenderWidgetHostViewBase::RemoveObserver(
391 RenderWidgetHostViewBaseObserver* observer) { 415 RenderWidgetHostViewBaseObserver* observer) {
392 observers_.RemoveObserver(observer); 416 observers_.RemoveObserver(observer);
393 } 417 }
394 418
395 bool RenderWidgetHostViewBase::IsChildFrameForTesting() const { 419 bool RenderWidgetHostViewBase::IsChildFrameForTesting() const {
396 return false; 420 return false;
397 } 421 }
398 422
399 cc::SurfaceId RenderWidgetHostViewBase::SurfaceIdForTesting() const { 423 cc::SurfaceId RenderWidgetHostViewBase::SurfaceIdForTesting() const {
400 return cc::SurfaceId(); 424 return cc::SurfaceId();
401 } 425 }
402 426
403 } // namespace content 427 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698