After bug 989198 lands, we should make UITest app to test this feature.
Steal this bug from dwi2, since he is busy on other issues.
Assignee: tzhuang → johu
According to wiki, I will add a page to uitest-privileged app to capture the following 4 keys and try to cancel them: *. Power *. Exit *. VolumeUp *. VolumeDown The expected behavior is VolumeUp and VolumeDown can be intercepted by app and cancelled.  https://wiki.mozilla.org/WebAPI/BrowserAPI/KeyboardEvent#Part_2_-_Key_event_handling_customization
A WIP patch can be found here: https://github.com/huchengtw-moz/gaia/commit/abd347f400d9fed0cd3312954fe74cf8da7cf04e This WIP patch depends on the patch of bug 1014418 and bug 989198.
Created attachment 8471451 [details] [review] patch for this bug Hi AlTasi, This patch needs bug 1014418 and bug 989198 to work or we can build a workable version for you. Please give us some feedback.
Attachment #8471451 - Flags: feedback?(atsai)
Plan to land it by the end of v2.1 sprint 3 (August 29).
Target Milestone: --- → 2.1 S3 (29aug)
A workable version of local build should be fine for me. I am still new to the project. Let me go to you for more details.
Comment on attachment 8471451 [details] [review] patch for this bug The patch seems good to me. Should I wait for the other two fixes for other bugs ready and execute testing?
Attachment #8471451 - Flags: feedback?(atsai) → feedback+
Bug 989198 may need to refactor to fix a critical issue. I will create a workable version for you once it is ready.
Confirmed with EM/EPM, and this can be landed before FL.
It doesn't look like a strong need to be landed in 2.1 and also we already passed the feature landing date for 1 week. Postpone this to the next version for now.
feature-b2g: 2.1 → 2.2?
Created attachment 8523726 [details] [review] patch for this bug atsai, This PR is similar to the previous one. The difference is that we may need to focus the iframe while test opened. I recreate the PR because the previous one is closed by other guys. Please review this patch. BTW, the prevent default behavior is malfunction because of bug 1094066. We may use this to test that issue.
Comment on attachment 8523726 [details] [review] patch for this bug R=me. Good to merge.
Attachment #8523726 - Flags: review?(atsai) → review+
Status: NEW → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.