Bug 1468771 - fix reftest manifest parsing to support test cases with ? in the name. r=ahal

Reftest manifest parsing fails to recognize individual tests with ? in the name.  We missed this as in aou

Tes Plan:

Differential Revision: https://phabricator.services.mozilla.com/D1667

layout/tools/reftest/manifest.jsm

@@ -667,9 +667,9 @@ function AddTestItem(aTest, aFilter) {
     var globalFilter = aFilter[0];
     var manifestFilter = aFilter[1];
     var invertManifest = aFilter[2];
-    if ((globalFilter && !globalFilter.test(url1.spec)) ||
+    if (globalFilter && !globalFilter.test(url1.spec))
-        (manifestFilter &&
+        return;
-         !(invertManifest ^ manifestFilter.test(url1.spec))))
+    if (manifestFilter && !(invertManifest ^ manifestFilter.test(url1.spec)))
         return;
     if (g.focusFilterMode == FOCUS_FILTER_NEEDS_FOCUS_TESTS &&
         !aTest.needsFocus)

layout/tools/reftest/runreftest.py

@@ -197,7 +197,7 @@ class ReftestResolver(object):
                     break
             if found:
                 rv = [(os.path.join(dirname, default_manifest),
-                       r".*(?:/|\\)%s(?:[#?].*)?$" % pathname)]
+                       r".*(?:/|\\)%s(?:[#?].*)?$" % pathname.replace('?', '\?'))]
 
         return rv