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

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

Issue 7867018: Add routing id and notification type to accessibility logging. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Add NOTREACHED() call Created 9 years, 3 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 | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2011 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2011 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_view.h" 5 #include "content/renderer/render_view.h"
6 6
7 #include <algorithm> 7 #include <algorithm>
8 #include <cmath> 8 #include <cmath>
9 #include <string> 9 #include <string>
10 #include <vector> 10 #include <vector>
(...skipping 1585 matching lines...) Expand 10 before | Expand all | Expand 10 after
1596 1596
1597 ViewHostMsg_AccessibilityNotification_Params param; 1597 ViewHostMsg_AccessibilityNotification_Params param;
1598 WebAccessibilityNotificationToViewHostMsg( 1598 WebAccessibilityNotificationToViewHostMsg(
1599 pending_accessibility_notifications_[i].type, &param.notification_type); 1599 pending_accessibility_notifications_[i].type, &param.notification_type);
1600 param.acc_obj = WebAccessibility( 1600 param.acc_obj = WebAccessibility(
1601 obj, accessibility_.get(), notification.ShouldIncludeChildren()); 1601 obj, accessibility_.get(), notification.ShouldIncludeChildren());
1602 notifications.push_back(param); 1602 notifications.push_back(param);
1603 1603
1604 #ifndef NDEBUG 1604 #ifndef NDEBUG
1605 if (accessibility_logging_) { 1605 if (accessibility_logging_) {
1606 LOG(INFO) << "Accessibility update: \n" 1606 LOG(INFO) << "Accessibility update:\n"
1607 << param.acc_obj.DebugString(true); 1607 << "(routing id: " << routing_id_ << " notification type: "
1608 << ViewHostMsg_AccEvent::DebugString(param.notification_type)
1609 << "\n" << param.acc_obj.DebugString(true);
1608 } 1610 }
1609 #endif 1611 #endif
1610 } 1612 }
1611 pending_accessibility_notifications_.clear(); 1613 pending_accessibility_notifications_.clear();
1612 Send(new ViewHostMsg_AccessibilityNotifications(routing_id_, notifications)); 1614 Send(new ViewHostMsg_AccessibilityNotifications(routing_id_, notifications));
1613 accessibility_ack_pending_ = true; 1615 accessibility_ack_pending_ = true;
1614 } 1616 }
1615 1617
1616 bool RenderView::handleCurrentKeyboardEvent() { 1618 bool RenderView::handleCurrentKeyboardEvent() {
1617 if (edit_commands_.empty()) 1619 if (edit_commands_.empty())
(...skipping 2972 matching lines...) Expand 10 before | Expand all | Expand 10 after
4590 } 4592 }
4591 4593
4592 void RenderView::OnEnableViewSourceMode() { 4594 void RenderView::OnEnableViewSourceMode() {
4593 if (!webview()) 4595 if (!webview())
4594 return; 4596 return;
4595 WebFrame* main_frame = webview()->mainFrame(); 4597 WebFrame* main_frame = webview()->mainFrame();
4596 if (!main_frame) 4598 if (!main_frame)
4597 return; 4599 return;
4598 main_frame->enableViewSourceMode(true); 4600 main_frame->enableViewSourceMode(true);
4599 } 4601 }
OLDNEW
« content/common/view_messages.h ('K') | « content/common/view_messages.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698