956761 - Marionette interprets keyboard size incorrectly when multiple keyboards enabled

Status: VERIFIED WORKSFORME

(Remote Protocol :: Marionette, defect)

Description

[Zac C (:zac)]

STR
1. Open the keyboard (I used Contacts, +Add contact) and poll its size:
> keyb = m.find_element('css selector', '#keyboards iframe')
> keyb.size
{u'width': 320, u'height': 480}
keyb.is_displayed()
True

2. Go into Settings app
3. Choose Keyboards > Selected Keyboard > Add more keyboards
4. Select espanol (or some other)
5. Return to contacts and poll its size:
> keyb = m.find_element('css selector', '#keyboards iframe')
> keyb.size
{u'width': 0, u'height': 0}
keyb.is_displayed()
False

By watching in app manager it appears the html is torn down and re-generated which may be a factor. 

This must be done on device where OOP keyboard is enabled.


Device Hamachi
Gaia      711d8bf4efde6b1aaaf1f61bd422312c17435810
Gecko     http://hg.mozilla.org/mozilla-central/rev/cf80c0d4f46e
BuildID   20140105040201
Version   29.0a1
ro.build.version.incremental=eng.archermind.20131114.105818
ro.build.date=Thu Nov 14 10:58:33 CST 2013

Comment 1

[Jonathan Griffin (:jgriffin)]

Are we certain the keyboards are still inside the #keyboards iframe when multiple keyboards are enabled?

Comment 2

[Zac C (:zac)]

Yes the keyboards and its HTML content is there and facing the user. I prodded it with the App Manager and it all looked fine.

What I am querying here is the iframe itself.

This just seemed to pop up (sorry) when they turned on the OOP Keyboard.

Comment 3

[Zac C (:zac)]

It's also functional to the user.

Comment 4

[Askeing Yen[:askeing][:fyen]]

Attachment: logcat.log

Buri
Gaia      9f30198a420632359b271de7e680abde92ccbcd6
Gecko     http://hg.mozilla.org/mozilla-central/rev/e7a366c1036c
BuildID   20140107040206
Version   29.0a1
ro.build.version.incremental=eng.archermind.20131114.105818
ro.build.date=Thu Nov 14 10:58:33 CST 2013

I can reproduce.

1. tap text feild to enable keyboard.
> (Pdb) self.marionette.switch_to_frame()
True
> (Pdb) keyb = self.marionette.find_element('css selector', '#keyboards iframe')
> (Pdb) keyb.size
{u'width': 320, u'height': 480}
> (Pdb) keyb.is_displayed()
True
2. add "Espanol" keyboard in settings app.
3. tap text feild to enable keyboard.
> (Pdb) keyb.size
{u'width': 0, u'height': 0}
> (Pdb) keyb.is_displayed()
False

Comment 5

[David Burns :automatedtester]

(In reply to Askeing Yen[:askeing] from comment #4)
> Created attachment 8356941 [details]
> logcat.log
> 
> Buri
> Gaia      9f30198a420632359b271de7e680abde92ccbcd6
> Gecko     http://hg.mozilla.org/mozilla-central/rev/e7a366c1036c
> BuildID   20140107040206
> Version   29.0a1
> ro.build.version.incremental=eng.archermind.20131114.105818
> ro.build.date=Thu Nov 14 10:58:33 CST 2013
> 
> I can reproduce.
> 
> 1. tap text feild to enable keyboard.
> > (Pdb) self.marionette.switch_to_frame()
> True
> > (Pdb) keyb = self.marionette.find_element('css selector', '#keyboards iframe')
> > (Pdb) keyb.size
> {u'width': 320, u'height': 480}
> > (Pdb) keyb.is_displayed()
> True
> 2. add "Espanol" keyboard in settings app.
> 3. tap text feild to enable keyboard.
> > (Pdb) keyb.size
> {u'width': 0, u'height': 0}
> > (Pdb) keyb.is_displayed()
> False


Can you try find the keyboard again instead of reusing the ID to see if that makes a difference

Comment 6

[Bob Silverberg [:bsilverberg]]

The test in question is now passing and I can no longer reproduce this issue. Marking it as fixed.

Comment 7

[Stephen Donner [:stephend] Not actively reading bugmail]

(In reply to Bob Silverberg [:bsilverberg] from comment #6)
> The test in question is now passing and I can no longer reproduce this
> issue. Marking it as fixed.

Since we can't point to a changeset, I think we should probably mark this as INVALID or WORKSFORME, no?

Comment 8

[Bob Silverberg [:bsilverberg]]

Good point Stephen. Updated.

Comment 9

[Stephen Donner [:stephend] Not actively reading bugmail]

(In reply to Bob Silverberg [:bsilverberg] from comment #8)
> Good point Stephen. Updated.

Thanks, Bob.  Didn't mean to be so passive-aggressive; I could've marked it myself, too :-)