884896 - Tidy up search/home bits into same package

Status: RESOLVED FIXED

(Firefox for Android Graveyard :: Awesomescreen, defect)

Description

[Lucas Rocha (:lucasr)]

1. Search-related classes should be moved to home package
2. We shoudn't make all classes in home public, unless strictly necessary

Comment 1

[Lucas Rocha (:lucasr)]

Attachment: (1/2) Move all search-related classes to home package

Comment 2

[Lucas Rocha (:lucasr)]

Attachment: (2/2) Only expose home's public API when strictly necessary

Comment 3

[Sriram Ramasubramanian [:sriram]]

(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.

Comment 4

[Lucas Rocha (:lucasr)]

Pushed:
http://hg.mozilla.org/projects/fig/rev/86b02c1b7654
http://hg.mozilla.org/projects/fig/rev/03069010d40f

Comment 5

[Ed Morley [:emorley]]

https://hg.mozilla.org/mozilla-central/rev/86b02c1b7654
https://hg.mozilla.org/mozilla-central/rev/03069010d40f