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/ssl
This CL was uploaded by git cl split.

R=estark@chromium.org

Change-Id: Ifc807a2bc88c612e3099ec92c5a14d693a2fccb6
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2435581
Auto-Submit: Kevin McNee <mcnee@chromium.org>
Reviewed-by: Emily Stark <estark@chromium.org>
Commit-Queue: Emily Stark <estark@chromium.org>
Cr-Commit-Position: refs/heads/master@{#811599}

chrome/browser/ssl/ssl_browsertest.cc

@@ -4410,7 +4410,7 @@ IN_PROC_BROWSER_TEST_F(SSLUITest, BadCertFollowedByGoodCert) {
 
   Profile* profile = Profile::FromBrowserContext(tab->GetBrowserContext());
   StatefulSSLHostStateDelegate* state =
-      reinterpret_cast<StatefulSSLHostStateDelegate*>(
+      static_cast<StatefulSSLHostStateDelegate*>(
           profile->GetSSLHostStateDelegate());
 
   // First check that frame requests revoke the decision.
@@ -4457,7 +4457,7 @@ IN_PROC_BROWSER_TEST_F(SSLUITest, BadCertFollowedByBlobUrl) {
   WebContents* tab = browser()->tab_strip_model()->GetActiveWebContents();
   Profile* profile = Profile::FromBrowserContext(tab->GetBrowserContext());
   StatefulSSLHostStateDelegate* state =
-      reinterpret_cast<StatefulSSLHostStateDelegate*>(
+      static_cast<StatefulSSLHostStateDelegate*>(
           profile->GetSSLHostStateDelegate());
 
   // Proceed through the interstitial, accepting the broken cert.
@@ -8132,7 +8132,7 @@ IN_PROC_BROWSER_TEST_F(RecurrentInterstitialBrowserTest,
       net::ERR_CERTIFICATE_TRANSPARENCY_REQUIRED);
 
   StatefulSSLHostStateDelegate* state =
-      reinterpret_cast<StatefulSSLHostStateDelegate*>(
+      static_cast<StatefulSSLHostStateDelegate*>(
           browser()->profile()->GetSSLHostStateDelegate());
   state->ResetRecurrentErrorCountForTesting();