Closed
Bug 918326
Opened 11 years ago
Closed 11 years ago
[app manager] Can't start Firefox if the ADB addon is install
Categories
(DevTools Graveyard :: WebIDE, defect, P1)
Tracking
(firefox26 fixed)
RESOLVED
FIXED
Firefox 27
Tracking | Status | |
---|---|---|
firefox26 | --- | fixed |
People
(Reporter: paul, Assigned: ochameau)
References
Details
No STR yet. But I can't start Firefox because of the Adb addon.
Reporter | ||
Comment 1•11 years ago
|
||
I can't reproduce anymore :(
Reporter | ||
Updated•11 years ago
|
Priority: -- → P1
Reporter | ||
Comment 3•11 years ago
|
||
This happened to me again this morning. I'm not entirely sure what happened. After ~20 minutes of debugging, it just started working again.
Here is what I managed to understand:
- I needed to restart Firefox because the Adb Addon was not listing my device anymore (even after unplugging / plugging the phone)
- Firefox won't start until _isAdbRunning resolves the promise.
- done() in _isAdbRunning is called ~10 seconds after subprocess.call()
- a very long time is spent in:
> let isAdbRunning = lines.some(function(line) {
> return regex.test(line);
> });
- in this case, lines.length == 180
Reporter | ||
Comment 4•11 years ago
|
||
Ok, I figured it out: hangs happen when the output of "ps aux" is large (during a compilation for example).
Basically, if there are long lines in the output of "ps aux":
1) it takes a long time to subprocess.js to call the done() callback (~10s)
2) calling regex.test() on all of these lines takes a long time too (several minutes)
Try this code:
> let rx = /.*? \d+ .*? .*? \d+\s+\d+ .*? .*? .*? .*? adb .*fork\-server/
> let str = "paul 2023 0.0 0.0 14468 1428 pts/0 S+ 09:58 0:00 /usr/bin/make interfaces/base_libs interfaces/canvas_libs interfaces/core_libs interfaces/html_libs interfaces/events_libs interfaces/devicestorage_libs interfaces/contacts_libs interfaces/settings_libs interfaces/stylesheets_libs interfaces/sidebar_libs interfaces/css_libs interfaces/traversal_libs interfaces/range_libs interfaces/xbl_libs interfaces/xpath_libs interfaces/xul_libs interfaces/storage_libs interfaces/json_libs interfaces/offline_libs interfaces/geolocation_libs interfaces/notification_libs interfaces/permission_libs interfaces/svg_libs interfaces/smil_libs interfaces/apps_libs interfaces/gamepad_libs apps_libs base_libs activities_libs bindings_libs battery_libs bluetooth_libs browser-element_libs contacts_libs phonenumberutils_libs alarm_libs devicestorage_libs encoding_libs file_libs fmradio_libs media_libs messages_libs power_libs push_libs quota_libs settings_libs mobilemessage_libs src_libs time_libs locales_libs network_libs permission_libs plugins/base_libs plugins/ipc_libs indexedDB_libs system_libs ipc_libs identity_libs workers_libs camera_libs audiochannel_libs promise_libs wappush_libs telephony_libs webidl_libs gamepad_libs"
> rx.test(str)
This takes 2 seconds here.
Reporter | ||
Updated•11 years ago
|
Assignee: nobody → poirot.alex
Assignee | ||
Comment 5•11 years ago
|
||
It should be fixed by the new version (0.2.0) I pushed this morning to AMO. Please reopen if it still fails.
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Updated•11 years ago
|
Target Milestone: --- → Firefox 27
Reporter | ||
Updated•11 years ago
|
status-firefox26:
--- → fixed
Updated•6 years ago
|
Product: Firefox → DevTools
Updated•4 years ago
|
Product: DevTools → DevTools Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•