Bug 1978675 Comment 4 Edit History

Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.

Original comment by
on 
Actually maybe the cause is somewhere else?

```
 0:13.05 GECKO(33340) C:\Users\sasch\AppData\Local\Temp\tmpszs1nawg.mozrunner\user.js:235: prefs parse error: unexpected character
```

And line 235 has this:

```
user_pref("testing.browserTestHarness.timeout", 240.0);
```

Which seemingly is from

```
        if mozinfo.info["os"] == "win" and mozinfo.info["processor"] == "aarch64":
            test_timeout = self.DEFAULT_TIMEOUT * 4
            self.log.info(
                f"Increasing default timeout to {test_timeout} seconds (win aarch64)"
            )
```

where `self.DEFALUT_TIMEOUT` is `60.0`. 

It seems so far it worked only because the test only conditionally read the value before bug 1978111:

```js
    if (prefs.prefHasUserValue("testing.browserTestHarness.timeout")) {
      gTimeoutSeconds = prefs.getIntPref("testing.browserTestHarness.timeout");
    }
```
Revision 1 by
on 
Actually maybe the cause is somewhere else?

```
 0:13.05 GECKO(33340) C:\Users\sasch\AppData\Local\Temp\tmpszs1nawg.mozrunner\user.js:235: prefs parse error: unexpected character
```

And line 235 has this:

```
user_pref("testing.browserTestHarness.timeout", 240.0);
```

Which seemingly is from

```py
        if mozinfo.info["os"] == "win" and mozinfo.info["processor"] == "aarch64":
            test_timeout = self.DEFAULT_TIMEOUT * 4
            self.log.info(
                f"Increasing default timeout to {test_timeout} seconds (win aarch64)"
            )
```

where `self.DEFALUT_TIMEOUT` is `60.0`. (Yes I'm on aarch64 Windows.)

It seems so far it worked only because the test only conditionally read the value before bug 1978111:

```js
    if (prefs.prefHasUserValue("testing.browserTestHarness.timeout")) {
      gTimeoutSeconds = prefs.getIntPref("testing.browserTestHarness.timeout");
    }
```
Revision 2 by
on 
Actually maybe the cause is somewhere else?

```
 0:13.05 GECKO(33340) C:\Users\sasch\AppData\Local\Temp\tmpszs1nawg.mozrunner\user.js:235: prefs parse error: unexpected character
```

And line 235 has this:

```
user_pref("testing.browserTestHarness.timeout", 240.0);
```

Which seemingly is from

```py
        if mozinfo.info["os"] == "win" and mozinfo.info["processor"] == "aarch64":
            test_timeout = self.DEFAULT_TIMEOUT * 4
            self.log.info(
                f"Increasing default timeout to {test_timeout} seconds (win aarch64)"
            )
```

where `self.DEFALUT_TIMEOUT` is `60.0`. (Looks like the pref parser only supports integers, and yes I'm on aarch64 Windows.)

It seems so far it worked only because the test only conditionally read the value before bug 1978111:

```js
    if (prefs.prefHasUserValue("testing.browserTestHarness.timeout")) {
      gTimeoutSeconds = prefs.getIntPref("testing.browserTestHarness.timeout");
    }
```

Back to Bug 1978675 Comment 4