Use an OOPIF-compatible call chain when finding the top-level frame

components/autofill/content/renderer/password_autofill_agent.cc

Index: components/autofill/content/renderer/password_autofill_agent.cc
diff --git a/components/autofill/content/renderer/password_autofill_agent.cc b/components/autofill/content/renderer/password_autofill_agent.cc
index 2d8f6c9177b394dcded406d730647744d3e3bc09..565e9aca6300cacb6b987984fad375ca06b7ba2c 100644
--- a/components/autofill/content/renderer/password_autofill_agent.cc
+++ b/components/autofill/content/renderer/password_autofill_agent.cc
@@ -853,12 +853,10 @@ bool PasswordAutofillAgent::ShowSuggestions(
     if ((element.isPasswordField() ||
          HasAutocompleteAttributeValue(element, "username")) &&
         security_state::IsHttpWarningInFormEnabled() &&
-        !content::IsOriginSecure(url::Origin(render_frame()
-                                                 ->GetRenderView()
-                                                 ->GetMainRenderFrame()
-                                                 ->GetWebFrame()
-                                                 ->getSecurityOrigin())
-                                     .GetURL())) {
+        !content::IsOriginSecure(
+            url::Origin(
+                render_frame()->GetWebFrame()->top()->getSecurityOrigin())

[nasko, 2017/01/27 21:43:35]

Can't we use getSecurityOrigin().isPotentiallyTrustworthy() here? The multiple
conversions going on are too hard to follow and might lead to subtle mistakes.

+                .GetURL())) {
       autofill_agent_->ShowNotSecureWarning(element);
       return true;
     }