Issue 5028 - Use browser namespace

ext/common.js

 /*
  * This file is part of Adblock Plus <https://adblockplus.org/>,
  * Copyright (C) 2006-present 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
  * GNU General Public License for more details.
  *
  * You should have received a copy of the GNU General Public License
  * along with Adblock Plus.  If not, see <http://www.gnu.org/licenses/>.
  */
 
 "use strict";
 
 (function()
 {
-  if (typeof chrome == "undefined")
-    window.chrome = {};
-
   if (typeof ext == "undefined")
     window.ext = {};
 
   function Page(source)
   {
     this._source = source;
   }
   Page.prototype =
   {
     sendMessage(message)
@@ skipping 119 matching lines @@
       return;
 
     let rawCatalog = JSON.parse(xhr.responseText);
     for (let msgId in rawCatalog)
     {
       if (!(msgId in catalog))
         catalog[msgId] = parseMessage(rawCatalog[msgId]);
     }
   };
 
-  chrome.i18n = {
+  browser.i18n = {

[Sebastian Noack, 2017/10/10 23:23:08]

I think this code should be moved to polyfill.js now.

[Manish Jethani, 2017/10/11 11:29:06]

Done.

     getUILanguage()
     {
       return locales[0].replace(/_/g, "-");
     },
     getMessage(msgId, substitutions)
     {
       while (true)
       {
         let message = catalog[msgId];
         if (message)
@@ skipping 13 matching lines @@
         if (locales.length == 0)
           return "";
 
         let locale = locales.shift();
         readCatalog(locale, "common.json");
         readCatalog(locale, catalogFile);
       }
     }
   };
 }());