Closed
Bug 709595
Opened 11 years ago
Closed 11 years ago
name threads and timers
Categories
(Firefox for Android Graveyard :: General, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: dougt, Assigned: dougt)
Details
Attachments
(1 file)
4.41 KB,
patch
|
blassey
:
review+
|
Details | Diff | Splinter Review |
traceview shows threads. be nice and set a name for every thread and timer.
Assignee | ||
Comment 1•11 years ago
|
||
Assignee: nobody → doug.turner
Attachment #580756 -
Flags: review?(blassey.bugs)
Comment 2•11 years ago
|
||
Comment on attachment 580756 [details] [diff] [review] patch v.1 Review of attachment 580756 [details] [diff] [review]: ----------------------------------------------------------------- ::: mobile/android/base/GeckoAppShell.java @@ +145,5 @@ > public SynchronousQueue<Handler> mHandlerQueue = > new SynchronousQueue<Handler>(); > > public void run() { > + setName("GeckoLooper Thread"); maybe call this one the "Main UI thread"
Attachment #580756 -
Flags: review?(blassey.bugs) → review+
Comment 3•11 years ago
|
||
Comment on attachment 580756 [details] [diff] [review] patch v.1 Review of attachment 580756 [details] [diff] [review]: ----------------------------------------------------------------- ::: mobile/android/base/GeckoAppShell.java @@ +145,5 @@ > public SynchronousQueue<Handler> mHandlerQueue = > new SynchronousQueue<Handler>(); > > public void run() { > + setName("GeckoLooper Thread"); got this one wrong, please name it "Background Thread"
Assignee | ||
Comment 4•11 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/0ce8da61c6cc
Assignee | ||
Comment 5•11 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/05d5c4ef7efc
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Updated•1 year ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•