Merge pull request #31886 from gmarty/Bug-1202740-Enter-search-addres…

…s-replaced-New-home-screen-switching-locales

Bug 1202740 - [new homescreen] 'Enter search or address' is replaced by 'New home screen' when switching locales. r=apastor

apps/system/js/app_chrome.js

@@ -687,7 +687,11 @@
       if (this._fixedTitle) {
         return;
       }
-      this.title.textContent = this.app.name;
+      if (!this.app.isHomescreen && !this.isSearchApp()) {
+        this.title.textContent = this.app.name;
+      } else {
+        this.title.setAttribute('data-l10n-id', 'search-or-enter-address');
+      }
     };
 
   AppChrome.prototype.handleScrollAreaChanged = function(evt) {

apps/system/test/unit/app_chrome_test.js

@@ -520,6 +520,19 @@ suite('system/AppChrome', function() {
       assert.equal(chrome.title.textContent, 'Phone2');
     });
 
+    test('should not update the name of homescreen on change', function() {
+      var app = new AppWindow(cloneConfig(fakeWebSite));
+      app.isHomescreen = true;
+      app.name = 'Home Screen';
+      var chrome = new AppChrome(app);
+      assert.equal(chrome.title.textContent, '');
+
+      app.name = 'Home Screen 2';
+      var evt = new CustomEvent('_namechanged');
+      app.element.dispatchEvent(evt);
+      assert.equal(chrome.title.textContent, '');
+    });
+
     test('localized app is not immediately overridden by titlechange event',
       function() {
       var app = new AppWindow(cloneConfig(fakeWebSite));