Closed Bug 1194219 Opened 9 years ago Closed 9 years ago

[gui] add the ability to use addons

Categories

(Testing :: mozregression, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: parkouss, Assigned: sabergeass, Mentored)

Details

Attachments

(1 file)

49 bytes, text/x-github-pull-request
parkouss
: review+
parkouss
: feedback+
Details | Review
This is equivalent to: -e PATH1, --addon PATH1 addon to install; repeat for multiple addons. command line option. So, I think we should add in the Profile wizard page a list to be able to add/remove addons. This could looks like: http://doc.qt.io/qt-4.8/images/assistant-preferences-documentation.png After this is done, we should pass the addon list (or None if there is none) to the Bisection - from memory, it should looks like the profile. The code in mozregression/main.py might help: https://github.com/mozilla/mozregression/blob/0bfa1b8df0d865e3641814a1aa6fefa7c0c11ff9/mozregression/main.py#L361.
Attached file PR for bug 1194219
Hmm, not sure about it. No error shows up, but when I test it locally, It seems like also found nothing to me. Just ask feedback for now, and I will keep diging it until I figure out the problem.
Attachment #8649340 - Flags: feedback?(j.parkouss)
Comment on attachment 8649340 [details] [review] PR for bug 1194219 It works for me with a simple fix: diff --git a/gui/mozregui/bisection.py b/gui/mozregui/bisection.py index 61ebc44..063b4e2 100644 --- a/gui/mozregui/bisection.py +++ b/gui/mozregui/bisection.py @@ -258,8 +258,7 @@ class BisectRunner(QObject): launcher_kwargs[name] = value # add add-ons path to the app luncher - for path in options['addons']: - launcher_kwargs['addons'] = path + launcher_kwargs['addons'] = options['addons'] self.bisector.test_runner.launcher_kwargs = launcher_kwargs self.thread.start() I am able to run a process with an addon after that. :) This is just because launcher_kwargs['addons'] needs a list of string. Also, I will make some comment on the PR, but otherwise this is good!
Attachment #8649340 - Flags: feedback?(j.parkouss) → feedback+
Attachment #8649340 - Flags: review?(j.parkouss)
Comment on attachment 8649340 [details] [review] PR for bug 1194219 This is great! Thanks MikeLing - I did not tested yet but I assume it works (will try it before merging). Have you tested it ? I commented a few things on travis PR, just minor nits. I'll be happy to merge this in after you fixed them!
Attachment #8649340 - Flags: review?(j.parkouss) → review+
(In reply to Julien Pagès from comment #3) > Comment on attachment 8649340 [details] [review] > PR for bug 1194219 > > This is great! > > Thanks MikeLing - I did not tested yet but I assume it works (will try it > before merging). Have you tested it ? > > I commented a few things on travis PR, just minor nits. > > I'll be happy to merge this in after you fixed them! Just test the PR and it works fine I think :)
Thanks MikeLing for the work here! Just merged this manually in https://github.com/mozilla/mozregression/commit/bd1110e2ee242450458663dd2a1c7e1155a89f7f.
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: