From 66c9b2211c536797f47314e9386cf323b0ab89ed Mon Sep 17 00:00:00 2001
From: Michael Comella <michael.l.comella@gmail.com>
Date: Wed, 17 Feb 2016 15:31:26 -0800
Subject: [PATCH] Bug 1244295 - Add regression warning to ClientID.jsm
 getClientID method. r=gfritzsche

MozReview-Commit-ID: Eci1sG9HPem
---
 toolkit/modules/ClientID.jsm | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/toolkit/modules/ClientID.jsm b/toolkit/modules/ClientID.jsm
index c557e220eda..019b19c7962 100644
--- a/toolkit/modules/ClientID.jsm
+++ b/toolkit/modules/ClientID.jsm
@@ -48,6 +48,11 @@ this.ClientID = Object.freeze({
    * data reporting (FHR & Telemetry). Previously exising FHR client IDs are
    * migrated to this.
    *
+   * WARNING: This functionality is duplicated for Android (see GeckoProfile.getClientId
+   * for more). There are Java tests (TestGeckoProfile) to ensure the functionality is
+   * consistent and Gecko tests to come (bug 1249156). However, THIS IS NOT FOOLPROOF.
+   * Be careful when changing this code and, in particular, the underlying file format.
+   *
    * @return {Promise<string>} The stable client ID.
    */
   getClientID: function() {