The Android icon looks weird

[mgol]

Open http://swarm.jquery.org/project/jqueryperiodic. The Android icon is as follows:

instead of just:

The actual icon is actually two ones - the first is:

and, set as a background for the ::after pseudo-element is the Android icon. This is set as

testswarm/css/testswarm.css

Lines 476 to 478 in ba50f43

	.swarm-os-android::after {
	background-image: url(../img/os-android.png);
	}

It seems to me that the browser is not recognized so the image is just "an unknown browser under Android"; is that true? For Android <4.4 the Android browser should just have its own icon. For >=4.4 the Chrome icon could appear as that's what we're testing here. But a regular robot icon would still be fine, I guess since BrowserStack presents it just as an Android 4.4/5.0 Browser and not as Chrome.

[Krinkle]

This is because the swarm.jquery.org configuration uses a user-agent match that only looks for the OS family, not any particular browser.

I've added icons for the Android and Chrome Mobile browsers. They will be used on the home page once we update our configuration to specify the "Android" and "Chrome Mobile" browserFamily accordingly.

[Krinkle]

Hm.. it may not be that simple. It's not browser "Android" for Android 2.3 and browser "Chrome Mobile" for Android 4 and higher.

Using http://swarm.jquery.org/job/929 as example, we see the following user agents / ua-parser data:

• Android 2.3 – http://swarm.jquery.org/result/472803 – http://swarm.jquery.org/client/65156 – "Android 2.3.3 / Android 2.3.3"
• Android 4.0 – http://swarm.jquery.org/result/472805 – http://swarm.jquery.org/client/65189 – "Android 4.0.3 / Android 4.0.3"
• Android 4.1 – http://swarm.jquery.org/result/472801 – http://swarm.jquery.org/client/65188 – "Android 4.1.2 / Android 4.1.2"
• Android 4.2 – http://swarm.jquery.org/result/472787 – http://swarm.jquery.org/client/65184 – "Android 4.2.2 / Android 4.2.2"
• Android 4.3 – http://swarm.jquery.org/result/472721 – http://swarm.jquery.org/client/65155 – "Android 4.3 / Android 4.3"
• Android 4.4 – http://swarm.jquery.org/result/472758 – http://swarm.jquery.org/client/65183 – "Chrome 30.0.0 / Android 4.4.2" (Not "Chrome Mobile", but "Chrome")
• Android 5.0 – http://swarm.jquery.org/result/472711 – http://swarm.jquery.org/client/65170 – "Chrome 37.0.0 / Android 5.0" (Not "Chrome Mobile", but "Chrome")

[mgol]

The Chrome icon should be fine, I guess, if only we see info about the Android version in TestSwarm as well.

BTW, this should only concern Android 4.4 or newer, Android 4.0-4.3 uses the default Android Browser that was removed in 4.4.

I have doubts if it makes sense to test Android versions for Android >= 5.0 since they just use Chrome Mobile and that matters, not the Android version used. Unfortunately, that's what we currently get from BrowserStack. Android 4.4 is an odd one since it has Chrome as a default browser but it's WebView has a frozen Chromium version. In Android 5.0+ both auto-update.

[Krinkle]

There's another problem I forgot about. The BrowserStack API doesn't provide browser property for mobile devices. Adding a browserFamily to these would mean the tests no longer get run.

I've reverted the jquery/infrastructure patch that added it for Android 2.3 because it was building up a backlog of unrun runs.

[mgol]

The Browserstack API has a browser field for mobile browsers; for the Android one it's "Android Browser". OTH, it returns "Android Browser" even for Android >= 4.4 which seems incorrect.

[Krinkle]

Aha, so we can use it but will need a mapping in testswarm-browserstack since ua-parser calls it browserFamily="Android" and BrowserStack uses browser="Android Browser".

[Krinkle]

So... adding that mapping didn't work. testswarm-browserstack is still unable to map the browser.

This is because testswarm-browserstack explicitly ignores the browser.browser property from BrowserStack (see testswarm-browserstack.js#L323-L330) due to an inconsistency in the API between /workers and /browsers.

/browsers lists browser: "Android Browser", but /worker doesn't accept that property.

[Krinkle]

This is now fixed.

• jquery/testswarm: The UA-parser refers to the Android browser as "Android", but BrowserStack refers to it as "Android Browser". Fixed in jquery/testswarm-browserstack@6be074a.
• browserstack/api: BrowserStack API v3 had a bug where the /browsers endpoint exposed a browser name, but /workers did not support spawning a mobile browser by browser name, only by OS and device. Fixed upstream in BrowserStack v4.
• testswarm-browserstack: Updated to use API v4 by @mgol in jquery/testswarm-browserstack@846a55e.
• testswarm-browserstack: Removed hack for absence of .browser in v3 worker objects so that we can start using the v4 model, in jquery/testswarm-browserstack@31044fb.
• jquery/infrastructure config: Added browserFamily: Android to the android user agent configs for our swarm, in https://github.com/jquery/infrastructure/commit/553ac20746c09e7e586b34ce22f72154fe9b5f24.

[Krinkle]

Per jquery/testswarm-browserstack@f246bb8 and https://github.com/jquery/infrastructure/commit/953ccd7b7e08db1fd5db5aa8f68c519be4737f4d, Chrome Mobile on Android 4.4+ now also works properly.