Issue 6292 - Make issue reporter compatible with test server

issue-reporter.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";
 
-window.ext = {};

[saroyanm, 2018/03/07 16:16:59]

We are safe to remove this as the "original" common.js is taking care of setting
the ext. ->
https://hg.adblockplus.org/adblockpluschrome/file/tip/ext/common.js#l21

[a.giammarchi, 2018/03/07 17:11:04]

Off topic side note: it'd be awesome to stop assuming globals here and there and
use proper modules instead, even to bring in mocks about web extension, IMO

[saroyanm, 2018/03/08 15:14:27]

That's true.

Another important side note: this Page implementation didn't go through my or
Thomas review.

-
 let reportData = new DOMParser().parseFromString("<report></report>",
                                                  "text/xml");
 
 let pages = {
   typeSelectorPage: [initTypeSelector, leaveTypeSelector],
   commentPage: [initCommentPage, leaveCommentPage],
   sendPage: [initSendPage, leaveSendPage]
 };
 
 document.addEventListener("DOMContentLoaded", () =>
@@ skipping 477 matching lines @@
       progress.value = event.loaded;
     }
   });
   request.send(serializeReportData());
 }
 
 function leaveSendPage()
 {
   closeMe();
 }