790880 - Reader Mode failed to add a page to Reading List

Status: RESOLVED DUPLICATE of bug 800899

(Firefox for Android Graveyard :: Reader View, defect, P1)

Description

[Cristian Nicolae (:xti)]

Attachment: screenshot

Firefox 18.0a1 (2012-09-12)
Device: Galaxy Nexus
OS: Android 4.1.1

Steps to reproduce:
1. Open Firefox for Android
2. Go to http://goo.gl/Rcd0S (http://mobile.philly.com/news/?wss=/philly/news/nation_world/&id=169563286)
3. Tap on Reader Mode icon from URL Bar
4. Scroll down the article and tap on "Read More" link
5. Wait until the new page is loaded and repeat step 3
6. Tap on "Add to Reading List" button on Reader Mode toolbar
7. Tap on device Back button until the page opened at step 2 is displayed
8. Pan down the webpage and tap on "Read More" link
9. Repeat steps 5&6

Expected result:
After step 9, the article is removed from the Reading List.

Actual result:
After step 9, on Reader toolbar the "Add to Reading List" button is displayed, instead of "Remove from Reading List" button. Tapping on it, a toaster notification is triggered as you can see in the attached screenshot: "Failed to add page to your Reading List". After this, the Remove button is displayed as expected.

Comment 1

[Cristian Nicolae (:xti)]

Note: The same happens if after step 8, you try to add the page to reading list via custom menu. In this case the toaster notification mentioned above will be always triggered and the menu option will be never updated as expected.

Comment 2

[Aaron Train [:aaronmt]]

On add

E/GeckoConsole(13961): Reader: Error storing article in the cache DB: http://mobile.philly.com/news/?wss=/philly/news/nation_world/&id=169563286&viewAll=y
E/GeckoConsole(13961): AboutReader: Reader:Add (in reader) success=false
I/GeckoTabs(13961): Got message: Reader:Added
E/GeckoConsole(13961): [JavaScript Error: "ConstraintError" {file: "chrome://browser/content/browser.js" line: 6403}]
E/NotificationService(  328): Suppressing toast from package org.mozilla.firefox_beta by user request.

On remove

E/GeckoConsole(13961): Reader: Removed article from the cache DB: http://mobile.philly.com/news/?wss=/philly/news/nation_world/&id=169563286&viewAll=y
E/GeckoConsole(13961): AboutReader: Reader:Remove (in reader) success=true
I/GeckoTabs(13961): Got message: Reader:Removed

On add again

E/GeckoConsole(13961): AboutReader: Reader:Add (in reader) success=true
I/GeckoTabs(13961): Got message: Reader:Added
E/NotificationService(  328): Suppressing toast from package org.mozilla.firefox_beta by user request.

Comment 3

[Aaron Train [:aaronmt]]

Does this P1 assignment post brach tracking decision influence any decision to make this tracking 16 instead?

Comment 4

[Lucas Rocha (:lucasr)]

(In reply to Aaron Train [:aaronmt] from comment #3)
> Does this P1 assignment post brach tracking decision influence any decision
> to make this tracking 16 instead?

I'm only using the priority to organize my work on reader. It doesn't necessarily have any influence of target/tracking releases.

Comment 5

[Mark Finkle (:mfinkle) (use needinfo?)]

Lucas - Any thoughts on ETA?

Comment 6

[Lucas Rocha (:lucasr)]

I think I know what's happening. Should be able to fix it this week.