Don't focus the location bar in a phishy situation.

content/browser/web_contents/web_contents_impl.cc

Index: content/browser/web_contents/web_contents_impl.cc
diff --git a/content/browser/web_contents/web_contents_impl.cc b/content/browser/web_contents/web_contents_impl.cc
index 61bddb1d90a0f21c98a8c9f035a2ddaeb3a328f4..562a8507114972cc02e1498bd271036c3ce195bc 100644
--- a/content/browser/web_contents/web_contents_impl.cc
+++ b/content/browser/web_contents/web_contents_impl.cc
@@ -2915,8 +2915,11 @@ void WebContentsImpl::ResumeLoadingCreatedWebContents() {
 
 bool WebContentsImpl::FocusLocationBarByDefault() {
   NavigationEntry* entry = controller_.GetVisibleEntry();
-  if (entry && entry->GetURL() == GURL(url::kAboutBlankURL))
+  if (entry &&
+      entry->GetTransitionType() != ui::PageTransition::PAGE_TRANSITION_LINK &&
+      entry->GetURL() == GURL(url::kAboutBlankURL)) {

[Charlie Reis, 2016/02/10 05:47:48]

Here's what I would recommend (if we decide to keep the focus-about-blank logic
for startup):

// If we are starting at about:blank, give the omnibox focus to let the user
easily edit it.  We intentionally check the pending entry rather than the
visible one, since we don't want to be tricked by non-visible pending URLs.  See
https://crbug.com/567445.
NavigationEntryImpl* entry = controller_.GetPendingEntry();
if (controller_.IsInitialNavigation() &&
    entry && !entry->is_renderer_initiated() &&
    entry->GetURL() == GURL(url::kAboutBlankURL)) {
...

[palmer, 2016/02/10 23:15:25]

OK, I did that, and added a sketch of a test that tests nothing. :) Do you have
any clues about how I can make a test that does do something? Do I need to
migrate it to chrome browsertests, or is there some vestigial LocationBar in
content/ (doesn't look like it)?

     return true;
+  }
   return delegate_ && delegate_->ShouldFocusLocationBarByDefault(this);
 }