Use static_cast instead of reinterpret_cast for downcasting

static_cast is better suited for downcasting since it can check that the
types are related.

/chrome/browser/ui/views
This CL was uploaded by git cl split.

R=pbos@chromium.org

Change-Id: Ifd8b52796b8a642ac5fbcf8b4c069f0eac730437
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2435804
Auto-Submit: Kevin McNee <mcnee@chromium.org>
Reviewed-by: Peter Boström <pbos@chromium.org>
Commit-Queue: Peter Boström <pbos@chromium.org>
Cr-Commit-Position: refs/heads/master@{#811396}

chrome/browser/ui/views/keyboard_access_browsertest.cc

@@ -193,8 +193,7 @@ void KeyboardAccessTest::TestMenuKeyboardAccess(bool alternate_key_sequence,
 
   ui_test_utils::TabAddedWaiter tab_add(browser());
 
-  BrowserView* browser_view = reinterpret_cast<BrowserView*>(
-      browser()->window());
+  BrowserView* browser_view = static_cast<BrowserView*>(browser()->window());
   SendKeysMenuListener menu_listener(
       browser_view->toolbar_button_provider()->GetAppMenuButton(), browser(),
       false);
@@ -365,8 +364,7 @@ void KeyboardAccessTest::TestMenuKeyboardAccessAndDismiss() {
 
   int original_view_id = GetFocusedViewID();
 
-  BrowserView* browser_view = reinterpret_cast<BrowserView*>(
-      browser()->window());
+  BrowserView* browser_view = static_cast<BrowserView*>(browser()->window());
   SendKeysMenuListener menu_listener(
       browser_view->toolbar_button_provider()->GetAppMenuButton(), browser(),
       true);