Import bookmarks and history from stock browser (Samsung S4, Android 4.4.2) doesn't work

RESOLVED FIXED in Firefox 43

Status

()

defect
RESOLVED FIXED
4 years ago
3 years ago

People

(Reporter: barbara, Assigned: liuche)

Tracking

(Blocks 1 bug)

Trunk
Firefox 43
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox42 affected, firefox43 fixed, fennec43+)

Details

Attachments

(1 attachment)

Reference to comment here https://bugzilla.mozilla.org/show_bug.cgi?id=1172623#c14

Samsung Galaxy S2 & Google Nexus tablet
NO  -->  Saved bookmark in Samsung default browser: did not import to Fennec build
NO  --> Saved bookmark in Opera browser: did not import to Fennec build
NO  --> Saved bookmark in Firefox browser: did not import to Fennec build (might not have to be considered)
YES --> Saved bookmark in Chrome browser (any bookmark folder): _did_ import to Fennec build

It seems that it's only importing history and bookmarks from Chrome and no other installed browser. It would be nice to find out if you can detect the user's default browser and import these bookmarks/history.

Since most of our users are on Samsung Galaxy's, we should investigate this issue. If the import functionality doesn't work at first, the user will loose trust and will be frustrated.
We can only import history and bookmarks from applications that support making that data public. All applications on Android are by default sandboxed, and the data is private.

The Android Stock Browser, and Chrome afterwards, provide a permission and API to access history and bookmarks. We use that system. Other browsers, including Firefox, do not support that system.
Good to know, when you say Android Stock browser, do you mean the default browser on e.g a Google phone like Nexus, and consequently a Samsung default stock browser could potentially not allow any access to the data?

