Actually I believe this happens when opening any file:// URL, not just when opening from Finder.

Tested on reported #59.0.3071.115 and unable to reproduce the issue. Attached is the screencast for reference.

Tried to open the test file using the keyboard controls command+o and pasting the URL 'file:///Users/pnangunoori/Downloads/test2.html' in omni box.

@lenny.khazan -- Could you please look into this and provide us the consistent steps to reproduce this issue.

Thanks in advance.

Deleted: 749117.mov 6.4 MB

749117.mov 6.4 MB Download

@pnangunoori: Apologies, I should have clarified that CHROME_IPC_LOGGING is an environment variable, I don't think Chromium supports that as a CLI flag (I'm not seeing the IPC messages show up for you in Terminal).

Thank you for providing more feedback. Adding requester "pnangunoori@chromium.org" to the cc list and removing "Needs-Feedback" label.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot

Adding the label 'TE-NeedsTriageHelp' for further investigation.

[MacTriage] Hitting DCHECK at host_port_pair.cc:72.

[mac triage]

std::string HostPortPair::HostForURL() const {
  // TODO(rtenneti): Add support for |host| to have '\0'.
  if (host_.find('\0') != std::string::npos) {
    std::string host_for_log(host_);
    size_t nullpos;
    while ((nullpos = host_for_log.find('\0')) != std::string::npos) {
      host_for_log.replace(nullpos, 1, "%00");
    }
    LOG(DFATAL) << "Host has a null char: " << host_for_log;
  }
  // Check to see if the host is an IPv6 address.  If so, added brackets.
  if (host_.find(':') != std::string::npos) {
    DCHECK_NE(host_[0], '[');
*   return base::StringPrintf("[%s]", host_.c_str());
  }

  return host_;
}

DCHECK added in r53403 - maybe we can just remove it?

Hmm, it's been a while since I touched that code.

Well, the input that's failing the DCHECK is "[2607:f8b0:4001:c20::68]". Seems wrong to turn that into "[[2607:f8b0:4001:c20::68]]". Someone is probably calling set_host() with the wrong input host. The comment for HostPortPair::host_ (written by me) says |host_| should not contain brackets for IPv6 addresses.

The set_host() in content::RenderFrameImpl::DidCommitProvisionalLoad() is the most obvious culprit. Not sure if we want to HostPortPair to allow brackets in |host_|, or to add more DCHECKs to prevent set_host("[ipv6_addr]") and fix the callers.

My own preference is to fix callers - a caller that puts in a hostname with braces is likely a caller that expects to get out a hostname in the same format.