Closed
Bug 1310331
Opened 9 years ago
Closed 9 years ago
Support match_about_blank in tabs.executeScript
Categories
(WebExtensions :: Untriaged, defect, P3)
Tracking
(firefox53 verified)
VERIFIED
FIXED
mozilla53
Tracking | Status | |
---|---|---|
firefox53 | --- | verified |
People
(Reporter: zombie, Assigned: zombie)
References
(Depends on 1 open bug, Blocks 1 open bug)
Details
(Whiteboard: triaged)
Attachments
(1 file)
+++ This bug was initially created as a clone of Bug #1272890 +++
Filed as followup after I scoped the original down to only content scripts.
Assignee | ||
Comment 1•9 years ago
|
||
For this to work on about:blank documents loaded in top-level windows (tabs), I believe we need to check permissions against the tab's opener, so marking this blocked on bug 1238314.
We don't need openerTabId, but the decision/platform support on how to reliably determine the opener from that bug.
Depends on: 1238314
Comment 2•9 years ago
|
||
This shouldn't depend on tab opener support. `tabs.create` doesn't support inherited origins, and that won't change with opener support. Origins inherited in windows created with `window.open` might come into play, but only the actual principal of the document should matter in that case.
No longer depends on: 1238314
Comment hidden (mozreview-request) |
Assignee | ||
Updated•9 years ago
|
Assignee: nobody → tomica
Status: NEW → ASSIGNED
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Assignee | ||
Updated•9 years ago
|
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Assignee | ||
Updated•9 years ago
|
Attachment #8807101 -
Flags: review?(kmaglione+bmo)
Comment 8•9 years ago
|
||
mozreview-review |
Comment on attachment 8807101 [details]
bug 1310331 - implement matchAboutBlank for tabs.executeScript
https://reviewboard.mozilla.org/r/90378/#review92554
Nice! Thanks
Attachment #8807101 -
Flags: review?(kmaglione+bmo) → review+
Assignee | ||
Updated•9 years ago
|
Keywords: checkin-needed
Pushed by cbook@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/4971c7511bbc
implement matchAboutBlank for tabs.executeScript r=kmag
Keywords: checkin-needed
Comment 10•9 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
status-firefox53:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla53
Comment 11•8 years ago
|
||
Based on https://bugzilla.mozilla.org/show_bug.cgi?id=1272890#c0 and https://bugzilla.mozilla.org/show_bug.cgi?id=1272890#c36 this issue is verified as fixed on Firefox 53.0a1 (20161121030224) under Windows 7 64-bit and Ubuntu 16.04 LTS 32-bit, after the script is executed '44' is displayed in the browser console and in the web page.
Here is a video: http://screencast.com/t/1FJCDlCx
Status: RESOLVED → VERIFIED
Comment 12•8 years ago
|
||
I have tried it with Firefox 53.0a2 (2017-01-30) (64-bit) on Windows 10 64-bit and it works for executeScript (file or code) but not tabs.sendMessage, which is the method I use to pass some parameters to the injected script.
I have the "Could not establish connection. Receiving end does not exist." error if I execute the sendMessage on the id of the new "about:blank" tab, and "An unexpected error occurred" error if I execute it in the callback of the executeScript.
Is there any limitation on sendMessage for about:blank page? I haven't found any information about that or a matchAboutBlank parameter.
Is this a wanted limitation or another bug?
Thanks.
Assignee | ||
Comment 13•8 years ago
|
||
I'm not sure I follow, but if you can provide a simplified extension that exhibits the problem, please feel free to file a new bug.
Comment 14•8 years ago
|
||
My bad, the issue is the submission on a custom form on the blank tab (the form is created through a content script injected in the blank tab).
This bug is fixed, I will submit a new for the submission issue.
Updated•7 years ago
|
Product: Toolkit → WebExtensions
You need to log in
before you can comment on or make changes to this bug.
Description
•