The "Settings" button is not responsive when setting up Sync from "Accounts & Sync" settings on HTC Desire

VERIFIED FIXED in Firefox 15

Status

Android Background Services
Android Sync
P1
normal
VERIFIED FIXED
5 years ago
4 years ago

People

(Reporter: AdrianT, Assigned: nalexander)

Tracking

unspecified
mozilla17
ARM
Android

Firefox Tracking Flags

(firefox14- wontfix, firefox15+ verified, firefox16 fixed, firefox17 fixed, fennec15+)

Details

(Whiteboard: [qa+])

Attachments

(4 attachments, 2 obsolete attachments)

(Reporter)

Description

5 years ago
Created attachment 642539 [details]
logs

Firefox Mobile Native 14.0.1 build 2
Device: HTC Desire (Android 2.2)

Steps to reproduce:
1. Open Settings and go to Accounts&Sync.
2. Choose to add a new Firefox sync account.
3. Setup the sync account.
4. Tap the "Settings" button to return to go to the account details.

Expected results:
The user is redirected to the Accounts&Sync.

Actual results:
Nothing happens.

Notes:
Please see the attached log. The first part of the log is the account setup from Accounts&Sync and the second is from Firefox mobile where the Settings button works.
The issue could not be reproduced on HTC Desire Z (Android 2.3.3)
I'm unable to reproduce this on the HTC Nexus One (2.3.4), Galaxy SII (4.0.4), or Galaxy Nexus (4.1.1).

HTC Desire is blocked on Google Play so I'm not too concerned.
Also this needs confirmation if this is reproducible, if it's a regression, and what builds it affects.
(Assignee)

Comment 3

5 years ago
07-16 15:01:37.405: I/ActivityManager(105): Starting activity: Intent { act=android.settings.SYNC_SETTINGS flg=0x10080000 cmp=com.htc.settings.accountsync/.ManageAccountsSettings }

