Adrian Aichner [:anaran] Reporter
	Description • 10 years ago

Here's my problem in a nutshell: http://pastebin.mozilla.org/7949929

See [1] at end.

Anything I am obviously doing wrong here?
  
I can download blob's data by clicking the link (download attribute is set)
but "Open in new tab (or window)" gives that error.

Copy link location and I can open it in a new tab manually.

https://developer.mozilla.org/en-US/Add-ons/Working_with_multiprocess_Firefox
does not tell me how I might violate e10s in this add-on, works fine without e10s.

Could this be related to bug 1058470?

[1]

12:59:20.925 NS_NOINTERFACE: Component returned failure code: 0x80004002 (NS_NOINTERFACE) [nsISupports.QueryInterface] <unknown>
12:59:20.949 Security Error: Content at resource://jid1-he38y6vsw9tphg-at-jetpack/data/display.html may not load data from blob:resource://jid1-he38y6vsw9tphg-at-jetpack/a343ec2a-b261-4331-a711-799d2cc777b5.
12:59:20.950 uncaught exception: Load of blob:resource://jid1-he38y6vsw9tphg-at-jetpack/a343ec2a-b261-4331-a711-799d2cc777b5 from resource://jid1-he38y6vsw9tphg-at-jetpack/data/display.html denied. <unknown>

The blob in question is set by the content script as follows:

(sources at https://github.com/anaran/JotFirefox)

    self.port.on('setJotEntriesBlob', function(data) {
      let blob = new window.Blob([data.content], {
        type: 'text/plain; charset=utf-8'
      });
      if ( !! data.download) {
        DEBUG_ADDON &&
          console.log('we will dowload as well', data);
      }
      if (links[data.type]) {
        if ( !! data.download) {
          links[data.type].click();
        } else {
          links[data.type].href = window.URL.createObjectURL(blob);
          links[data.type].download = data.filename;
        }
      } else {
        DEBUG_ADDON &&
          console.error('Don\'t know how to handle content type ' + data.type);
      }
    });

	Andrea Marchesini [:baku]
	Comment 1 • 10 years ago

Actually this issue can be reproduced with a normal html page and e10s. Right click on the link -> nothing happens except this:

[Parent 8856] WARNING: No docshells for remote frames!: file /home/baku/Sources/m/foobar/src/dom/base/nsFrameLoader.cpp, line 514
[Parent 8856] WARNING: No docshells for remote frames!: file /home/baku/Sources/m/foobar/src/dom/base/nsFrameLoader.cpp, line 514
JavaScript error: , line 0: NS_NOINTERFACE: Component returned failure code: 0x80004002 (NS_NOINTERFACE) [nsISupports.QueryInterface]
JavaScript error: , line 0: NS_NOINTERFACE: Component returned failure code: 0x80004002 (NS_NOINTERFACE) [nsISupports.QueryInterface]
JavaScript error: , line 0: NS_NOINTERFACE: Component returned failure code: 0x80004002 (NS_NOINTERFACE) [nsISupports.QueryInterface]
JavaScript error: , line 0: NS_NOINTERFACE: Component returned failure code: 0x80004002 (NS_NOINTERFACE) [nsISupports.QueryInterface]
JavaScript error: , line 0: NS_NOINTERFACE: Component returned failure code: 0x80004002 (NS_NOINTERFACE) [nsISupports.QueryInterface]
JavaScript error: , line 0: uncaught exception: Load of blob:http://localhost/3afeba68-d38e-4149-b357-29de9b09d776 from http://localhost/a.html denied.

	Adrian Aichner [:anaran] Reporter
	Comment 2 • 9 years ago

Apparently I have filed this bug long ago, but after the swift confirmation and reduction by :baku nothing happened.

This seems to be in e10s land, not add-on SDK.

Shouldn't this issue get some attention by now? I bet it's not limited to Windows either.

	Dave Townsend [:mossop]
	Comment 3 • 9 years ago

Looks likely to be some kind of networking issue.

	Brad Lassey [:blassey] (use needinfo?)
	Comment 4 • 9 years ago

Blake, could you have a look at this?

	Blake Kaplan (:mrbkap) (inactive) Assignee
	Comment 5 • 9 years ago

The context menu code is busted, we should fix it.

	Neil Deakin
	Comment 6 • 9 years ago

The testcase here seems to work fine. Is this fixed?

	Loic
	Comment 7 • 9 years ago

(In reply to Neil Deakin from comment #6)
> The testcase here seems to work fine. Is this fixed?

I don't think so, you can't open the blob in another tab or window, and "save as" (to save the blob as .txt) doesn't work.

	Andy McKay
	Comment 9 • 8 years ago

Would you have time to test this again? We wonder if its been fixed with some other changes in the mean time.

	:shell escalante
	Comment 10 • 8 years ago

please re-open if you still see in your testing.

	Adrian Aichner [:anaran] Reporter
	Comment 11 • 8 years ago

Works for me too in latest nightly on linux.