Bug 1462879 - PerformanceNavigationTiming must be notified correctly - part 2 - getEntryByName, r=valentin

dom/performance/PerformanceMainThread.cpp

@@ -394,16 +394,14 @@ PerformanceMainThread::GetEntriesByName(const nsAString& aName,
     return;
   }
 
-  if (aName.EqualsLiteral("document")) {
-    aRetval.Clear();
+  Performance::GetEntriesByName(aName, aEntryType, aRetval);
 
-    if (mDocEntry) {
-      aRetval.AppendElement(mDocEntry);
-    }
+  // The navigation entry is the first one. If it exists and the name matches,
+  // let put it in front.
+  if (mDocEntry && mDocEntry->GetName().Equals(aName)) {
+    aRetval.InsertElementAt(0, mDocEntry);
     return;
   }
-
-  Performance::GetEntriesByName(aName, aEntryType, aRetval);
 }
 
 } // dom namespace

testing/web-platform/meta/navigation-timing/nav2_test_instance_accessors.html.ini

@@ -1,5 +0,0 @@
-[nav2_test_instance_accessors.html]
-  expected: TIMEOUT
-  [Performance navigation timing entries are accessible through three different accessors.]
-    expected: TIMEOUT
-