1580241 - Fix usage of nsIDocShellTreeItem in nsDocShell::CanSavePresentation

Status: RESOLVED FIXED

(Core :: DOM: Navigation, defect, P2)

Description

[Kannan Vijayan [:djvj]]

https://searchfox.org/mozilla-central/source/docshell/base/nsDocShell.cpp#7155

This should be a simple fix, but there is an existing confusion in the code here (it seems to be correct, but for the wrong reasons).

  nsCOMPtr<nsIDocShellTreeItem> root;
  GetInProcessSameTypeParent(getter_AddRefs(root));
  if (root && root != this) {
    return false;  // this is a subframe load
  }

The variable is called root, and the code checks the variable using logic that would work if it was actually the root, but the variable is set to the parent docshell, which incidentally happens to also work under the logic (if the parent exists and returns non-null, then the root != this condition will always be true anyway).

Getting back to the Fission fix - this should be changed to use the BrowsingContext to query for the same-type parent, and also change the code to call the variable parent, and the conditional to simply if (parent) { ... }.

Comment 1

[Neha Kochar [:neha]]

This blocks toolkit/components/passwordmgr/test/mochitest/test_autofill_from_bfcache.html for fission.

Comment 2

[Kannan Vijayan [:djvj]]

Attachment: Bug 1580241 - Fix usage of nsIDocShellTreeItem in nsDocShell::CanSavePresentation r?kmag

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/dcd04d1caf93
Fix usage of nsIDocShellTreeItem in nsDocShell::CanSavePresentation r=kmag

Comment 4

[Narcis Beleuzu [:NarcisB]]

https://hg.mozilla.org/mozilla-central/rev/dcd04d1caf93

Comment 5

[Matthew N. [:MattN]]

(In reply to Neha Kochar [:neha] from comment #1)

This blocks toolkit/components/passwordmgr/test/mochitest/test_autofill_from_bfcache.html for fission.

Note that this bug didn't enable that test and it still fails with the fix.