Closed Bug 1265202 Opened 3 years ago Closed 3 years ago

Install mozscreenshots as a temporary add-on


(Testing :: mozscreenshots, defect)

Not set


(firefox48 fixed)

Tracking Status
firefox48 --- fixed


(Reporter: MattN, Assigned: MattN)


(Blocks 1 open bug)



(2 files)

mozscreenshots got signed by bug 1250616. It would be more convenient if rather than being signed, they could be installed as temporary addons.

Bug 1251633 was coming up with a general approach (which I don't fully understand) but we can just go back to the attachment 8612148 [details] [diff] [review] approach of installing the extension ourselves.
Comment on attachment 8742091 [details]
MozReview Request: Bug 1265202 - Install mozscreenshots as a temporary add-on. r=bgrins
Attachment #8742091 - Flags: review?(bgrinstead) → review+
Comment on attachment 8742090 [details]
MozReview Request: Bug 1265202 - Back out changeset ef98cb11ba62 from bug 12506160 which used a signed XPI. r=bgrins

If that file wasn't added I'd rubber stamp the rest, but I don't know enough about the build system to confidently review that.  Can you find another reviewer?

::: browser/tools/mozscreenshots/mozscreenshots/extension/
(Diff revision 1)
> +# License, v. 2.0. If a copy of the MPL was not distributed with this
> +# file, You can obtain one at
> +
> +TEST_EXTENSIONS_DIR = $(DEPTH)/_tests/testing/mochitest/extensions

Looks like this file has to do with the xpi, which is being removed, right?  Why's this needed?
Attachment #8742090 - Flags: review?(bgrinstead)

This is a straight backout so this code was already reviewed during its previous landing.

> Looks like this file has to do with the xpi, which is being removed, right?  Why's this needed?

We still need it as we still need to stage the extension. The test code is pointing to the output from the build system (like it was before the signing change), not the source directory.

Eventually it may be possible to treat the extension files like regular test files that happen to be in a directory structure appropriate for an extension and avoid any preprocessing but that's a separate issue really.
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla48
Component: Mochitest → mozscreenshots
Target Milestone: mozilla48 → ---
You need to log in before you can comment on or make changes to this bug.