-
Notifications
You must be signed in to change notification settings - Fork 335
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: Adds binaryArgs to pass additional flags to Firefox binary #1563
Changes from 1 commit
420b395
f6cd95f
7a8d346
0076326
baa8cab
14af123
117553b
28cb262
41a85b1
22753b7
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -217,10 +217,9 @@ export class FirefoxDesktopExtensionRunner { | |
startUrl, | ||
firefoxApp, | ||
firefoxClient, | ||
binaryArgs = [], | ||
} = this.params; | ||
|
||
const binaryArgs = []; | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. In this version of the PR the arguments are still prepended instead of appended. Restore the There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Oh, looks like I misunderstood you. Does it look fine now - 28cb262? |
||
|
||
if (browserConsole) { | ||
binaryArgs.push('-jsconsole'); | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. uhm... I'm thinking that it may be better to append the additional arguments that the user may be passing to a @Rob--W what do you think about it? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Appending rather than prepending sounds good to me. |
||
} | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This change seems unrelated.