#13983 closed bug (fixed)
Switch to //# for sourcemap directives
| Reported by: | dmethvin | Owned by: | m_gol |
|---|---|---|---|
| Priority: | high | Milestone: | 1.11/2.1 |
| Component: | build | Version: | 1.10.1 |
| Keywords: | Cc: | ||
| Blocked by: | Blocking: |
Description
The sourcemap r3 proposal has been changed at our request to avoid the issues from #13274. Since nobody recognizes //# at the moment we should give them a few months to update.
Change History (15)
comment:1 Changed 10 years ago by
| Component: | unfiled → build |
|---|---|
| Milestone: | None → 1.11/2.1 |
| Priority: | undecided → low |
| Status: | new → open |
comment:2 Changed 10 years ago by
comment:4 Changed 10 years ago by
| Milestone: | 1.11/2.1 → 1.10.2/2.0.3 |
|---|---|
| Priority: | low → blocker |
Well, thanks Chrome for barfing out a warning. We'll need to change this immediately to prevent a barrage of duplicate tickets.
comment:6 Changed 10 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | open → closed |
Fix #13983. Switch to # for sourcemaps.
Normally we'd avoid doing this on a patch release, but Chrome Canary is spewing deprecation warnings already and that's generating bug tickets.
Changeset: d53ddc90c1f119fb9148a553443ef3fbc3f3cc99
comment:7 Changed 10 years ago by
Revert "Fix #13983. Switch to # for sourcemaps."
Per the discussion here: https://code.google.com/p/chromium/issues/detail?id=256636
Thanks to the Chrome team for the quick response!
This reverts commit d53ddc90c1f119fb9148a553443ef3fbc3f3cc99.
Changeset: 6e630fbbb19306e1914712502a2b22872e94ddad
comment:8 Changed 10 years ago by
| Resolution: | fixed |
|---|---|
| Status: | closed → reopened |
comment:9 Changed 10 years ago by
| Milestone: | 1.10.2/2.0.3 → 1.next |
|---|---|
| Priority: | blocker → high |
| Status: | reopened → open |
comment:10 Changed 10 years ago by
| Milestone: | 1.next → 1.11/2.1 |
|---|
We can do it now, Chromium has implemented the change in v29:
https://src.chromium.org/viewvc/blink?revision=151755&view=revision[[BR]]
http://omahaproxy.appspot.com/ (v29 has a higher revision number)
and both Firefox & WebKit implemented //@ in the same version as the change to //#.
comment:11 Changed 10 years ago by
(to be more precise, Firefox has implemented support for both pragmas in v24, Safari will have it starting from 6.1/7.0)
comment:12 Changed 10 years ago by
| Owner: | set to m_gol |
|---|---|
| Status: | open → assigned |
comment:13 Changed 10 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | assigned → closed |
Fix #13983. Switch to # for sourcemap directives.
Changeset: dfaee326e6605dc6d848c4c5022e39069d2cee65
see also http://updates.html5rocks.com/2013/06/sourceMappingURL-and-sourceURL-syntax-changed