823645 - [B2G][Keyboard] Keyboard does not show for advanced HTML editors (e.g. Gmail compose)

Status: VERIFIED WORKSFORME

(Firefox OS Graveyard :: General, defect, P3)

Description

[Angela Hubenya]

Description:
When user attempts to compose/reply to an email from Gmail navigated through the Browser, virtual keyboard does not show up.

Repro Steps:
1. Launch Internet Browser from device
2. Navigate to www.gmail.com from URL link
3. Enter a valid username & password
4. Select "Compose" OR open existing email and tap "reply"
5. Select inside the text body area

Actual Results: Virtual keyboard does not show up, not giving user the option to compose email.


Expected Results:
User must be presented with a virtual keyboard.


Notes:
Reproduced 6/6 times on 2 accounts.
Build ID: 20121217070202

Comment 1

[Angela Hubenya]

defect is still reproducing with Unagi, build ID 20121231070201

Comment 2

[Joe Cheng [:jcheng] (please needinfo)]

Triage: QAWANTED to confirm this bug

Comment 3

[Walter Chen[:ypwalter][:wachen]]

Hi,
I marked this as "bb?", I confirmed this in recent build (20130101)

Comment 4

[Naoki Hirata :nhirata (please use needinfo instead of cc)]

Pretty sure this is bug 823832? bug 823835 is a similar regression.  I looked at 20130102 build and cannot reproduce this issue.  Can you double check which build you are on please, Walter?

Comment 5

[Angela Hubenya]

Attachment: keyboard does not appear

Comment 6

[Angela Hubenya]

this defect only reproduces when accessing Gmail through the browser, NOT Gmail app on Unagi. I just reproduced this on build 20130102070202

Comment 7

[Naoki Hirata :nhirata (please use needinfo instead of cc)]

Turns out I was the one that made the mistake.  I was looking at the email app and not the browser.  Too much on my mind I guess.

I can reproduce on the 20130102 build using the browser app.

I can also reproduce the same thing in a smaller test case : 
http://people.mozilla.com/~nhirata/html_tp/bug573447.htm
try tapping in the middle box.

It's an iframe text area.

Comment 8

[Naoki Hirata :nhirata (please use needinfo instead of cc)]

I thought I had written a bug on this before, but I can't seem to find it.

Comment 9

[Walter Chen[:ypwalter][:wachen]]

Joe,
I again confirmed this bug.


All,
It's not related with Bug 823832 and Bug 823835.
It's related with https://bugzilla.mozilla.org/show_bug.cgi?id=573447
And, just like Naoki says, it's a iframe for advanced editor.

Comment 10

[Joe Cheng [:jcheng] (please needinfo)]

Triage: BB+, P3, C4 - severe usability for not able to launch keyboard

Comment 11

[Vivien Nicolas (:vingtetun) (:21) - (NOT reading bugmails, needinfo? please)]

Not directly a Gaia issue. The failure lives in b2g/chrome/content/forms.js.

Tim do you  have time to look at it?

Comment 12

[Tim Taubert [:ttaubert] (inactive)]

We're getting a focus event for the contentWindow of an iframe. Gmail is doing some JavaScript magic to have some kind of HTML editor. I'm not sure how we can separate that from other windows/frames that we don't want to have a keyboard for.

Comment 13

[Tim Taubert [:ttaubert] (inactive)]

Attachment: WIP v1

Comment 14

[Tim Taubert [:ttaubert] (inactive)]

Attachment: patch v2

Vivien, I think you might be a good fit to review this. If you don't have time, feel free to pass it on!

Comment 15

[Tim Taubert [:ttaubert] (inactive)]

BTW, the logic is all shamelessly stolen from Firefox for Android:

http://mxr.mozilla.org/mozilla-central/source/mobile/xul/chrome/content/forms.js#500

Comment 16

[Vivien Nicolas (:vingtetun) (:21) - (NOT reading bugmails, needinfo? please)]

Comment on attachment 697489 [details] [diff] [review]
patch v2

Review of attachment 697489 [details] [diff] [review]:
-----------------------------------------------------------------

Lovely code ;)

Comment 17

[Tim Taubert [:ttaubert] (inactive)]

https://hg.mozilla.org/integration/mozilla-inbound/rev/79190c8b1ae5

Comment 18

[Phil Ringnalda (:philor)]

https://hg.mozilla.org/mozilla-central/rev/79190c8b1ae5

Comment 19

[Tim Taubert [:ttaubert] (inactive)]

https://hg.mozilla.org/releases/mozilla-aurora/rev/f7efbe5afafe
https://hg.mozilla.org/releases/mozilla-b2g18/rev/8fafd2a20d51

Comment 20

[Angela Hubenya]

defect no longer reproduces. checked on unagi build id: 20130117070201