Fix flakiness in DumpAccessibilityEvent* tests.

content/browser/renderer_host/legacy_render_widget_host_win.cc

 // Copyright (c) 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/browser/renderer_host/legacy_render_widget_host_win.h"
 
 #include "base/command_line.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/win/windows_version.h"
 #include "content/browser/accessibility/browser_accessibility_manager_win.h"
@@ skipping 106 matching lines @@
       reinterpret_cast<void **>(window_accessible_.Receive()));
   DCHECK(SUCCEEDED(hr));
 
   if (!BrowserAccessibilityState::GetInstance()->IsAccessibleBrowser()) {
     // Attempt to detect screen readers or other clients who want full
     // accessibility support, by seeing if they respond to this event.
     NotifyWinEvent(EVENT_SYSTEM_ALERT, hwnd(), kIdScreenReaderHoneyPot,
                    CHILDID_SELF);
   }
 
-  // http://crbug.com/440579 TODO(dmazzoni): remove this logging when
-  // flakiness is fixed.
-  LOG(INFO) << "LegacyRenderWidgetHostHWND::Init hwnd=" << hwnd();
-
   return !!SUCCEEDED(hr);
 }
 
 // static
 ui::WindowEventTarget* LegacyRenderWidgetHostHWND::GetWindowEventTarget(
     HWND parent) {
   return reinterpret_cast<ui::WindowEventTarget*>(ui::ViewProp::GetValue(
       parent, ui::WindowEventTarget::kWin32InputEventTarget));
 }
 
 LRESULT LegacyRenderWidgetHostHWND::OnEraseBkGnd(UINT message,
                                                  WPARAM w_param,
                                                  LPARAM l_param) {
   return 1;
 }
 
 LRESULT LegacyRenderWidgetHostHWND::OnGetObject(UINT message,
                                                 WPARAM w_param,
                                                 LPARAM l_param) {
   // Only the lower 32 bits of l_param are valid when checking the object id
   // because it sometimes gets sign-extended incorrectly (but not always).
   DWORD obj_id = static_cast<DWORD>(static_cast<DWORD_PTR>(l_param));
 
-  // http://crbug.com/440579 TODO(dmazzoni): remove this logging when
-  // flakiness is fixed.
-  LOG(INFO) << "LegacyRenderWidgetHostHWND::OnGetObject"
-            << " message=" << message
-            << " w_param=" << w_param
-            << " l_param=" << l_param
-            << " obj_id=" << obj_id
-            << " host_=" << host_;
-
   if (kIdScreenReaderHoneyPot == obj_id) {
     // When an MSAA client has responded to our fake event on this id,
     // enable screen reader support.
     BrowserAccessibilityState::GetInstance()->OnScreenReaderDetected();
     return static_cast<LRESULT>(0L);
   }
 
   if (OBJID_CLIENT != obj_id || !host_)
     return static_cast<LRESULT>(0L);
 
   RenderWidgetHostImpl* rwhi = RenderWidgetHostImpl::From(
       host_->GetRenderWidgetHost());
-  if (!rwhi) {
-    LOG(WARNING) << "No RWHI";
+  if (!rwhi)
     return static_cast<LRESULT>(0L);
-  }
 
   BrowserAccessibilityManagerWin* manager =
       static_cast<BrowserAccessibilityManagerWin*>(
           rwhi->GetRootBrowserAccessibilityManager());
-  if (!manager) {
-    LOG(WARNING) << "No manager";
+  if (!manager)
     return static_cast<LRESULT>(0L);
-  }
 
   base::win::ScopedComPtr<IAccessible> root(
       manager->GetRoot()->ToBrowserAccessibilityWin());
   return LresultFromObject(IID_IAccessible, w_param,
       static_cast<IAccessible*>(root.Detach()));
 }
 
 // We send keyboard/mouse/touch messages to the parent window via SendMessage.
 // While this works, this has the side effect of converting input messages into
 // sent messages which changes their priority and could technically result
@@ skipping 191 matching lines @@
   // generate the legacy WM_VSCROLL/WM_HSCROLL messages.
   // We add these styles to ensure that trackpad/trackpoint scrolling
   // work.
   long current_style = ::GetWindowLong(hwnd(), GWL_STYLE);
   ::SetWindowLong(hwnd(), GWL_STYLE,
                   current_style | WS_VSCROLL | WS_HSCROLL);
   return 0;
 }
 
 }  // namespace content