1416251 - Remove conditional catch consumers in mobile/android/.

Status: RESOLVED FIXED

(Firefox for Android Graveyard :: General, enhancement, P2)

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

[Nevin Chen(Not active on Bugzilla)]

I t

Comment 2

[Tooru Fujisawa [:arai]]

Attachment: Remove conditional catch consumers in mobile/android/.

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/76bdc9a9451d
Remove conditional catch consumers in mobile/android/. r=nechen

Comment 4

[Tooru Fujisawa [:arai]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/76bdc9a9451de58725f2f31553d4b22bae3694d1
Bug 1416251 - Remove conditional catch consumers in mobile/android/. r=nechen

Comment 5

[Bogdan Tara[:bogdan_tara | bogdant]]

https://hg.mozilla.org/mozilla-central/rev/76bdc9a9451d