Bug 1581537 Comment 7 Edit History

Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.

The test failure is because

https://searchfox.org/mozilla-central/rev/35873cfc312a6285f54aa5e4ec2d4ab911157522/browser/base/content/test/static/browser_misused_characters_in_strings.js#97-101

```
  {
    file: "dom.properties",
    key: "PatternAttributeCompileFailure",
    type: "single-quote",
  },
```

as a string that would ordinarily be required to use curly quotes. The test removes entries from the list of exceptions when it encounters them, and because we now package this file twice, the second time it encounters the entry, there is no exception in the list anymore so it fails.

The simplest fix would be to duplicate the exception entry I cited, probably with a comment along the lines of "dom.properties is packaged twice so we need to have 2 exceptions for this string".
The test failure is because

https://searchfox.org/mozilla-central/rev/35873cfc312a6285f54aa5e4ec2d4ab911157522/browser/base/content/test/static/browser_misused_characters_in_strings.js#97-101

```
  {
    file: "dom.properties",
    key: "PatternAttributeCompileFailure",
    type: "single-quote",
  },
```

dom.properties has a string that would ordinarily be required to use curly quotes. The test removes entries from the list of exceptions when it encounters them, and because we now package this file twice, the second time it encounters the entry, there is no exception in the list anymore so it fails.

The simplest fix would be to duplicate the exception entry I cited, probably with a comment along the lines of "dom.properties is packaged twice so we need to have 2 exceptions for this string".

Back to Bug 1581537 Comment 7