Builder linux-rel-css Build #4540

Results:

Build successful

SourceStamp:

BuildSlave:

Reason:

The AnyBranchScheduler scheduler named 'servo-auto' triggered this build

Steps and Logfiles:

1. git update ( 4 secs )
   1. stdio
2. generic generic ( 0 secs )
   1. - no logs -
3. generic_1 generic_1 ( 0 secs )
   1. - no logs -
4. shell 'pkill -x ...' ( 0 secs )
   1. stdio
5. shell__1 './mach clean-nightlies ...' ( 8 secs )
   1. stdio
6. compile ran build --release --with-debug-assertions warnings ( 17 mins, 1 secs )
   1. warnings (2)
7. test ran test-css --release --processes 16 --always-succeed ( 9 mins, 47 secs )
   1. css-errorsummary.log
8. shell__2 './mach filter-intermittents ...' ( 0 secs )
   1. stdio
   2. filtered-css-errorsummary.log
   3. intermittents.log
9. compile__1 ran build-cef --release --with-debug-assertions warnings ( 36 secs )
   1. stdio
   2. warnings (2)
10. compile__2 ran build-geckolib --release ( 6 mins, 22 secs )
    1. stdio
11. test__1 ran test-stylo --release ( 6 mins, 18 secs )
    1. - no logs -
12. shell__3 'bash ./etc/ci/lockfile_changed.sh' ( 0 secs )
    1. stdio
13. shell__4 'bash ./etc/ci/manifest_changed.sh' ( 0 secs )
    1. stdio

Build Properties:

Forced Build Properties:

Responsible Users:

1. bors-servo

   lbergstrom+borsohnoyoudont@mozilla.com

Timing:

All Changes:

:

1. Change #50889

   Category	None
   Changed by	bors-servo <lbergstrom+borsohnoyoudont@mozilla.com>
   Changed at	Mon 12 Jun 2017 04:23:46
   Repository	https://github.com/servo/servo
   Project	servo/servo
   Branch	auto
   Revision	87140641a4f8636b431db41777d01302f8f6ad3d

   Comments

   Auto merge of #16477 - sadmansk:url_param_browser, r=paulrouget
   
   Pass URL to Browser::new(), delegate url checking logic to 3rd party
   
   <!-- Please describe your changes on the following line: -->
   1. Move the logic of computing the initial url from `opts.rs` to `/ports/servo/main.rs`
   2. Add a `ServoUrl` argument to `Browser::new`
   
   Based on the requested changes by @paulrouget:
   >We can read the pref in main() instead. shell.homepage would be used if the url is not passed as an argument. I'm trying to decouple the "app" logic and the "web engine" logic. I think it's up to the app to set the initial URL, and I'm not sure the initial url should be part of opts.
   
   ---
   <!-- Thank you for contributing to Servo! Please replace each `[ ]` by `[X]` when the step is complete, and replace `__` with appropriate data: -->
   - [x] `./mach build -d` does not report any errors
   - [x] `./mach test-tidy` does not report any errors
   - [x] These changes fix #15636
   
   <!-- Either: -->
   - [ ] There are tests for these changes
   
   <!-- Also, please make sure that "Allow edits from maintainers" checkbox is checked, so that we can help you if you get stuck somewhere along the way.-->
   
   <!-- Pull requests that do not address these steps are welcome, but they will require additional verification as part of the review process. -->
   
   <!-- Reviewable:start -->
   ---
   This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/16477)
   <!-- Reviewable:end -->

   Changed files

   • components/config/opts.rs
   • components/servo/lib.rs
   • ports/cef/browser.rs
   • ports/servo/main.rs