Issue 6692 - Update frame structure for about:blank frames earlier

ext/background.js

Index: ext/background.js
diff --git a/ext/background.js b/ext/background.js
index 7bf5e393f74b5785b4c83580edc63bee67410d85..2335d144a47875f21d87453cee81b1731e516e29 100644
--- a/ext/background.js
+++ b/ext/background.js
@@ -274,6 +274,18 @@
     }
   });
 
+  browser.webNavigation.onBeforeNavigate.addListener(details =>
+  {
+    // We also need to update the frame structure for about:blank frames, in
+    // order to properly attribute the requests they make.

[Sebastian Noack, 2018/05/29 17:35:09]

This comment is a bit vague. In my tests about:blank frames are handled by
onCommitted, but as it seems too late (i.e. after some sub-requests might
already have been sent).

[kzar, 2018/05/29 17:46:18]

Done.

+    let {url} = details;

[Sebastian Noack, 2018/05/29 17:35:08]

Nit: I don't think this temporary variable is worth to define, it's only used
twice and the property name is quite short.

[kzar, 2018/05/29 17:46:18]

Done.

+    if (url == "about:blank")
+    {
+      updatePageFrameStructure(details.frameId, details.tabId, url,
+                               details.parentFrameId);
+    }
+  });
+
   function forgetTab(tabId)
   {
     ext.pages.onRemoved._dispatch(tabId);