Bug 1753083 - [geckodriver] Remove deprecated check for "page load" timeout (Firefox <53). r=webdriver-reviewers,jgraham

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

testing/geckodriver/marionette/src/common.rs

@@ -124,7 +124,6 @@ pub struct Timeouts {
     #[serde(
         default,
         rename = "pageLoad",
-        alias = "page load",
         skip_serializing_if = "Option::is_none"
     )]
     pub page_load: Option<u64>,
@@ -221,10 +220,6 @@ mod tests {
             &data,
             json!({"implicit":1000,"pageLoad":200000,"script":60000}),
         );
-        assert_de(
-            &data,
-            json!({"implicit":1000,"page load":200000,"script":60000}),
-        );
     }
 
     #[test]

testing/geckodriver/src/marionette.rs

@@ -455,13 +455,10 @@ impl MarionetteSession {
                         "Failed to interpret script timeout duration as u64"
                     ),
                 };
-                // Check for the spec-compliant "pageLoad", but also for "page load",
-                // which was sent by Firefox 52 and earlier.
                 let page_load = try_opt!(
                     try_opt!(
                         resp.result
-                            .get("pageLoad")
-                            .or_else(|| resp.result.get("page load")),
+                            .get("pageLoad"),
                         ErrorStatus::UnknownError,
                         "Missing field: pageLoad"
                     )