1243713 - TestDownloadContentCatalog.testCatalogIsBootstrappedIfFileDoesNotExist() fails if MOZ_ANDROID_EXCLUDE_FONTS is not set

Status: RESOLVED FIXED

(Firefox for Android Graveyard :: Testing, defect)

Description

[Sebastian Kaspari (:sebastian; :pocmo)]

TestDownloadContentCatalog.testCatalogIsBootstrappedIfFileDoesNotExist() currently depends on MOZ_ANDROID_EXCLUDE_FONTS being set and fails if it's not.

Comment 1

[Sebastian Kaspari (:sebastian; :pocmo)]

Attachment: MozReview Request: Bug 1243713 - TestDownloadContentCatalog: Skip testCatalogIsBootstrapedIfFileDoesNotExist() if MOZ_ANDROID_EXCLUDE_FONTS is not set. r?nalexander

Review commit: https://reviewboard.mozilla.org/r/32751/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/32751/

Comment 2

[Sebastian Kaspari (:sebastian; :pocmo)]

Comment on attachment 8713145 [details]
MozReview Request: Bug 1243713 - TestDownloadContentCatalog: Skip testCatalogIsBootstrapedIfFileDoesNotExist() if MOZ_ANDROID_EXCLUDE_FONTS is not set. r?nalexander

The reviewboard hg server is down, so we did a quick IRC review:
> 20:08:27 <•nalexander> sebastian: oh, snap.  r+ with new knowledge for reviewer!

Comment 3

[Sebastian Kaspari (:sebastian; :pocmo)]

https://hg.mozilla.org/integration/fx-team/rev/6a96995d887a74d4fb867d6da5cd7382679effc6
Bug 1243713 - Skip testCatalogIsBootstrappedIfFileDoesNotExist() if MOZ_ANDROID_EXCLUDE_FONTS is not set. r=nalexander

Comment 4

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/6a96995d887a