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

Side by Side Diff: content/browser/accessibility/accessibility_event_recorder_win.cc

Issue 2561963002: base: Remove the string logging from CHECK(). (Closed)
Patch Set: checkstring: rebase Created 4 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 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 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/accessibility/accessibility_event_recorder.h" 5 #include "content/browser/accessibility/accessibility_event_recorder.h"
6 6
7 #include <oleacc.h> 7 #include <oleacc.h>
8 #include <stdint.h> 8 #include <stdint.h>
9 9
10 #include <string> 10 #include <string>
(...skipping 118 matching lines...) Expand 10 before | Expand all | Expand 10 after
129 DWORD event_time) { 129 DWORD event_time) {
130 if (instance_) { 130 if (instance_) {
131 instance_->OnWinEventHook(handle, event, hwnd, obj_id, child_id, 131 instance_->OnWinEventHook(handle, event, hwnd, obj_id, child_id,
132 event_thread, event_time); 132 event_thread, event_time);
133 } 133 }
134 } 134 }
135 135
136 AccessibilityEventRecorderWin::AccessibilityEventRecorderWin( 136 AccessibilityEventRecorderWin::AccessibilityEventRecorderWin(
137 BrowserAccessibilityManager* manager) 137 BrowserAccessibilityManager* manager)
138 : AccessibilityEventRecorder(manager) { 138 : AccessibilityEventRecorder(manager) {
139 CHECK(!instance_) << "There can be only one instance of" 139 // There can be only one instance of WinAccessibilityEventMonitor at a time.
140 << " WinAccessibilityEventMonitor at a time."; 140 CHECK(!instance_);
141 instance_ = this; 141 instance_ = this;
142 win_event_hook_handle_ = SetWinEventHook( 142 win_event_hook_handle_ = SetWinEventHook(
143 EVENT_MIN, 143 EVENT_MIN,
144 EVENT_MAX, 144 EVENT_MAX,
145 GetModuleHandle(NULL), 145 GetModuleHandle(NULL),
146 &AccessibilityEventRecorderWin::WinEventHookThunk, 146 &AccessibilityEventRecorderWin::WinEventHookThunk,
147 GetCurrentProcessId(), 147 GetCurrentProcessId(),
148 0, // Hook all threads 148 0, // Hook all threads
149 WINEVENT_INCONTEXT); 149 WINEVENT_INCONTEXT);
150 CHECK(win_event_hook_handle_); 150 CHECK(win_event_hook_handle_);
(...skipping 136 matching lines...) Expand 10 before | Expand all | Expand 10 after
287 if (accessibility_hwnd != hwnd) 287 if (accessibility_hwnd != hwnd)
288 return E_FAIL; 288 return E_FAIL;
289 289
290 IAccessible* obj = ToBrowserAccessibilityWin(manager_->GetRoot()); 290 IAccessible* obj = ToBrowserAccessibilityWin(manager_->GetRoot());
291 obj->AddRef(); 291 obj->AddRef();
292 *ppv_object = obj; 292 *ppv_object = obj;
293 return S_OK; 293 return S_OK;
294 } 294 }
295 295
296 } // namespace content 296 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698