771610 - Reader Mode: Added items to reader list items do not enter about:reader

Status: VERIFIED FIXED

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

Description

[Aaron Train [:aaronmt]]

Currently, added items to the Reader List are simply treated as alternative bookmarks. When one adds a page to the reader list and then proceeds to visit the page from the Reader List folder (in the Awesome Screen), the page does not enter about:reader.

ER: Reader List opens added item about with an about:reader URI
AR: Reader List item is opened (basically a bookmark)

I used any article from http://neowin.net to test. I opened the article, opened my menu and selected 'Reader List'. I then went to the folder in the Awesome Screen and selected the item.

Comment 1

[Brian Nicholson (:bnicholson)]

Attachment: Open Reader Mode for Reading List bookmarks

Reading list items currently work in the all pages tab; we also need to check for them in the bookmarks tab.

Comment 2

[:Margaret Leibovic]

Comment on attachment 640863 [details] [diff] [review]
Open Reader Mode for Reading List bookmarks

I feel like it would be nicer to do the exact same check we're doing in AllPagesTab [1], but it looks like our bookmarks query doesn't give us back Combined.DISPLAY, so we couldn't do that without changing the query, and I don't think it's worth doing that.

Maybe you should just add a comment about what isInReadingList() actually means. I was initially confused that it didn't take a parameter, but it rather refers to the state of the bookmarks UI.

[1] http://mxr.mozilla.org/mozilla-central/source/mobile/android/base/awesomebar/AllPagesTab.java#202

Comment 3

[Brian Nicholson (:bnicholson)]

http://hg.mozilla.org/integration/mozilla-inbound/rev/f35365fb2a63

Comment 4

[Ed Morley [:emorley]]

https://hg.mozilla.org/mozilla-central/rev/f35365fb2a63

Comment 5

[Aaron Train [:aaronmt]]

Nightly (07/12), Samsung Galaxy Nexus (Android 4.1.1)

Comment 6

[Adrian Tamas (:AdrianT)]

A new test case was created under the Reading Mode&Reading List test suite in the BFTs run:
https://moztrap.mozilla.org/manage/cases/_detail/6364/