948558 - New Permaorange: /tests/content/events/test/test_bug864040.html | TEXTAREA: failed to set selection

Status: RESOLVED FIXED

(Firefox OS Graveyard :: Gaia::Keyboard, defect)

Description

[Gregor Wagner [:gwagner]]

On b2g emulator builds with debug-gecko:

Regression range:
https://tbpl.mozilla.org/?tree=Pine&rev=c17bf8506bff

Comment 1

[Jonathan Watt [:jwatt]]

I don't think my patches can have caused this, since they should only affect <input>, and there's no <input> used in test_bug864040.html. Masayuki, could your patches for bug 930374 have caused this?

Comment 2

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)]

> Masayuki, could your patches for bug 930374 have caused this?

I don't think so. It fails to set selection via nsIDOMWindowUtils::SendSelectionEvent()...

Comment 3

[Jonathan Watt [:jwatt]]

What is Pine, anyway, and how is it related to m-i?

Comment 4

[Gregor Wagner [:gwagner]]

Pine is a project branch to add new testing frameworks for b2g. Currently we are trying to add mochitests on b2g-emulators running debug-gecko (right now we only have opt builds running mochitests). We keep regressing it and this prevents us from deploying to all tbpl pages.

Comment 5

[Robert O'Callahan (:roc) (email my personal email if necessary)]

Moving test_bug864040.html to dom/events/test in bug 946065 causes this test to perma-fail too.

I think the test can only ever have worked on B2G by luck, and should be disabled on B2G.

Comment 6

[Robert O'Callahan (:roc) (email my personal email if necessary)]

(In reply to Gregor Wagner [:gwagner] from comment #0)
> Regression range:
> https://tbpl.mozilla.org/?tree=Pine&rev=c17bf8506bff

This link is dead unfortunately.

Comment 7

[Robert O'Callahan (:roc) (email my personal email if necessary)]

Attachment: disable test on B2G

Comment 8

[Andrew McCreight [:mccr8]]

This patch I landed and then had backed out also made that test go permaorange on B2G ICS Emulator Opt for some reason:
  https://hg.mozilla.org/integration/mozilla-inbound/rev/99eb8a370b21

Comment 9

[Gregor Wagner [:gwagner]]

Comment on attachment 8357582 [details] [diff] [review]
disable test on B2G

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

I think this is a debug-only test failure and we disable this test with a lot of other failing tests in bug 929122. I am not sure that disabling on other b2g-environments is needed.

Comment 10

[Robert O'Callahan (:roc) (email my personal email if necessary)]

It is not debug-only with Andrew's patch from comment #8 or when I simply move the test from content/events to dom/events. That strongly suggests to me the test is only passing on b2g opt by luck.

Comment 11

[Robert O'Callahan (:roc) (email my personal email if necessary)]

https://hg.mozilla.org/integration/mozilla-inbound/rev/f7035947e1de

Comment 12

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/f7035947e1de