Closed Bug 1166969 Opened 9 years ago Closed 9 years ago

[Browser][Text Selection] Long pressing improperly activates text selection carets around the built in browser media player.

Categories

(Core :: DOM: Selection, defect)

Product:
Core
Component:
DOM: Selection
Platform:
ARM
Gonk (Firefox OS)
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
VERIFIED FIXED
Milestone:
mozilla41
Tracking Flags:
Tracking Status
firefox41 --- fixed
b2g-v2.2 --- affected
b2g-master --- verified

People

(Reporter: Marty, Assigned: mtseng)

References

(Blocks 1 open bug,
URL
)

Details

(Whiteboard: [3.0-Daily-Testing])

Attachments

(2 files)

logcat-browser-text select.txt
38.42 KB, text/plain
Details
Video element in the video document shouldn't be selected.
580 bytes, patch
roc
: review+
Details | Diff | Splinter Review 
Description:
When the user accesses a media file directly in the browser, they can long press outside of the player, and activate text selection carets on either side of the player.


Repro Steps:
1) Update a Flame to 20150520010202
2) Launch the Browser app and access a media file (such as .mp3 or .3gp. can be found at http://download.wavetlan.com/svv/)
3) Long press the blank space below the media player.

Actual:
Text selection carets appear on either side of the browser media player.

Expected:
Erroneous text selection carets do not appear around the media player

Environmental Variables:
Device: Flame 3.0 (319MB)(Full Flash)
Build ID: 20150520010202
Gaia: 600fd8249960b8256af9de67d9171025bb9a3ff3
Gecko: ac277e615f8f
Gonk: 040bb1e9ac8a5b6dd756fdd696aa37a8868b5c67
Version: 41.0a1 (3.0)
Firmware Version: v18D-1
User Agent: Mozilla/5.0 (Mobile; rv:41.0) Gecko/41.0 Firefox/41.0

Repro frequency: 8/8
See attached: logcat, video (URL)
This issue DOES occur on Flame 2.2 builds.
Text selection carets appear on either side of the browser media player.

Environmental Variables:
Device: Flame 2.2 (319MB)(Full Flash)
Build ID: 20150519002500
Gaia: 732acec6f37d13ccea6b0ddc48904a53a2970894
Gecko: 1389e6b8c065
Gonk: bd9cb3af2a0354577a6903917bc826489050b40d
Version: 37.0 (2.2)
Firmware Version: v18D-1
User Agent: Mozilla/5.0 (Mobile; rv:37.0) Gecko/37.0 Firefox/37.0

------------------------------------------------------------------------------

This issue does NOT occur on Flame 2.1 builds.
Text selection is not implemented on this branch.
QA Whiteboard: [QAnalyst-Triage?]
Flags: needinfo?(pbylenga)
Text selection is a new feature as of 2.2. NI Cut/Copy/Paste owner for input
QA Whiteboard: [QAnalyst-Triage?] → [QAnalyst-Triage+]
Flags: needinfo?(pbylenga) → needinfo?(gchang)
Component: Gaia::Browser → Selection
Product: Firefox OS → Core
George, can you take a look?
Flags: needinfo?(gduan)
Hi Morris,
I think it's probably a gecko's issue. The browser page only contain a video tag and the result of window.getSeleciton() is {focusNode: <body>, isCollapsed: true, rangeCount:1 } . 
Could you take a look at it?
Flags: needinfo?(gduan) → needinfo?(mtseng)
Assignee: nobody → mtseng
Status: NEW → ASSIGNED
Flags: needinfo?(mtseng)
Flags: needinfo?(gchang)
https://hg.mozilla.org/mozilla-central/rev/a1425b09ef3e
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
status-firefox41: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla41
This issue is verified fixed on the latest Spark 2.5, and Flame 2.5 builds.
Erroneous text selection carets do not appear around the media player

Environmental Variables:
Device: Aries 2.5
BuildID: 20150812135535
Gaia: 6fef72357971934c8774578044ea7a442be3a75d
Gecko: b7efed3a2640fba681d48846c3b2ab80474c6f77
Gonk: 2916e2368074b5383c80bf5a0fba3fc83ba310bd
Version: 43.0a1 (2.5) 
Firmware Version: D5803_23.1.A.1.28_NCB.ftf
User Agent: Mozilla/5.0 (Mobile; rv:43.0) Gecko/43.0 Firefox/43.0


Environmental Variables:
Device: Flame 2.5
BuildID: 20150812030210
Gaia: c7875bbc8b32e7b95cc55c9690b03b140905d84d
Gecko: d4f3a8a75577e4af2914a4e899ca2e724f9715c4
Gonk: c4779d6da0f85894b1f78f0351b43f2949e8decd
Version: 43.0a1 (2.5) 
Firmware Version: v18D
User Agent: Mozilla/5.0 (Mobile; rv:43.0) Gecko/43.0 Firefox/43.0
Status: RESOLVED → VERIFIED
QA Whiteboard: [QAnalyst-Triage+] → [QAnalyst-Triage?]
status-b2g-master: affected → verified
Flags: needinfo?(ktucker)
QA Whiteboard: [QAnalyst-Triage?] → [QAnalyst-Triage+]
Flags: needinfo?(ktucker)
You need to log in before you can comment on or make changes to this bug.