Closed Bug 1632388 Opened 5 years ago Closed 5 years ago

Devtools properties-db.js.in (template) and properties-db.js (generated file) have the wrong path to their script

Categories

(DevTools :: Shared Components, task)

task

Tracking

(firefox77 fixed)

RESOLVED FIXED
Firefox 77
Tracking Status
firefox77 --- fixed

People

(Reporter: dholbert, Assigned: dholbert)

Details

Attachments

(2 files)

properties-db.js.in and properties-db.js have a code-comment saying:

  • [...] The actual script
  • to generate these files can be found at devtools/shared/css/generate-properties-db.js.

However, no such file exists at that location in the tree.

There's just a missing subdirectory -- the script lives in the generated/ subdirectory within the referenced css directory. We should fix the code comment to match reality.

Bugbug thinks this bug should belong to this component, but please revert this change in case of error.

Component: General → Shared Components

This patch's changes were automatically generated by the following command:
./mach devtools-css-db

Depends on D72097

Attachment #9142623 - Attachment description: Bug 1632388 part 1: Update devtools properties-db.js.in template to correct an obsolete reference to an in-tree directory. r?gregtatum → Bug 1632388 part 1: Update devtools properties-db.js.in template to correct a typo in a code-comment's reference to an in-tree directory. r?gregtatum

Perhaps it's overkill, but for maximal purity/cleanliness, I split this into two parts -- editing the template in "part 1", and regenerating the generated file in "part 2" (so that changes to manually-written files vs. auto-generated files are in separate commits).

Pushed by dholbert@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/a72b3f178ca3 part 1: Update devtools properties-db.js.in template to correct a typo in a code-comment's reference to an in-tree directory. r=gregtatum https://hg.mozilla.org/integration/autoland/rev/cbc332a9c91d part 2: Regenerate the devtools CSS db (to pick up a typo-fix from its template). r=gregtatum
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 77
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: