diff options
author | bors-servo <lbergstrom+bors@mozilla.com> | 2017-06-11 21:16:06 -0700 |
---|---|---|
committer | GitHub <noreply@github.com> | 2017-06-11 21:16:06 -0700 |
commit | 87140641a4f8636b431db41777d01302f8f6ad3d (patch) | |
tree | e64190add5ad3db8c01fa5d24445f386ad5c8eb7 /python | |
parent | 03074f4b0e9908d80558efe519301f39aadc436e (diff) | |
parent | 708c561ae698788c39d7d0ef7d796f349d7f218b (diff) | |
download | servo-87140641a4f8636b431db41777d01302f8f6ad3d.tar.gz servo-87140641a4f8636b431db41777d01302f8f6ad3d.zip |
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 -->
Diffstat (limited to 'python')
0 files changed, 0 insertions, 0 deletions