From 56f78cb30c0127111308ff46010c92d5c8b7ac1f Mon Sep 17 00:00:00 2001 From: Michael Comella Date: Tue, 29 Sep 2020 14:12:20 -0700 Subject: [PATCH] No issue: elaborate on Components class doc. This is to clarify a misunderstanding I found in PR review. --- .../main/java/org/mozilla/fenix/components/Components.kt | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/app/src/main/java/org/mozilla/fenix/components/Components.kt b/app/src/main/java/org/mozilla/fenix/components/Components.kt index 83ac7fa94..92b9d97d0 100644 --- a/app/src/main/java/org/mozilla/fenix/components/Components.kt +++ b/app/src/main/java/org/mozilla/fenix/components/Components.kt @@ -31,7 +31,11 @@ import java.util.concurrent.TimeUnit private const val DAY_IN_MINUTES = 24 * 60L /** - * Provides access to all components. + * Provides access to all components. This class is an implementation of the Service Locator + * pattern, which helps us manage the dependencies in our app. + * + * Note: these aren't just "components" from "android-components": they're any "component" that + * can be considered a building block of our app. */ @Mockable class Components(private val context: Context) {