1044493 - Don't sync rewritten reader mode URLs

Status: VERIFIED DUPLICATE of bug 977167

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

Description

[Jeremy B. Merrill]

User Agent: Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:31.0) Gecko/20100101 Firefox/31.0 (Beta/Release)
Build ID: 20140715214327

Steps to reproduce:

0. Have sync set up between a mobile device and a desktop device. (In my case, that's Android 4.2 with Firefox for Mobile 31.0 and Ubuntu 14.04 with Firefox 31.0, but I imagine this is across all pairings of a mobile and desktop platform)
1. On a Firefox mobile device, open a link, in my case http://ancientscripts.com/epiolmec.html
2. Click the reader button in the URL bar on the mobile device. (And note that the URL has changed to about:reader?url=http%3A%2F%2Fwww.ancientscripts.com%2Fepiolmec.html&tabId=0)
3. On the desktop device, open the Tabs from Other Devices page (by clicking History -> Tabs from Other Devices)
4. Note the existence of the page from mobile in the Tabs from Other Devices list (on desktop) -- with the about:reader URL.
5. Click the entry of the page from mobile to open it in a tab on the desktop device.


Actual results:

Note the "The address isn't valid The URL is not valid and cannot be loaded." error message form the URL (that begins about-reader:).


Expected results:

The "about-reader:"-protocol links should be redirected on desktop to the ordinary site -- in this case, http://ancientscripts.com/epiolmec.html should have been opened instead.

(An alternative would be to implement the reader on Desktop, but that seems like a lot more work for less payoff.)

Comment 1

[Richard Newman [:rnewman]]

Thanks for the report.

The original page URLs should be being synced, not the rewritten URLs.

Comment 2

[Tristan Pollitt]

Tried to reproduce on Fennec 34.0a1 and Firefox 31.0. All tabs except those which are in reader mode show up under 'Tabs From Other Devices', I'm not getting the "about-reader:" tabs at all.

Comment 3

[Richard Newman [:rnewman]]

Ah, you're testing 31. See Bug 977167, fixed in 32+.

Comment 4

[James Bonacci [:jbonacci]]

OK.

Comment 5

[Jeremy B. Merrill]

Ah, glad this was already solved! Thanks for y'all's response here and your great work building this browser. I'll wait patiently for the fix in early September. :)