Skip to content
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

Adding "proper" message port for fake worker. #7624

Merged
merged 1 commit into from Sep 12, 2016
Merged

Adding "proper" message port for fake worker. #7624

merged 1 commit into from Sep 12, 2016

Conversation

Snuffleupagus
Copy link
Collaborator

@Snuffleupagus Snuffleupagus commented Sep 12, 2016

This PR contains the patch from PR #6948, rebased to master, with review nits addressed, and obviously maintaining the correct author information.

Attempt to properly fix fake worker port. See also #6938.

Edit: Supersedes, hence fixes #6948.

@Snuffleupagus
Copy link
Collaborator Author

Given that we don't test with workers disabled (which is a good thing, given how long that'd probably take), I'm not sure how much value there is in running the tests but let's do it for good measure anyway.

/botio test

@pdfjsbot
Copy link

From: Bot.io (Windows)


Received

Command cmd_test from @Snuffleupagus received. Current queue size: 0

Live output at: http://107.22.172.223:8877/ac3e7fcd5c333b4/output.txt

@pdfjsbot
Copy link

From: Bot.io (Linux)


Received

Command cmd_test from @Snuffleupagus received. Current queue size: 0

Live output at: http://107.21.233.14:8877/4a7409ffc2fdb44/output.txt

@pdfjsbot
Copy link

From: Bot.io (Windows)


Success

Full output at http://107.22.172.223:8877/ac3e7fcd5c333b4/output.txt

Total script time: 24.47 mins

  • Font tests: Passed
  • Unit tests: Passed
  • Regression tests: Passed

@pdfjsbot
Copy link

From: Bot.io (Linux)


Success

Full output at http://107.21.233.14:8877/4a7409ffc2fdb44/output.txt

Total script time: 39.97 mins

  • Font tests: Passed
  • Unit tests: Passed
  • Regression tests: Passed

@Snuffleupagus Snuffleupagus merged commit 230b1e3 into mozilla:master Sep 12, 2016
@Snuffleupagus Snuffleupagus deleted the yury-fake-cloning branch September 12, 2016 10:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants