776142 - NullPointerException from missing registerGeckoEventListener method in tests

Status: RESOLVED FIXED

(Firefox for Android Graveyard :: General, defect)

Description

[Kartikaya Gupta (email:[e-mail])]

See https://tbpl.mozilla.org/php/getParsedLog.php?id=13720352&full=1&branch=mozilla-inbound

The top of the stack has:

java.lang.NoSuchMethodException
	at java.lang.Class.getDeclaredMethods(Native Method)
	at java.lang.ClassCache.getDeclaredPublicMethods(ClassCache.java:166)
	at java.lang.ClassCache.getDeclaredMethods(ClassCache.java:179)
	at java.lang.ClassCache.findAllMethods(ClassCache.java:249)
	at java.lang.ClassCache.getFullListOfMethods(ClassCache.java:223)
	at java.lang.ClassCache.getAllPublicMethods(ClassCache.java:204)
	at java.lang.Class.getMethod(Class.java:984)
	at org.mozilla.fennec.FennecNativeDriver.<init>(FennecNativeDriver.java:97)

and line 97 of FennecNativeDriver is this:

mRegisterGEL = mGas.getMethod("registerGeckoEventListener", parameters);

For some reason the method isn't being found even though it exists.

Comment 1

[Kartikaya Gupta (email:[e-mail])]

Happens in talos too:

https://tbpl.mozilla.org/php/getParsedLog.php?id=13721853&tree=Mozilla-Inbound

Comment 2

[Ryan VanderMeulen [:RyanVM]]

https://tbpl.mozilla.org/php/getParsedLog.php?id=13723620&tree=Mozilla-Inbound

Comment 3

[Ryan VanderMeulen [:RyanVM]]

https://tbpl.mozilla.org/php/getParsedLog.php?id=13742356&tree=Mozilla-Inbound

Comment 4

[Ryan VanderMeulen [:RyanVM]]

https://tbpl.mozilla.org/php/getParsedLog.php?id=13743298&tree=Firefox

Comment 5

[Ryan VanderMeulen [:RyanVM]]

https://tbpl.mozilla.org/php/getParsedLog.php?id=13744039&tree=Mozilla-Inbound

Comment 6

[Ryan VanderMeulen [:RyanVM]]

https://tbpl.mozilla.org/php/getParsedLog.php?id=13744225&tree=Mozilla-Inbound

Comment 7

[Ryan VanderMeulen [:RyanVM]]

https://tbpl.mozilla.org/php/getParsedLog.php?id=13744368&tree=Mozilla-Inbound

Comment 8

[Justin Wood (:Callek)]

This turned a whole bunch of (talos) jobs on a recent push red... as well as robocop units orange.

We either need to backout its cause asap or fix it, imo.

Comment 9

[Ryan VanderMeulen [:RyanVM]]

https://tbpl.mozilla.org/php/getParsedLog.php?id=13744648&tree=Mozilla-Inbound

Comment 10

[Ryan VanderMeulen [:RyanVM]]

(Note that I'm starring most of the talos red with a;r instead of this bug)

Comment 11

[Ryan VanderMeulen [:RyanVM]]

https://tbpl.mozilla.org/php/getParsedLog.php?id=13744827&tree=Firefox
https://tbpl.mozilla.org/php/getParsedLog.php?id=13744832&tree=Firefox

Comment 12

[Ryan VanderMeulen [:RyanVM]]

https://tbpl.mozilla.org/php/getParsedLog.php?id=13745743&tree=Firefox
https://tbpl.mozilla.org/php/getParsedLog.php?id=13745374&tree=Firefox
https://tbpl.mozilla.org/php/getParsedLog.php?id=13745087&tree=Firefox
https://tbpl.mozilla.org/php/getParsedLog.php?id=13745579&tree=Firefox
https://tbpl.mozilla.org/php/getParsedLog.php?id=13745869&tree=Firefox
https://tbpl.mozilla.org/php/getParsedLog.php?id=13745581&tree=Firefox
https://tbpl.mozilla.org/php/getParsedLog.php?id=13745601&tree=Firefox

Comment 13

[Bill Gianopoulos [:WG9s]]

Could this be rlalted to Bug 765766?

Comment 14

[Bill Gianopoulos [:WG9s]]

(In reply to Bill Gianopoulos [:WG9s] from comment #13)
> Could this be rlalted to Bug 765766?
                ^^^^^^^
                related

Comment 15

[:Ms2ger (he/him; ⌚ UTC+1/+2)]

https://hg.mozilla.org/integration/mozilla-inbound/rev/6cfd941c49a6

Comment 16

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/6cfd941c49a6

Comment 17

[Bill Gianopoulos [:WG9s]]

Hasn't this been fixed by the new patch on bug 687267?

Comment 18

[Kartikaya Gupta (email:[e-mail])]

Well it was fixed by the backout cset in comment 15, and the new patch on bug 687267 didn't reintroduce it.

Comment 19

[Kartikaya Gupta (email:[e-mail])]

Why does this affect 16 and 15? The patch that introduced it was never uplifted, AFAIK.

Comment 20

[Scoobidiver (away)]

(In reply to Kartikaya Gupta (:kats) from comment #19)
> Why does this affect 16 and 15? The patch that introduced it was never
> uplifted, AFAIK.
I though it was introduced by bug 687267.

Comment 21

[Kartikaya Gupta (email:[e-mail])]

It was introduced by cset 8571114112b2, which was backed out before being uplifted. bug 687267 was fixed with a different patch that didn't have this problem.

Also I checked the tbpl pages for aurora and beta, and the robocop tests are passing fine, so this bug definitely doesn't affect those branches.