1409405 - Implement MozURL::GetOrigin

Status: RESOLVED FIXED

(Core :: Networking, enhancement, P2)

Description

[Andrea Marchesini [:baku]]

This is needed for the URL API

Comment 1

[Valentin Gosu [:valentin] {{ IETF until 25 July - PTO until 3 August }}]

Attachment: Bug 1409405 - Implement MozURL::GetOrigin

Review commit: https://reviewboard.mozilla.org/r/191744/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/191744/

Comment 2

[Honza Bambas (:mayhemer)]

Comment on attachment 8920727 [details]
Bug 1409405 - Implement MozURL::GetOrigin

https://reviewboard.mozilla.org/r/191744/#review197740

::: netwerk/base/rust-url-capi/src/lib.rs:234
(Diff revision 1)
>    *has_fragment = url.fragment().is_some();
>    NS_OK
>  }
>  
>  #[no_mangle]
> +pub extern "C" fn rusturl_get_origin(urlptr: Option<&Url>, cont: &mut nsACString) -> nsresult {

'cont'?  :)

Comment 3

[Valentin Gosu [:valentin] {{ IETF until 25 July - PTO until 3 August }}]

Comment on attachment 8920727 [details]
Bug 1409405 - Implement MozURL::GetOrigin

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/191744/diff/1-2/

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/231f831a969c
Implement MozURL::GetOrigin r=mayhemer

Comment 5

[Attila Craciun [:arny]]

https://hg.mozilla.org/mozilla-central/rev/231f831a969c