Issue 3199 - "Firefox" string is displayed on several error messages

mozharness/abb/transform_locales.py

 # This file is part of Adblock Plus
 # Copyright (C) 2006-2015 Eyeo GmbH
 #
 # Adblock Plus is free software: you can redistribute it and/or modify
 # it under the terms of the GNU General Public License version 3 as
 # published by the Free Software Foundation.
 #
 # Adblock Plus is distributed in the hope that it will be useful,
 # but WITHOUT ANY WARRANTY; without even the implied warranty of
 # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
@@ skipping 41 matching lines @@
               "yahoo",
               "google",
               "wikipedia",
               "amazon"
               ]
 }
 
 _FIREFOX_REPLACE_STR = "Firefox"
 _ABB_REPLACEMENT_STR = "Adblock Browser"
 
-# Some string values that contains Firefox such as 'Firefox Sync' shouldn't be

[Felix Dahlke, 2017/01/02 14:27:16]

Nit: s/contains/contain/

+# Some string values that contain Firefox such as 'Firefox Sync' shouldn't be
 # replaced, so we keep a list of ids that are exceptions
 _ENTITY_EXCEPTIONS = [
     "overlay_no_synced_devices",
     "home_remote_tabs_need_to_sign_in",
     "home_remote_tabs_need_to_finish_migrating",
     "home_remote_tabs_need_to_verify",
     "syncBrand.fullName.label",
     "sync.subtitle.connectlocation2.label",
     "sync.subtitle.failmultiple.label",
     "fxaccount_full_label",
@@ skipping 229 matching lines @@
         locale_path = os.path.join(chrome_path, locale, "locale", locale)
         if os.path.exists(locale_path):
             _transform_locale(locale, locale_path, logger)
         else:
             logger.error("Missing folder for locale '%s' in path: %s" %
                          (locale, locale_path))
 
     for locale in values_locales:
         locale_path = os.path.join(res_path, "values-" + locale)
         _transform_values_locale(locale, locale_path, logger)