946547 - Change TestConstants package to org.mozilla.gecko.tests

Status: RESOLVED FIXED

(Firefox for Android Graveyard :: Testing, defect)

Description

[Michael Comella (:mcomella) [NI reported issues only: ex-Mozilla]]

For some reason, it is org.mozilla.gecko, despite being in the mobile/android/base/tests/ dir.

Comment 1

[Michael Comella (:mcomella) [NI reported issues only: ex-Mozilla]]

Attachment: Change TestConstants package to org.mozilla.gecko.tests.

Comment 2

[Brian Nicholson (:bnicholson)]

Comment on attachment 8343918 [details] [diff] [review]
Change TestConstants package to org.mozilla.gecko.tests.

Review of attachment 8343918 [details] [diff] [review]:
-----------------------------------------------------------------

Package change looks fine, but I think all of the imports you have here are redundant since TestConstants is in the same package as the files importing it.

Comment 3

[Michael Comella (:mcomella) [NI reported issues only: ex-Mozilla]]

Attachment: Change TestConstants package to org.mozilla.gecko.tests. v2

(In reply to Brian Nicholson (:bnicholson) from comment #2)
> Package change looks fine, but I think all of the imports you have here are
> redundant since TestConstants is in the same package as the files importing
> it.

You are correct.

Comment 4

[Michael Comella (:mcomella) [NI reported issues only: ex-Mozilla]]

remote:   https://hg.mozilla.org/integration/fx-team/rev/445cafa077dc

Comment 5

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/445cafa077dc