1013814 - Create DOMUtils.getRelativeRuleLine(nsIDOMCSSStyleRule *aRule)

Status: RESOLVED FIXED

(Core :: CSS Parsing and Computation, defect)

Description

[Joe Walker [:jwalker]]

DOMUtils.getRelativeRuleLine is just like its cousin DOMUtils.getRuleLine except when the style rule is in a <style> element, in which case it returns a line number relative to the top of the <style> element rather than to the containing HTML.

Alternatively maybe DOMUtils.getStyleElementLine could get the line number of the <style> element in the HTML document.

Comment 1

[David Baron :dbaron: (⌚️UTC-5, no longer working on Mozilla)]

The latter sounds easier.

Comment 3

[Tom Tromey :tromey]

Needed for as-authored.

Comment 4

[Tom Tromey :tromey]

Attachment: add inIDOMUtils.getRelativeRuleLine

Comment 5

[Tom Tromey :tromey]

Comment on attachment 8646996 [details] [diff] [review]
add inIDOMUtils.getRelativeRuleLine

I think I need reviews for both the layout and devtools bits.
I'm not sure if the (minor) dom changes need separate review.

Comment 6

[Cameron McCormack (:heycam)]

Comment on attachment 8646996 [details] [diff] [review]
add inIDOMUtils.getRelativeRuleLine

Review of attachment 8646996 [details] [diff] [review]:
-----------------------------------------------------------------

r=me on the layout/ (and dom/) changes.

::: dom/base/nsIStyleSheetLinkingElement.h
@@ +103,5 @@
> +   *
> +   * @return the line number of this element; or 1 if no line number
> +   *         was set
> +   */
> +  virtual uint32_t GetLineNumber() = 0;

Please rev the uuid for nsIStyleSheetLinkingElement.

::: layout/inspector/tests/test_getRelativeRuleLine.html
@@ +30,5 @@
> +                           .getService(SpecialPowers.Ci.inIDOMUtils);
> +
> +  let tests = [
> +    { sheetNo: 0, ruleNo: 0, lineNo: 1, columnNo: 1 },
> +    { sheetNo: 1, ruleNo: 0, lineNo: 2, columnNo: 15 },

Not to do with this bug, but is it expected that column 15 is reported here (after the "@supports") instead of 5 (before it)?

Comment 7

[Patrick Brosset <:pbro>]

Comment on attachment 8646996 [details] [diff] [review]
add inIDOMUtils.getRelativeRuleLine

Review of attachment 8646996 [details] [diff] [review]:
-----------------------------------------------------------------

Looking good. Thanks Tom for cleaning that part of the devtools code up.

Comment 8

[Tom Tromey :tromey]

(In reply to Cameron McCormack (:heycam) from comment #6)

> Not to do with this bug, but is it expected that column 15 is reported here
> (after the "@supports") instead of 5 (before it)?

Yeah, it is expected in the sense that the parser gives the rule the location
of the token following the @suppports keyword:

https://dxr.mozilla.org/mozilla-central/source/layout/style/nsCSSParser.cpp#4135

This seems to be done consistently (I spot checked a couple others) for
@-keyword rules.

Whether this is the most useful location, I am not sure.  It's possible that
the devtools would prefer the location of the @-keyword itself.  I made a note in
my project to-do list to check on this.  I tend to think it isn't an issue; but if
it is I will file a separate bug for it.

Comment 9

[Tom Tromey :tromey]

Attachment: add inIDOMUtils.getRelativeRuleLine

Address review comment.

Comment 10

[Tom Tromey :tromey]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=acbbad063da5

Comment 11

[Tom Tromey :tromey]

(In reply to Tom Tromey :tromey from comment #8)

> Whether this is the most useful location, I am not sure.  It's possible that
> the devtools would prefer the location of the @-keyword itself.  I made a
> note in
> my project to-do list to check on this.  I tend to think it isn't an issue;
> but if
> it is I will file a separate bug for it.

I looked at this, and I think that at least for the time being, the current
locations are ok.  The rule view exposes ordinary rules and the contents of
@supports and @keyframes; and there's no way in the rule view to edit the
latter two directly (just their "ordinary rule" contents).

Comment 12

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/integration/mozilla-inbound/rev/880092cace77

Comment 13

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/880092cace77