1510647 - Lint Warning: Missing @IntDef in Switch

Status: RESOLVED FIXED

(GeckoView :: General, defect, P5)

Description

[Emily Toop (:fluffyemily)]

This check warns if a switch statement does not explicitly include all the values declared by the typedef @IntDef declaration.

Affected classes:

GeckoMediaDrmBridgeV21
SessionTextInput
Util

Comment 1

[pranshukumar1999]

I want to contribute to this bug. Please assign it to me.

Comment 2

[Makoto Kato [:m_kato]]

OK, I assign this to you. Current warning is the following.

Missing @IntDef in Switch
mobile/android/geckoview/src/main/java/org/mozilla/gecko/media/GeckoMediaDrmBridgeV21.java:419: Switch statement on an int with known associated constant missing case MediaDrm.EVENT_SESSION_RECLAIMED

Comment 3

[pranshukumar1999]

Attachment: Bug 1510647 Lint Warning: Missing @IntDef in Switch. r?m_kato

Comment 4

[pranshukumar1999]

I pushed a commit for this issue. https://phabricator.services.mozilla.com/D151653

Comment 5

[pranshukumar1999]

Sir, I pushed a commit for the issue. But then I realized that it had that same linting, so I followed the steps you had told me in the previous bug and pushed the commit at that time only. So I don't think it have that same linting warning as of now.

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/1b8d623937fc
Lint Warning: Missing @IntDef in Switch. r=m_kato,geckoview-reviewers

Comment 7

[Iulian Moraru]

https://hg.mozilla.org/mozilla-central/rev/1b8d623937fc