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

Side by Side Diff: content/renderer/accessibility/renderer_accessibility_complete.cc

Issue 598483002: Fix WeakPtrFactory member ordering in content (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Changed order of friend class Created 6 years, 2 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/renderer/accessibility/renderer_accessibility_complete.h" 5 #include "content/renderer/accessibility/renderer_accessibility_complete.h"
6 6
7 #include <queue> 7 #include <queue>
8 8
9 #include "base/bind.h" 9 #include "base/bind.h"
10 #include "base/message_loop/message_loop.h" 10 #include "base/message_loop/message_loop.h"
(...skipping 17 matching lines...) Expand all
28 using blink::WebPoint; 28 using blink::WebPoint;
29 using blink::WebRect; 29 using blink::WebRect;
30 using blink::WebSettings; 30 using blink::WebSettings;
31 using blink::WebView; 31 using blink::WebView;
32 32
33 namespace content { 33 namespace content {
34 34
35 RendererAccessibilityComplete::RendererAccessibilityComplete( 35 RendererAccessibilityComplete::RendererAccessibilityComplete(
36 RenderFrameImpl* render_frame) 36 RenderFrameImpl* render_frame)
37 : RendererAccessibility(render_frame), 37 : RendererAccessibility(render_frame),
38 weak_factory_(this),
39 tree_source_(render_frame), 38 tree_source_(render_frame),
40 serializer_(&tree_source_), 39 serializer_(&tree_source_),
41 last_scroll_offset_(gfx::Size()), 40 last_scroll_offset_(gfx::Size()),
42 ack_pending_(false) { 41 ack_pending_(false),
42 weak_factory_(this) {
43 WebView* web_view = render_frame_->GetRenderView()->GetWebView(); 43 WebView* web_view = render_frame_->GetRenderView()->GetWebView();
44 WebSettings* settings = web_view->settings(); 44 WebSettings* settings = web_view->settings();
45 settings->setAccessibilityEnabled(true); 45 settings->setAccessibilityEnabled(true);
46 46
47 #if !defined(OS_ANDROID) 47 #if !defined(OS_ANDROID)
48 // Skip inline text boxes on Android - since there are no native Android 48 // Skip inline text boxes on Android - since there are no native Android
49 // APIs that compute the bounds of a range of text, it's a waste to 49 // APIs that compute the bounds of a range of text, it's a waste to
50 // include these in the AX tree. 50 // include these in the AX tree.
51 settings->setInlineTextBoxAccessibilityEnabled(true); 51 settings->setInlineTextBoxAccessibilityEnabled(true);
52 #endif 52 #endif
(...skipping 359 matching lines...) Expand 10 before | Expand all | Expand 10 after
412 render_frame_->GetRenderView()->GetWebView()->clearFocusedElement(); 412 render_frame_->GetRenderView()->GetWebView()->clearFocusedElement();
413 else 413 else
414 obj.setFocused(true); 414 obj.setFocused(true);
415 } 415 }
416 416
417 void RendererAccessibilityComplete::OnFatalError() { 417 void RendererAccessibilityComplete::OnFatalError() {
418 CHECK(false) << "Invalid accessibility tree."; 418 CHECK(false) << "Invalid accessibility tree.";
419 } 419 }
420 420
421 } // namespace content 421 } // namespace content
OLDNEW
« no previous file with comments | « content/renderer/accessibility/renderer_accessibility_complete.h ('k') | content/renderer/npapi/webplugin_impl.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698