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

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

Issue 1507413003: clang/win: Let some chromium_code targets build with -Wextra. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: content_browsertests Created 5 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 (c) 2014 The Chromium Authors. All rights reserved. 1 // Copyright (c) 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/renderer_host/legacy_render_widget_host_win.h" 5 #include "content/browser/renderer_host/legacy_render_widget_host_win.h"
6 6
7 #include "base/command_line.h" 7 #include "base/command_line.h"
8 #include "base/memory/scoped_ptr.h" 8 #include "base/memory/scoped_ptr.h"
9 #include "base/win/windows_version.h" 9 #include "base/win/windows_version.h"
10 #include "content/browser/accessibility/browser_accessibility_manager_win.h" 10 #include "content/browser/accessibility/browser_accessibility_manager_win.h"
(...skipping 148 matching lines...) Expand 10 before | Expand all | Expand 10 after
159 // because it sometimes gets sign-extended incorrectly (but not always). 159 // because it sometimes gets sign-extended incorrectly (but not always).
160 DWORD obj_id = static_cast<DWORD>(static_cast<DWORD_PTR>(l_param)); 160 DWORD obj_id = static_cast<DWORD>(static_cast<DWORD_PTR>(l_param));
161 161
162 if (kIdScreenReaderHoneyPot == obj_id) { 162 if (kIdScreenReaderHoneyPot == obj_id) {
163 // When an MSAA client has responded to our fake event on this id, 163 // When an MSAA client has responded to our fake event on this id,
164 // enable screen reader support. 164 // enable screen reader support.
165 BrowserAccessibilityState::GetInstance()->OnScreenReaderDetected(); 165 BrowserAccessibilityState::GetInstance()->OnScreenReaderDetected();
166 return static_cast<LRESULT>(0L); 166 return static_cast<LRESULT>(0L);
167 } 167 }
168 168
169 if (OBJID_CLIENT != obj_id || !host_) 169 if (static_cast<DWORD>(OBJID_CLIENT) != obj_id || !host_)
170 return static_cast<LRESULT>(0L); 170 return static_cast<LRESULT>(0L);
171 171
172 RenderWidgetHostImpl* rwhi = RenderWidgetHostImpl::From( 172 RenderWidgetHostImpl* rwhi = RenderWidgetHostImpl::From(
173 host_->GetRenderWidgetHost()); 173 host_->GetRenderWidgetHost());
174 if (!rwhi) 174 if (!rwhi)
175 return static_cast<LRESULT>(0L); 175 return static_cast<LRESULT>(0L);
176 176
177 BrowserAccessibilityManagerWin* manager = 177 BrowserAccessibilityManagerWin* manager =
178 static_cast<BrowserAccessibilityManagerWin*>( 178 static_cast<BrowserAccessibilityManagerWin*>(
179 rwhi->GetRootBrowserAccessibilityManager()); 179 rwhi->GetRootBrowserAccessibilityManager());
(...skipping 224 matching lines...) Expand 10 before | Expand all | Expand 10 after
404 direct_manipulation_helper_->Activate(hwnd()); 404 direct_manipulation_helper_->Activate(hwnd());
405 } else if (window_pos->flags & SWP_HIDEWINDOW) { 405 } else if (window_pos->flags & SWP_HIDEWINDOW) {
406 direct_manipulation_helper_->Deactivate(hwnd()); 406 direct_manipulation_helper_->Deactivate(hwnd());
407 } 407 }
408 } 408 }
409 SetMsgHandled(FALSE); 409 SetMsgHandled(FALSE);
410 return 0; 410 return 0;
411 } 411 }
412 412
413 } // namespace content 413 } // namespace content
OLDNEW
« no previous file with comments | « content/browser/gamepad/raw_input_data_fetcher_win.cc ('k') | content/browser/system_message_window_win.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698