Closed Bug 1310863 Opened 5 years ago Closed 5 years ago
Remove DOM/phonenumberutils and related code
58 bytes, text/x-review-board-request
No description provided.
Comment on attachment 8802683 [details] Bug 1310863 - Remove DOM/phonenumberutils and related code. https://reviewboard.mozilla.org/r/87008/#review86086 r=me on the changes, but please note that PhoneNumberUtils is used in dom/contacts, dom/mobilemessage, dom/telephony and dom/wappush. The last three seem to be gonk only, not sure about the first one... Technically you should land this after all of those uses have been removed. But please verify that at least tests pass and whatnot, I won't be too sad if you land this as is right now assuming this won't break tests, and quickly follow up by removing those four directories.
Attachment #8802683 - Flags: review?(ehsan) → review+
And it looks like the contacts tests are failing on try. FWIW in the future when removing JSMs, look at what names are exported through EXPORTED_SYMBOLS, and search for those names in the code base to find those uses.
(In reply to :Ehsan Akhgari (Away Oct 25 - Nov 9) from comment #2) > Technically you should land this after all of those uses have been removed. > But please verify that at least tests pass and whatnot, I won't be too sad > if you land this as is right now assuming this won't break tests, and > quickly follow up by removing those four directories. Indeed. dom/(contacts, wappush, telephony, mobilemessage) are all due to be removed. In order to minimize issues I'll push them all in a single batch as soon as everything else's ready. For the failing tests, I'm not entirely sure whether those are related to this. I'll double-check and re-run everything before landing. But thanks for the info, though.
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/autoland/rev/ad87ed1bbed5 Remove DOM/phonenumberutils and related code. r=ehsan+251051
Adding ddn, just to check the docs for this.
I've checked, and it doesn't look like we documented any of this. It also doesn't look like it is particularly web dev facing. Let me know if you think any of this should be mentioned in the rel notes.
You need to log in before you can comment on or make changes to this bug.