Closed Bug 1471408 Opened 2 years ago Closed 2 years ago

Multiple top level Java classes inside .java files

Categories

(Firefox for Android :: General, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
Firefox 63
Tracking Status
firefox63 --- fixed

People

(Reporter: u614270, Assigned: gabriel-v)

References

Details

Attachments

(1 file)

There are two Java files in the tree that contain multiple top level classes:

- mobile/android/base/java/org/mozilla/gecko/mdns/MulticastDNSManager.java
- mobile/android/base/java/org/mozilla/gecko/RemotePresentationService.java

All other top level classes are contained in Java files with the same name.
 
Due to a bug in JaCoCo (https://github.com/jacoco/jacoco/issues/701), we will need to have these files split as to have each class in its own file.
Assignee: nobody → tvijiala
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Comment on attachment 8988009 [details]
Bug 1471408 - Move all top level Java classes into their own file.

https://reviewboard.mozilla.org/r/253268/#review260036
Attachment #8988009 - Flags: review?(snorp) → review+
Keywords: checkin-needed
Blocks: 1471646
Blocks: 1471660
Pushed by ntim.bugs@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/f02de3ec24e3
Move all top level Java classes into their own file r=snorp
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/f02de3ec24e3
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 63
Backout by csabou@mozilla.com:
https://hg.mozilla.org/mozilla-central/rev/6041c0307804
Backed out changeset f02de3ec24e3 for causing Android build bustages on VirtualPresentation.java. r=backout
Omitted to exclude VisualPresentatio.java from all the places where RemotePresentationService.java was excluded.

Checked for the same problem with MulticastDNSManager.java.
Flags: needinfo?(tvijiala)
Keywords: checkin-needed
Confirmed with :snorp, he's keeping the r+.

The failing BnoGPS build is now passing: https://treeherder.mozilla.org/#/jobs?repo=try&revision=a74309db308dd794b27b9b550846c52e92098da3
Pushed by cbrindusan@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/0259570da98e
Move all top level Java classes into their own file. r=snorp
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/0259570da98e
Status: REOPENED → RESOLVED
Closed: 2 years ago2 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 63
You need to log in before you can comment on or make changes to this bug.