1449097 - Expose function from Servo side to replace GetStatesForPseudoClass in InspectorUtils

Status: RESOLVED FIXED

(Core :: CSS Parsing and Computation, enhancement)

Comment 1

[Xidorn Quan [:xidorn] UTC+10]

Attachment: Bug 1449097 - Use Servo to implement GetStatesForPseudoClass function in InspectorUtils.

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

Comment 2

[Emilio Cobos Álvarez (:emilio)]

Comment on attachment 8962623 [details]
Bug 1449097 - Use Servo to implement GetStatesForPseudoClass function in InspectorUtils.

https://reviewboard.mozilla.org/r/231480/#review237036

::: servo/ports/geckolib/glue.rs:5125
(Diff revision 1)
> +pub unsafe extern "C" fn Servo_PseudoClass_GetStates(name: *const nsACString) -> u64 {
> +    let name = name.as_ref().unwrap().as_str_unchecked();
> +    match NonTSPseudoClass::parse_non_functional(name) {
> +        None => 0,
> +        // Ignore :any-link since it contains both visited and unvisited state.
> +        Some(NonTSPseudoClass::AnyLink) => 0,

nit: Maybe merge the branches?

```
    None |
    Some(NonTSPseudoClass::AnyLink) => 0,
```

(Not sure it's necessarily better, your call).

Comment 3

[Xidorn Quan [:xidorn] UTC+10]

Comment on attachment 8962623 [details]
Bug 1449097 - Use Servo to implement GetStatesForPseudoClass function in InspectorUtils.

https://reviewboard.mozilla.org/r/231480/#review237036

> nit: Maybe merge the branches?
> 
> ```
>     None |
>     Some(NonTSPseudoClass::AnyLink) => 0,
> ```
> 
> (Not sure it's necessarily better, your call).

I'd just keep it as-is. Don't really think it matters much.

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/44ffa321cfdf
Use Servo to implement GetStatesForPseudoClass function in InspectorUtils. r=emilio

Comment 5

[Cosmin Sabou [:CosminS]]

https://hg.mozilla.org/mozilla-central/rev/44ffa321cfdf