(In reply to David Spector from comment #7)

but I see no reason to delay further, as the documentation (see citation in my bug report) already states that the s flag exists. It can't be very difficult to implement, in my opinion, from a technical point of view.

I believe that Firefox documentation and software should be kept in synchronization somehow. Perhaps you don't store the documentation in the same git as the software? You might consider coordinating them better, as I've seen other examples of inconsistency.

Please note that MDN web docs is a browser comprehensive wiki documenting the web standards, it's not (just) a "Firefox documentation". And the browser compatibility section of the page you referenced in bug 1579867 clearly states that the "dotAll" feature covered here is not supported by Firefox yet.

As far as I can tell, the relevant code for this touches the files at https://searchfox.org/mozilla-central/search?q=&case=false&regexp=false&path=js%2F**%2FRegExp*, presumably mainly https://searchfox.org/mozilla-central/source/js/src/builtin/RegExp.cpp.

Having said that, I am far from being able to implement this myself.

Sebastian

(In reply to github from comment #10)

In 2018, the dotall flag was added to the specification. Here we are approaching 2020 and Firefox still hasn't implemented it. In what year will Mozilla implement the 2018 specification?

Please stop asking this question in every single regular expression related bug, you already got an answer in bug 1362154.

(In reply to David Spector from comment #13)

(In reply to Tom Schuster [:evilpie] from comment #11)

Please stop asking this question in every single regular expression related bug, you already got an answer in bug 1362154.

1. To whom are you directing this statement? To me? Someone else?
2. What answer are you referring to in #1362154? It is not obvious.

Sending brief, ambiguous, and nasty comments is not helpful to fixing old bugs.

Sorry, this is an answer to another comment that is now hidden. Ian is actively working on updating our copy of the v8 regular expression code that should enable us to run all new regular expression features from ES2018 (bug 1362154 comment 22). You can follow this work in the meta bug 1367105.

This patch adds the boilerplate necessary to support a new regexp flag. Externally visible changes (parsing the flag, the dotAll property on the prototype) are guarded behind ENABLE_NEW_REGEXP.

The actual implementation of dotAll comes for free with the fresh import of irregexp (bug 1367105).

There is one test (tests/non262/RegExp/prototype.js) that needs to be updated when this is turned on to add "dotAll" to the list of expected properties on the RegExp prototype. I will attach that change to my patch that flips ENABLE_NEW_REGEXP to be on by default.

Depends on D66844

Backed out 2 changesets for causing sm failures in RegExp/flags.js

Backout link: https://hg.mozilla.org/integration/autoland/rev/18dd38f7319c182e9467df8f127552187c123670

Push with failures: https://treeherder.mozilla.org/?#/jobs?repo=autoland&group_state=expanded&tochange=18dd38f7319c182e9467df8f127552187c123670&fromchange=e08a5c2df0476fe785f5524f5b0552320b5ca8f2&searchStr=sm&selectedJob=294111415

Failure log: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=294111415&repo=autoland&lineNumber=90362

Also seeing jsreftest failures:
https://treeherder.mozilla.org/#/jobs?repo=autoland&resultStatus=testfailed%2Cbusted%2Cexception%2Crunnable&revision=597e083be9d420dfca296be64bf8ebb01bb4d4ac&selectedJob=294113520

I missed one externally visible change: RegExp.prototype.flags can be called on non-regexp objects, which can have dotAll properties defined. I could have sworn that this code had been through several try builds, but I guess I must have changed RegExpFlagsGetter afterwards. I've uploaded a fix and kicked off a precautionary try build. I will reland once the try build completes.

The jsreftest failures are the same testcase.

https://hg.mozilla.org/mozilla-central/rev/7f288e81410d
https://hg.mozilla.org/mozilla-central/rev/736f7749e6a7

Since this is now closed, shouldn't today's nightly (mar/25/2020) be passing the test at http://kangax.github.io/compat-table/es2016plus/ then?

No, this depends on bug 1367105. There's no more dotAll-specific work to be done, but we won't have dotAll support until that lands.

Fixed by the landing of bug 1634135.

Developer docs:
https://developer.mozilla.org/en-US/docs/Mozilla/Firefox/Releases/78#JavaScript
https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/RegExp
https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/RegExp/dotAll