Samsung is leading the Android market and if we can't access the stock browser's data, this feature might not be as powerful and useful for a new Firefox user on first app run.
(In reply to Barbara Bermes [:bbermes] from comment #2)
> Good to know, when you say Android Stock browser, do you mean the default
> browser on e.g a Google phone like Nexus, and consequently a Samsung default
> stock browser could potentially not allow any access to the data?

Exactly. The standard Android stock browser is the one that ships with that AOSP code. I believe Samsung ships a modified version with extra features. As such, it might not allow accessing the history and bookmark data.

> Samsung is leading the Android market and if we can't access the stock
> browser's data, this feature might not be as powerful and useful for a new
> Firefox user on first app run.

Maybe it is possible to see if any data exists, and only show the "Import" if there is data to import?
Although if it is a Samsung specific problem, it doesn't affect all Samsung phones. I own a Samsung Galaxy S3 Mini and am able to import bookmarks and history from the stock browser just fine - I just tested it on Nightly.
tracking-fennec: --- → ?
Thanks JanH, indeed; it seems like a bug affecting Samsung Galaxy S4 (correction, I said S2 previoulsy) but not e.g. for Samsung Galaxy Note, or others. 


Bug reported for Samsung galaxy s4, Android 4.4.2

Our most used Samsung devices by users
- Samsung Galaxy S3 (phone)
- Samsung Galaxy S2 (phone)
- etc.

So we will need to see if we can reproduce this for S2 and S3 phones
Summary: Import bookmarks and history from stock Android (Samsung) doesn't work → Import bookmarks and history from stock browser (Samsung S4, Android 4.4.2) doesn't work
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → DUPLICATE
Duplicate of bug: 963661
(In reply to Mark Finkle (:mfinkle) from comment #3)
> Maybe it is possible to see if any data exists, and only show the "Import"
> if there is data to import?

Sounds sensible. Or pop up a notification if nothing was available. I'll file that.
(In reply to Aaron Train [:aaronmt] from comment #6)
> 
> *** This bug has been marked as a duplicate of bug 963661 ***

Is 963661 for Chrome only or also stock?
Flags: needinfo?(aaron.train)
Status: RESOLVED → REOPENED
tracking-fennec: ? → 43+
Resolution: DUPLICATE → ---
It was filed against Chrome
Flags: needinfo?(aaron.train)
mfinkle can you nominate somebody to fix this please, this blocks first run for now and needs to go into 43
Flags: needinfo?(mark.finkle)
(In reply to Gian-Carlo Pascutto [:gcp] from comment #7)
> (In reply to Mark Finkle (:mfinkle) from comment #3)
> > Maybe it is possible to see if any data exists, and only show the "Import"
> > if there is data to import?
> 
> Sounds sensible. Or pop up a notification if nothing was available. I'll
> file that.

See 
https://bugzilla.mozilla.org/show_bug.cgi?id=1188271
for some discussion.

The solution appropriate for the settings dialog might be different from the one appropriate for a first run experience.
(In reply to Barbara Bermes [:bbermes] from comment #10)
> mfinkle can you nominate somebody to fix this please, this blocks first run
> for now and needs to go into 43

I think liuche could own this bug, especially if she's on the hook for the first run experience :)
(In reply to :Margaret Leibovic from comment #12)
> (In reply to Barbara Bermes [:bbermes] from comment #10)
> > mfinkle can you nominate somebody to fix this please, this blocks first run
> > for now and needs to go into 43
> 
> I think liuche could own this bug, especially if she's on the hook for the
> first run experience :)

Did we decide what the fix would be?

Are we going to try to only show the "Import" if we are sure it works? The alternative is to try, perhaps futilely, to get the import to actually work.
Flags: needinfo?(mark.finkle)
Assignee

Updated

4 years ago
Blocks: 1190989
Assignee

Comment 14

4 years ago
I'm going to do some investigation and see if I can figure out what's wrong, and if there's a fix for it.
Assignee: nobody → liuche
Assignee

Comment 15

4 years ago
Okay, this should be pretty straightforward actually - it looks like some of Samsung's phones uses a different content provider ( content://com.sec.android.app.sbrowser.browser/bookmarks ), but you can still access it.

However, the two samsung devices we have here are older models (10" tablet, S2) and work just fine with the existing code. Samsung might start using this new content provider starting with a certain version, so I'll either check for that version, or if the phone is a Samsung, try both content provider addresses.

It looks like Galaxy Note, S2, and S3-mini work, and maybe the problem starts with S4/Android 4.4. Barbara, I'll make a build and send it to you to test on your S4, but it's annoying that we have to special case some (but not all) Samsung phones.
Assignee

Comment 16

4 years ago
Further investigation:
- Samsung Nexus S (4.1) can import from stock browser
- S2 (2.3) can import from stock browser
- S5 (5.0x) doesn't have a stock browser, and importing from Chrome works fine. The original 4.4 version does ship with a stock browser, but we don't have one to test on.

Barbara, here's a build here for you to try - I haven't tested it locally because I can't repro it on any of our Samsung devices.
http://people.mozilla.org/~liuche/bug-1186037/import1.apk

I have it pop up first run every time, so you don't have to clear your history when you test it.
Flags: needinfo?(bbermes)
I tried your latest apk, still not working :( on the Samsung S4, 4.4.2.
Flags: needinfo?(bbermes) → needinfo?(liuche)
Assignee

Updated

4 years ago
Blocks: 1196897
Assignee

Comment 18

4 years ago
Okay, I've gotten an S4 and I can repro this problem. I think we can also figure out if we don't see *any* history/bookmarks files, and not show the Import button if it ends up not being possible.
Assignee

Updated

4 years ago
Flags: needinfo?(liuche)
(In reply to Chenxia Liu [:liuche] from comment #18)
> Okay, I've gotten an S4 and I can repro this problem. I think we can also
> figure out if we don't see *any* history/bookmarks files, and not show the
> Import button if it ends up not being possible.

See
https://bugzilla.mozilla.org/show_bug.cgi?id=1188271
Assignee

Comment 20

4 years ago
Bug 1186037 - Import bookmarks and history from stock browser (Samsung S4, Android 4.4.2) doesn't work. r=mhaigh
Attachment #8651328 - Flags: review?(mhaigh)
Assignee

Comment 21

4 years ago
Cool, I got this working.

http://people.mozilla.org/~liuche/bug-1186037/
Assignee

Comment 22

4 years ago
Comment on attachment 8651328 [details]
MozReview Request: Bug 1186037 - Import bookmarks and history from stock browser (Samsung S4, Android 4.4.2) doesn't work. r=mhaigh

Bug 1186037 - Import bookmarks and history from stock browser (Samsung S4, Android 4.4.2) doesn't work. r=mhaigh
Attachment #8651328 - Flags: review?(mhaigh)
Assignee

Comment 23

4 years ago
Comment on attachment 8651328 [details]
MozReview Request: Bug 1186037 - Import bookmarks and history from stock browser (Samsung S4, Android 4.4.2) doesn't work. r=mhaigh

Bug 1186037 - Import bookmarks and history from stock browser (Samsung S4, Android 4.4.2) doesn't work. r=mhaigh
Attachment #8651328 - Flags: review?(mhaigh)
Comment on attachment 8651328 [details]
MozReview Request: Bug 1186037 - Import bookmarks and history from stock browser (Samsung S4, Android 4.4.2) doesn't work. r=mhaigh

https://reviewboard.mozilla.org/r/16933/#review15391
Attachment #8651328 - Flags: review?(mhaigh) → review+
https://hg.mozilla.org/mozilla-central/rev/8ad3f1c2b954
Status: REOPENED → RESOLVED
Closed: 4 years ago4 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 43
Recently we disabled and hid the feature on Android M+ devices because Google basically removed the public content provider for history and bookmarks. I wonder whether Samsung will keep their content provider when they release M devices and whether we should consider bringing it back.
No longer blocks: 1196897
You need to log in before you can comment on or make changes to this bug.