Builder linux-rel-css Build #1570

Results:

Build successful

SourceStamp:

BuildSlave:

Reason:

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

Steps and Logfiles:

1. git update ( 4 secs )
   1. - no logs -
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. compile ran build --release --with-debug-assertions ( 22 mins, 25 secs )
   1. - no logs -
6. test ran test-css --release --processes 16 --always-succeed ( 12 mins, 55 secs )
   1. css-errorsummary.log
7. shell__1 './mach filter-intermittents ...' ( 0 secs )
   1. stdio
   2. filtered-css-errorsummary.log
   3. intermittents.log
8. compile__1 ran build-cef --release --with-debug-assertions ( 48 secs )
   1. stdio
9. compile__2 ran build-geckolib --release warnings ( 4 mins, 6 secs )
   1. warnings (5)
10. test__1 ran test-stylo --release warnings ( 2 mins, 6 secs )
    1. warnings (4)
11. shell__2 'bash ./etc/ci/lockfile_changed.sh' ( 0 secs )
    1. stdio
12. shell__3 '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 #30722

   Category	None
   Changed by	bors-servo <lbergstrom+borsohnoyoudont@mozilla.com>
   Changed at	Sun 25 Dec 2016 08:49:53
   Repository	https://github.com/servo/servo
   Project	servo/servo
   Branch	auto
   Revision	fcc3447dfc63571a51abe94ad27a21961d8ede4b

   Comments

   Auto merge of #14722 - iamrohit7:check-secure-schemes, r=jdm
   
   Check for wss schemes in Cookie::appropriate_for_url
   
   * Also adds a new helper, `ServoUrl::is_secure_scheme`.
   * Refactored `CookieStorage::push` and `CookieStorage::remove` to use the new helper.
   <!-- 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 #14702
   
   <!-- Either: -->
   - [X] These changes do not require tests because we can't test the changes yet.
   
   <!-- 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/14722)
   <!-- Reviewable:end -->

   Changed files

   • components/net/cookie.rs
   • components/net/cookie_storage.rs
   • components/url/lib.rs