Closed
Bug 1130993
Opened 10 years ago
Closed 9 years ago
Received selectionstatechanged before Webapps:Launch when launch an app (~20ms)
Categories
(Firefox OS Graveyard :: Performance, defect)
Tracking
(feature-b2g:2.5+, b2g-v2.2 affected)
Tracking | Status | |
---|---|---|
b2g-v2.2 | --- | affected |
People
(Reporter: ting, Assigned: ting)
References
(Blocks 1 open bug)
Details
(Keywords: perf)
When launch an app, a selectionstatechanged event (sent from Homescreen process) always comes before Webapps:Launch. The profile from Flame shows it takes ~20ms on B2G main thread: http://people.mozilla.org/~bgirard/cleopatra/#report=a899fb11de4ad4b292f5f2e19b918949ed5041e9&select=1598,1619.
I am not sure is the event necessary.
Updated•10 years ago
|
Blocks: CopyPasteLegacy
Comment 1•10 years ago
|
||
I found that we have 1 selectionstatechanged event with reason blur before app launch. I don't think we have enough information to filter out this useless event. We should able to resolve this issue after refactoring Touch/Selection carets code.
Updated•10 years ago
|
Assignee | ||
Updated•10 years ago
|
Summary: Received selectionstatechanged before Webapps:Launch when launch an app → Received selectionstatechanged before Webapps:Launch when launch an app (~20ms)
Assignee | ||
Updated•10 years ago
|
Assignee | ||
Updated•10 years ago
|
status-b2g-v2.2:
--- → affected
Comment 2•10 years ago
|
||
I examine this bug after applying touch/selectioncaret refactor code. And I cannot find this event before app launch. So, let wait refactoring code getting landed and check it again!!
Comment 3•10 years ago
|
||
[Tracking Requested - why for this release]:
tracking-b2g:
--- → backlog
Keywords: perf
Assignee | ||
Comment 4•9 years ago
|
||
With the profile [1] today, I can still see it [119236,119283]...
Gecko: https://hg.mozilla.org/mozilla-central/rev/1af1b4e1c35a
[1] http://people.mozilla.org/~bgirard/cleopatra/#report=9ca00bbd19a1585f5e3cbeb8a624da404ccbc6c4
Flags: needinfo?(mtseng)
Comment 5•9 years ago
|
||
Yes, we didn't enable accessible caret by default. So this problem still exists. Bug 1172382 will enable it by default. Let's wait!
Depends on: 1172382
Flags: needinfo?(mtseng)
Comment 6•9 years ago
|
||
Bug 1172382 was landed. Can this issue still be reproduced?
Assignee | ||
Comment 7•9 years ago
|
||
(In reply to Ting-Yu Lin [:TYLin] (UTC+8) from comment #6)
> Bug 1172382 was landed. Can this issue still be reproduced?
Great! I don't see it anymore.
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Updated•9 years ago
|
tracking-b2g:
backlog → ---
Target Milestone: --- → FxOS-S4 (07Aug)
Updated•9 years ago
|
Assignee: nobody → janus926
You need to log in
before you can comment on or make changes to this bug.
Description
•