Skip to content
This repository has been archived by the owner on Sep 2, 2021. It is now read-only.

Merging changes to release branch for 1.14.1 patch release #671

Open
wants to merge 3 commits into
base: release
Choose a base branch
from

Conversation

g-217
Copy link
Contributor

@g-217 g-217 commented Nov 18, 2019

Merging the changes done by following PRs:
#668
#669
#673

…ingPort() needs a callback argument to work (adobe#668)

* Disabled remote-debugging by default and appshell.app.getRemoteDebuggingPort() needs a callback argument to work

* Addressing review comments

* Addressing review comments

* Updating error meesage for port validation

* Addressing review comments from vickramdhawal, also it is not possible to use g_handler to hold remote-debugging-port as g_handler does not get initialized while we are parsing command line

* More error message correction
@g-217
Copy link
Contributor Author

g-217 commented Nov 18, 2019

@shubhsnov cherry-picked the changes from master.

…r-to-infobar2

moving command line error to infobar and fixing windows port range issues
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants