From 597135d8990343089c70fa161150d334a589ec29 Mon Sep 17 00:00:00 2001
From: Nicolas Chevobbe <nchevobbe@mozilla.com>
Date: Mon, 18 Jul 2022 07:05:18 +0000
Subject: [PATCH] Bug 1779180 - [devtools] Replace usages of
 InspectorUtils.getRuleLine with InspectorUtils.getRelativeRuleLine.
 r=jdescottes.

For <style> elements, `getRelativeRuleLine` gives us the position of the rule
inside a stylesheet, when `getRuleLine` gives the position of the rule relatively
to the whole HTML source.
The former is more accurate for us when we want to display such line number in
DevTools UI (for example in the StyleEditor media query sidebar).

Test will be added in next patch of this queue.

Differential Revision: https://phabricator.services.mozilla.com/D151596
---
 devtools/server/actors/inspector/css-logic.js       | 2 +-
 devtools/server/actors/media-rule.js                | 2 +-
 devtools/server/actors/utils/stylesheets-manager.js | 2 +-
 3 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/devtools/server/actors/inspector/css-logic.js b/devtools/server/actors/inspector/css-logic.js
index ae7a1f912a30..3b89e212e274 100644
--- a/devtools/server/actors/inspector/css-logic.js
+++ b/devtools/server/actors/inspector/css-logic.js
@@ -959,7 +959,7 @@ function CssRule(cssSheet, domRule, element) {
   if (this._cssSheet) {
     // parse domRule.selectorText on call to this.selectors
     this._selectors = null;
-    this.line = InspectorUtils.getRuleLine(this.domRule);
+    this.line = InspectorUtils.getRelativeRuleLine(this.domRule);
     this.column = InspectorUtils.getRuleColumn(this.domRule);
     this.source = this._cssSheet.shortSource + ":" + this.line;
     if (this.mediaText) {
diff --git a/devtools/server/actors/media-rule.js b/devtools/server/actors/media-rule.js
index 3c9d662ef38b..43b54d3e7ace 100644
--- a/devtools/server/actors/media-rule.js
+++ b/devtools/server/actors/media-rule.js
@@ -35,7 +35,7 @@ var MediaRuleActor = protocol.ActorClassWithSpec(mediaRuleSpec, {
 
     this._matchesChange = this._matchesChange.bind(this);
 
-    this.line = InspectorUtils.getRuleLine(mediaRule);
+    this.line = InspectorUtils.getRelativeRuleLine(mediaRule);
     this.column = InspectorUtils.getRuleColumn(mediaRule);
 
     try {
diff --git a/devtools/server/actors/utils/stylesheets-manager.js b/devtools/server/actors/utils/stylesheets-manager.js
index 2433ee592293..4697087afe9b 100644
--- a/devtools/server/actors/utils/stylesheets-manager.js
+++ b/devtools/server/actors/utils/stylesheets-manager.js
@@ -603,7 +603,7 @@ class StyleSheetsManager extends EventEmitter {
         mediaText: rule.media.mediaText,
         conditionText: rule.conditionText,
         matches,
-        line: InspectorUtils.getRuleLine(rule),
+        line: InspectorUtils.getRelativeRuleLine(rule),
         column: InspectorUtils.getRuleColumn(rule),
       };
     });