Happens about 20 times.  I'm really loving these vendors that override Accounts and Sync settings.
Component: General → Android Sync
Product: Firefox for Android → Mozilla Services
Version: Firefox 14 → unspecified
Summary: The "Settings" button is not responsive when setting up the sync from Accounts&Sync on HTC Desire → The "Settings" button is not responsive when setting up Sync from "Accounts & Sync" settings on HTC Desire
So this is another instance of the MotoBlur bug, with a different alternate intent than that case?
(In reply to Mike Connor [:mconnor] from comment #4)
> So this is another instance of the MotoBlur bug, with a different alternate
> intent than that case?

Yes.

Doesn't apply to all HTC Sense devices.
Priority: -- → P1

Comment 6

5 years ago
This is reproducable on HTC Desire HD and HTC EVO 3D, using today's nightly (20120716).

Updated

5 years ago
tracking-firefox14: --- → ?

Comment 7

5 years ago
Another note: this bug will not appear during initial setup of sync accounts. However, it will occur if we remove the old sync account and re-adding it.
tracking-firefox14: ? → ---

Updated

5 years ago
tracking-firefox14: --- → ?

Comment 8

5 years ago
This appear to be happening starting fennec 14 beta 2.
I dont understand how this was unearthed now though, that's the weird thing. Anyways, thanks for looking Eric.
(In reply to Xiao Meng Wei :xwei from comment #8)
> This appear to be happening starting fennec 14 beta 2.

Thanks xwei.

Given that it's not a regression from 14.0, we'll prioritize to fix in 15.0. Wontfixing for 14.0.1.
Assignee: nobody → nalexander
status-firefox14: --- → wontfix
status-firefox15: --- → affected
tracking-firefox14: ? → -
tracking-firefox15: --- → +
tracking-fennec: ? → 15+
(Assignee)

Comment 11

5 years ago
See also Bug 721760.
(Assignee)

Comment 12

5 years ago
Could QA test

adb shell am start -a android.intent.action.VIEW -n com.htc.settings.accountsync/com.htc.settings.accountsync.ManageAccountsSettings

and

adb shell am start -a android.intent.action.VIEW -n com.htc.settings.accountsync/.ManageAccountsSettings

from the command line?

Comment 13

5 years ago
(In reply to Nick Alexander :nalexander from comment #12)
> Could QA test
> 
> adb shell am start -a android.intent.action.VIEW -n
> com.htc.settings.accountsync/com.htc.settings.accountsync.
> ManageAccountsSettings
> 
> and
> 
> adb shell am start -a android.intent.action.VIEW -n
> com.htc.settings.accountsync/.ManageAccountsSettings
> 
> from the command line?

They all brought up the same page, which is the account settings management page.
(Assignee)

Comment 14

5 years ago
http://people.mozilla.com/~nalexander/fennec-beta-htc.apk
(Assignee)

Comment 15

5 years ago
Created attachment 644423 [details] [diff] [review]
Patch against m-b

This generalizes the fix from Bug 721760, fixes that bug, adds the error checking that prevents Bug 773562, and fixes this bug.
Attachment #644423 - Flags: review?(rnewman)
(Assignee)

Comment 16

5 years ago
QA, we're going to need to test this functionality against:

* HTC phones that are affected
* HTC phones that are not affected
* Moto Blur phones that are affected
* Moto Blur phones that are not affected

I have tested this against my Samsung Galaxy SII, which is not affected.
(Assignee)

Updated

5 years ago
Keywords: qawanted
(In reply to Nick Alexander :nalexander from comment #16)
> * Moto Blur phones that are affected

Motorola Droid Pro (Android 2.3.4 with Moto Blur) -- No issues.

> * HTC phones that are not affected

HTC Nexus One (Android 2.3.4) -- No issues.

Comment 18

5 years ago
Created attachment 644448 [details]
Logcat from beta build on HTC EVO 3D

Comment 19

5 years ago
Created attachment 644449 [details]
Logcat from beta build on HTC Desire HD

Comment 20

5 years ago
This is still happening on the beta build on Desire HD and EVO 3D. I've attached logcats for constantly tapping on the settings button after sync setup.
(Assignee)

Comment 21

5 years ago
Hi xwei,

Sorry to put you through this.  Can you try

http://people.mozilla.com/~nalexander/fennec-beta-htc-no-task-flag.apk

and see if that helps.
(Assignee)

Comment 22

5 years ago
I'd also like some additional testing of the earlier apk (http://people.mozilla.com/~nalexander/fennec-beta-htc.apk).

Given that you say (https://bugzilla.mozilla.org/show_bug.cgi?id=774233#c7) that this only happens the second time round, I'd like to test scenario that load the settings page.  My guess is that somehow bringing the task to the front isn't working, and the second APK removes a flag that makes the activity start in a new task, which may change things.

Can you reboot the phone, navigate to Accounts & Settings *before* adding a first account, and then see if the button works?  If that makes the first run break -- then we know it's an HTC problem to do with loading, and we can try to find ways to work around that.

Also, can you add the first account, delete, re-add, verify failure, and then reboot and navigate to Account Settings from within Fennec?  If that works, then we know the HTC settings Activity can be started from anywhere, but only once the way we're doing it.

Thanks!

Comment 23

5 years ago
http://people.mozilla.com/~nalexander/fennec-beta-htc-no-task-flag.apk
This build has resolved this issue, as I could not reproduce this bug on this build on Desire HD or EVO 3D.
Attachment #644423 - Flags: review?(rnewman) → review+
(Assignee)

Comment 24

5 years ago
Just to update release drivers: I am moving on this and the related bugs.  This bug suggests that there are potential issues with Android Sync's task/activity management, which I am trying to get the bottom of.
(Assignee)

Comment 25

5 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/b5f2b664e57a
Status: NEW → ASSIGNED
Target Milestone: --- → mozilla17
Nick: there's an m-b patch on this bug, and also the fix you've just landed. Could you prep the patch that you want to land on 15 and set flags/obsolete accordingly?
(Assignee)

Updated

5 years ago
Depends on: 773562
(Assignee)

Comment 27

5 years ago
Created attachment 645054 [details] [diff] [review]
Patch against m-a and m-b

[Approval Request Comment]
Bug caused by (feature/regressing bug #):

This is not a regression: it's really a new bug.

User impact if declined:

Opening Sync Settings will work only the first time on some HTC devices.

Testing completed (on m-c, etc.):

QA has tested custom m-b builds with this.

Risk to taking this patch (and alternatives if risky):

Low risk.  It is possible that changing our Android stack management for this task is incorrect, but liuche and nalexander have discussed and tested several devices and we think this is the correct fix.

String or UUID changes made by this patch:

None.
Attachment #644423 - Attachment is obsolete: true
Attachment #645054 - Flags: approval-mozilla-beta?
Attachment #645054 - Flags: approval-mozilla-aurora?
(Assignee)

Comment 28

5 years ago
(In reply to Nick Alexander :nalexander from comment #27)
> Created attachment 645054 [details] [diff] [review]
> Patch against m-a and m-b

On m-b, this patch depends on the patch for Bug 773562 landing first.  (That Bug is already in place on m-a.)
Keywords: qawanted
Whiteboard: [qa+]
Comment on attachment 645054 [details] [diff] [review]
Patch against m-a and m-b

Review of attachment 645054 [details] [diff] [review]:
-----------------------------------------------------------------

Rubberstamp.

Go on, capitalize "do" :)
Attachment #645054 - Flags: review+
(Assignee)

Comment 30

5 years ago
Created attachment 645058 [details] [diff] [review]
Patch against m-a and m-b

[Approval Request Comment]
Bug caused by (feature/regressing bug #): 
User impact if declined: 
Testing completed (on m-c, etc.): 
Risk to taking this patch (and alternatives if risky): 
String or UUID changes made by this patch:

As above.
Attachment #645054 - Attachment is obsolete: true
Attachment #645054 - Flags: approval-mozilla-beta?
Attachment #645054 - Flags: approval-mozilla-aurora?
Attachment #645058 - Flags: approval-mozilla-beta?
Attachment #645058 - Flags: approval-mozilla-aurora?
(Assignee)

Comment 31

5 years ago
akeybl: ping on approvals, now that Bug 773562 has landed on beta?
https://hg.mozilla.org/mozilla-central/rev/b5f2b664e57a
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED

Updated

5 years ago
Attachment #645058 - Flags: approval-mozilla-beta?
Attachment #645058 - Flags: approval-mozilla-beta+
Attachment #645058 - Flags: approval-mozilla-aurora?
Attachment #645058 - Flags: approval-mozilla-aurora+
(Assignee)

Comment 33

5 years ago
https://hg.mozilla.org/releases/mozilla-beta/rev/9e81949496fd
https://hg.mozilla.org/releases/mozilla-aurora/rev/dbd697f53163
Looks like this is fixed across the board, we'll verify with the next beta build
status-firefox15: affected → fixed
status-firefox16: --- → fixed
status-firefox17: --- → fixed
(Reporter)

Comment 35

5 years ago
Issue is no longer reproducible on Firefox Mobile 15.0b2 on the HTC Desire running Android 2.2
status-firefox15: fixed → verified

Updated

5 years ago
Status: RESOLVED → VERIFIED
Component: Android Sync → Android Sync
Product: Mozilla Services → Android Background Services
You need to log in before you can comment on or make changes to this bug.