Bug 1262739 - Remove support for splash_screens member in Manifest Processor r=mconley
authorMarcos Caceres <marcos@marcosc.com>
Thu, 07 Apr 2016 14:18:27 -0700
changeset 348803 d335186c3be212f5922c8463be4df2a36f463ae5
parent 348802 7d110f6773c939a869789ccdecff695b5fee9a33
child 348804 7003ce1d246866d09a284379cf005ac06f6f73ad
push id14928
push userbmo:mh+mozilla@glandium.org
push dateFri, 08 Apr 2016 05:15:03 +0000
reviewersmconley
bugs1262739
milestone48.0a1
Bug 1262739 - Remove support for splash_screens member in Manifest Processor r=mconley MozReview-Commit-ID: CnTXh7vQsTW
dom/manifest/ManifestProcessor.jsm
dom/manifest/test/mochitest.ini
dom/manifest/test/test_ManifestProcessor_splash_screens.html
--- a/dom/manifest/ManifestProcessor.jsm
+++ b/dom/manifest/ManifestProcessor.jsm
@@ -82,19 +82,16 @@ this.ManifestProcessor = { // jshint ign
       'lang': processLangMember(),
       'start_url': processStartURLMember(),
       'display': processDisplayMember.call(this),
       'orientation': processOrientationMember.call(this),
       'name': processNameMember(),
       'icons': imgObjProcessor.process(
         rawManifest, manifestURL, 'icons'
       ),
-      'splash_screens': imgObjProcessor.process(
-        rawManifest, manifestURL, 'splash_screens'
-      ),
       'short_name': processShortNameMember(),
       'theme_color': processThemeColorMember(),
       'background_color': processBackgroundColorMember(),
     };
     processedManifest.scope = processScopeMember();
     return processedManifest;
 
     function processNameMember() {
--- a/dom/manifest/test/mochitest.ini
+++ b/dom/manifest/test/mochitest.ini
@@ -11,12 +11,11 @@ support-files =
 [test_ManifestProcessor_background_color.html]
 [test_ManifestProcessor_display.html]
 [test_ManifestProcessor_icons.html]
 [test_ManifestProcessor_JSON.html]
 [test_ManifestProcessor_lang.html]
 [test_ManifestProcessor_name_and_short_name.html]
 [test_ManifestProcessor_orientation.html]
 [test_ManifestProcessor_scope.html]
-[test_ManifestProcessor_splash_screens.html]
 [test_ManifestProcessor_start_url.html]
 [test_ManifestProcessor_theme_color.html]
 [test_ManifestProcessor_warnings.html]
deleted file mode 100644
--- a/dom/manifest/test/test_ManifestProcessor_splash_screens.html
+++ /dev/null
@@ -1,29 +0,0 @@
-<!DOCTYPE HTML>
-<html>
-<!--
-https://bugzilla.mozilla.org/show_bug.cgi?id=1162808
--->
-<head>
-  <meta charset="utf-8">
-  <title>Test for Bug 1162808</title>
-  <script type="application/javascript" src="/tests/SimpleTest/SimpleTest.js"></script>
-  <link rel="stylesheet" type="text/css" href="/tests/SimpleTest/test.css"/>
-  <script src="common.js"></script>
-  <script>
-/**
- * Manifest splash_screens member
- * http://www.w3.org/TR/appmanifest/#splash_screens-member
- **/
-
-'use strict';
-
-typeTests.forEach((type) => {
-  var expected = `Expect non-array splash_screens to be empty: ${typeof type}.`;
-  data.jsonText = JSON.stringify({
-    splash_screens: type
-  });
-  var result = processor.process(data);
-  is(result.splash_screens.length, 0, expected);
-});
-</script>
-</head>