The default bug view has changed. See this FAQ.

Status

()

Firefox for Android
General
P2
normal
VERIFIED FIXED
6 years ago
8 months ago

People

(Reporter: elan, Assigned: kats)

Tracking

({feature})

unspecified
ARM
Android
feature
Points:
---

Firefox Tracking Flags

(firefox11 fixed, fennec11+)

Details

Attachments

(3 attachments, 3 obsolete attachments)

Comment hidden (empty)
Priority: P1 → P2

Updated

6 years ago
Whiteboard: [QA+]
I can put in the plumbing for this pending UX mockups.
Assignee: nobody → kgupta
Created attachment 569986 [details] [diff] [review]
(1/3) Minor cleanup in Tab.java
Attachment #569986 - Flags: review?(sriram)
Attachment #569986 - Attachment description: (1/1) Minor cleanup in Tab.java → (1/3) Minor cleanup in Tab.java
Created attachment 569987 [details] [diff] [review]
(2/3) Plumbing for Forward

I found this led to much simpler code than turning mHistory into a list and keeping an index pointer. That approach has lots of corner cases when the list is empty, or the index is out of bounds.
Attachment #569987 - Flags: review?(sriram)
Created attachment 569988 [details] [diff] [review]
(3/3) Hook up forward as menu item

Making this a separate change so that it can be backed out if UX changes without affecting the plumbing part.
Attachment #569988 - Flags: review?(sriram)
Depends on: 697732
Attachment #569986 - Flags: review?(sriram) → review+
Attachment #569988 - Flags: review?(sriram) → review+
Comment on attachment 569987 [details] [diff] [review]
(2/3) Plumbing for Forward

Is there a reason for using 2 stacks? We need to pop from one and push into another. Why not have a List<HistoryEntry> and use an index variable? That operation is faster I guess.
Attachment #569987 - Flags: review?(sriram) → review+
Comment on attachment 569986 [details] [diff] [review]
(1/3) Minor cleanup in Tab.java

Moved this patch to 697732, since those patches need to go in first.
Attachment #569986 - Attachment is obsolete: true
Created attachment 570248 [details] [diff] [review]
(1/2) Add plumbing for forward

This patch applies on top of the ones from bug 695165
Attachment #569987 - Attachment is obsolete: true
Attachment #570248 - Flags: review?
Created attachment 570249 [details] [diff] [review]
(2/2) Hook up forward as menu item

Rebased the old patch, but no functional changes.
Attachment #569988 - Attachment is obsolete: true
Attachment #570249 - Flags: review+
Drive-by:

* Could we change "session-*" messages to "Session:*" ?
Created attachment 570252 [details] [diff] [review]
(3/2) Rename events for consistency
Attachment #570252 - Flags: review?(mark.finkle)
Comment on attachment 570252 [details] [diff] [review]
(3/2) Rename events for consistency

Thanks
Attachment #570252 - Flags: review?(mark.finkle) → review+
Attachment #570248 - Flags: review? → review+
https://hg.mozilla.org/projects/birch/rev/21fd85f89ad9
https://hg.mozilla.org/projects/birch/rev/7f8a9ba22b10
https://hg.mozilla.org/projects/birch/rev/591ac51b8359
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
20111031103218
http://hg.mozilla.org/projects/birch/rev/9f5570d58bca
Samsung Galaxy SII (Android 2.3.4)
Status: RESOLVED → VERIFIED
Whiteboard: [QA+]
tracking-fennec: --- → 11+
status-firefox11: --- → fixed
(Reporter)

Updated

5 years ago
Keywords: feature
You need to log in before you can comment on or make changes to this bug.