1416246 - Remove conditional catch consumers in accessible/.

Status: RESOLVED FIXED

(Core :: Disability Access APIs, enhancement)

Description

[Tooru Fujisawa [:arai]]

non-standard conditional catch syntax is going to be removed in bug 1228841.

there are 2 cases for replacement.

case 1: only conditional catch

from:
  try {
    A
  } catch (e if cond) {
    B
  }
to
  try {
    A
  } catch (e) {
    if (!cond) {
      throw e;
    }
    B
  }

case 2: conditional catch and unconditional catch

from:
  try {
    A
  } catch (e if cond) {
    B
  } catch {
    C
  }
to
  try {
    A
  } catch (e) {
    if (cond) {
      B
    } else {
      C
    }
  }

Comment 1

[Tooru Fujisawa [:arai]]

Attachment: Remove conditional catch consumers in accessible/.

Comment 2

[Marco Zehe (:MarcoZ)]

Comment on attachment 8928548 [details] [diff] [review]
Remove conditional catch consumers in accessible/.

Thanks!

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/ee65b4ddd8d3
Remove conditional catch consumers in accessible/. r=MarcoZ

Comment 4

[Tooru Fujisawa [:arai]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/ee65b4ddd8d37d4009f19db080a092d5a0d9e936
Bug 1416246 - Remove conditional catch consumers in accessible/. r=MarcoZ

Comment 5

[Raul Gurzau (:RaulG)]

https://hg.mozilla.org/mozilla-central/rev/ee65b4ddd8d3