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

Side by Side Diff: ui/views/controls/webview/webview.cc

Issue 880063002: Ensure WebView notifies desktop automation on creation, destruction, and change (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 10 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
« no previous file with comments | « chrome/renderer/resources/extensions/automation/automation_node.js ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 "ui/views/controls/webview/webview.h" 5 #include "ui/views/controls/webview/webview.h"
6 6
7 #include "content/public/browser/browser_accessibility_state.h" 7 #include "content/public/browser/browser_accessibility_state.h"
8 #include "content/public/browser/browser_context.h" 8 #include "content/public/browser/browser_context.h"
9 #include "content/public/browser/navigation_controller.h" 9 #include "content/public/browser/navigation_controller.h"
10 #include "content/public/browser/render_view_host.h" 10 #include "content/public/browser/render_view_host.h"
(...skipping 262 matching lines...) Expand 10 before | Expand all | Expand 10 after
273 void WebView::RenderProcessGone(base::TerminationStatus status) { 273 void WebView::RenderProcessGone(base::TerminationStatus status) {
274 NotifyMaybeTextInputClientChanged(); 274 NotifyMaybeTextInputClientChanged();
275 } 275 }
276 276
277 void WebView::RenderViewHostChanged(content::RenderViewHost* old_host, 277 void WebView::RenderViewHostChanged(content::RenderViewHost* old_host,
278 content::RenderViewHost* new_host) { 278 content::RenderViewHost* new_host) {
279 FocusManager* const focus_manager = GetFocusManager(); 279 FocusManager* const focus_manager = GetFocusManager();
280 if (focus_manager && focus_manager->GetFocusedView() == this) 280 if (focus_manager && focus_manager->GetFocusedView() == this)
281 OnFocus(); 281 OnFocus();
282 NotifyMaybeTextInputClientChanged(); 282 NotifyMaybeTextInputClientChanged();
283 NotifyAccessibilityEvent(ui::AX_EVENT_LAYOUT_COMPLETE, true);
dmazzoni 2015/01/27 21:53:45 What do you think about AX_EVENT_CHILDREN_CHANGED?
David Tseng 2015/01/27 22:04:50 Done.
283 } 284 }
284 285
285 void WebView::DidShowFullscreenWidget(int routing_id) { 286 void WebView::DidShowFullscreenWidget(int routing_id) {
286 if (embed_fullscreen_widget_mode_enabled_) 287 if (embed_fullscreen_widget_mode_enabled_)
287 ReattachForFullscreenChange(true); 288 ReattachForFullscreenChange(true);
288 } 289 }
289 290
290 void WebView::DidDestroyFullscreenWidget(int routing_id) { 291 void WebView::DidDestroyFullscreenWidget(int routing_id) {
291 if (embed_fullscreen_widget_mode_enabled_) 292 if (embed_fullscreen_widget_mode_enabled_)
292 ReattachForFullscreenChange(false); 293 ReattachForFullscreenChange(false);
(...skipping 92 matching lines...) Expand 10 before | Expand all | Expand 10 after
385 if (!contents) { 386 if (!contents) {
386 content::WebContents::CreateParams create_params( 387 content::WebContents::CreateParams create_params(
387 browser_context, NULL); 388 browser_context, NULL);
388 return content::WebContents::Create(create_params); 389 return content::WebContents::Create(create_params);
389 } 390 }
390 391
391 return contents; 392 return contents;
392 } 393 }
393 394
394 } // namespace views 395 } // namespace views
OLDNEW
« no previous file with comments | « chrome/renderer/resources/extensions/automation/automation_node.js ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698