Copy-paste of link from URL bar after opening link from Facebook Android app gives redirect link, not the target
VERIFIED
FIXED
in Firefox 11
Status
()
People
(Reporter: brion, Assigned: cpeterson)
Tracking
Firefox Tracking Flags
(firefox11 fixed, firefox12 fixed, fennec11+)
Details
(Whiteboard: [MTD])
Attachments
(2 attachments, 3 obsolete attachments)
2.52 KB,
patch
|
cpeterson
:
review+
blassey
:
approval-mozilla-aurora+
|
Details | Diff | Splinter Review |
5.72 KB,
patch
|
cpeterson
:
review+
blassey
:
approval-mozilla-aurora+
|
Details | Diff | Splinter Review |
User Agent: Mozilla/5.0 (Ubuntu; X11; Linux x86_64; rv:8.0) Gecko/20100101 Firefox/8.0 Build ID: 20111115183813 Steps to reproduce: Followed a few links from Facebook (using Android Facebook app) opening them directly in Firefox nightly (20111212). After loading, tap into the URL bar, tap-and hold for 'select all', then 'copy'. Actual results: A long ugly http://m.facebook.com/blah blah redirect URL is shown and copied. Expected results: The destination URL should have been copied. Note that I can't reproduce this simply opening an HTTP redirect page such as http://leuksman.com/misc/redirect1.php within Fennec itself. It may be specific to opening from another app, or from the way the facebook redirectors work.
(Reporter) | ||
Updated•7 years ago
|
Whiteboard: [MTD]
Updated•7 years ago
|
Assignee: nobody → cpeterson
Status: UNCONFIRMED → NEW
Ever confirmed: true
Priority: -- → P2
(Assignee) | ||
Updated•7 years ago
|
Status: NEW → ASSIGNED
Updated•7 years ago
|
OS: Linux → Android
Hardware: x86_64 → ARM
Updated•7 years ago
|
tracking-fennec: --- → 11+
(Assignee) | ||
Comment 1•7 years ago
|
||
Created attachment 588273 [details] [diff] [review] bug-709845-part-1-update-history-url.patch Update the tab history's URL after a redirect.
Attachment #588273 -
Flags: review?(doug.turner)
(Assignee) | ||
Comment 2•7 years ago
|
||
Created attachment 588274 [details] [diff] [review] bug-709845-part-2-fix-checkstyle-warnings.patch Tab.java: Fix some checkstyle warnings about extra imports, missing access modifiers, trailing whitespace.
Attachment #588274 -
Flags: review?(doug.turner)
Updated•7 years ago
|
Attachment #588273 -
Flags: review?(doug.turner) → review?(mbrubeck)
Comment 3•7 years ago
|
||
Comment on attachment 588274 [details] [diff] [review] bug-709845-part-2-fix-checkstyle-warnings.patch next time just factor code cleanup in its own bug.
Attachment #588274 -
Flags: review?(doug.turner) → review+
Updated•7 years ago
|
Attachment #588273 -
Flags: review?(mbrubeck) → review+
(Assignee) | ||
Updated•7 years ago
|
Keywords: checkin-needed
(Assignee) | ||
Comment 4•7 years ago
|
||
Created attachment 589246 [details] [diff] [review] bug-709845-part-1-update-history-url-v1-rebased.patch Rebased patch. r=mbrubeck
Attachment #589246 -
Flags: review+
(Assignee) | ||
Comment 5•7 years ago
|
||
Created attachment 589248 [details] [diff] [review] bug-709845-part-1-update-history-url-v1-rebased.patch r=mbrubeck
Attachment #588273 -
Attachment is obsolete: true
Attachment #589246 -
Attachment is obsolete: true
Attachment #589248 -
Flags: review+
(Assignee) | ||
Comment 6•7 years ago
|
||
Created attachment 589250 [details] [diff] [review] bug-709845-part-2-fix-checkstyle-warnings-v1-rebased.patch Rebased patch. r=dougt
Attachment #588274 -
Attachment is obsolete: true
Attachment #589250 -
Flags: review+
(Assignee) | ||
Comment 7•7 years ago
|
||
checkin-needed for rebased patches.
Comment 8•7 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/d27ddb7ad032 https://hg.mozilla.org/integration/mozilla-inbound/rev/339d1d51af7a
status-firefox11: --- → affected
status-firefox12: --- → affected
Keywords: checkin-needed
Comment 9•7 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/d27ddb7ad032 https://hg.mozilla.org/mozilla-central/rev/339d1d51af7a
Status: ASSIGNED → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 12
(Assignee) | ||
Comment 10•7 years ago
|
||
Comment on attachment 589248 [details] [diff] [review] bug-709845-part-1-update-history-url-v1-rebased.patch [Approval Request Comment] Regression caused by (bug #): None User impact if declined: Bookmarked and copy/pasted URLs will grab the REDIRECTING page's URL, not the REDIRECTED TO page's URL. Testing completed (on m-c, etc.): Landed on m-c yesterday. Risk to taking this patch (and alternatives if risky): Tab history might mix up pages' URLs.
Attachment #589248 -
Flags: approval-mozilla-aurora?
(Assignee) | ||
Comment 11•7 years ago
|
||
Comment on attachment 589250 [details] [diff] [review] bug-709845-part-2-fix-checkstyle-warnings-v1-rebased.patch [Approval Request Comment] Regression caused by (bug #): None User impact if declined: None because this patch correct code style warnings. Testing completed (on m-c, etc.): Landed on m-c yesterday. Risk to taking this patch (and alternatives if risky): Probably none.
Attachment #589250 -
Flags: approval-mozilla-aurora?
Updated•7 years ago
|
Attachment #589248 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Updated•7 years ago
|
Attachment #589250 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Comment 12•7 years ago
|
||
https://hg.mozilla.org/releases/mozilla-aurora/rev/ab94103a2cdd
status-firefox11: affected → fixed
status-firefox12: affected → fixed
Comment 14•6 years ago
|
||
Verified fixed on: -build: Firefox for Android 21.0a2 (2013-02-27), Firefox for Android 22.0a1 (2013-02-27) -device: Samsung Galaxy Tab 10.1 -OS: Android 3.1
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•