Closed Bug 884896 Opened 6 years ago Closed 6 years ago

Tidy up search/home bits into same package

Categories

(Firefox for Android :: Awesomescreen, defect)

All
Android
defect
Not set

Tracking

()

RESOLVED FIXED
Firefox 26

People

(Reporter: lucasr, Assigned: lucasr)

References

Details

(Whiteboard: fixed-fig)

Attachments

(2 files)

1. Search-related classes should be moved to home package
2. We shoudn't make all classes in home public, unless strictly necessary
Attachment #764830 - Flags: review?(sriram) → review+
(In reply to Lucas Rocha (:lucasr) from comment #0)
> 1. Search-related classes should be moved to home package
> 2. We shoudn't make all classes in home public, unless strictly necessary

Agreed. We are making everything public. We should make a call on what should have public access and what should have package access.
Attachment #764831 - Flags: review?(sriram) → review+
https://hg.mozilla.org/mozilla-central/rev/86b02c1b7654
https://hg.mozilla.org/mozilla-central/rev/03069010d40f
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 26
You need to log in before you can comment on or make changes to this bug.