Closed
Bug 763036
Opened 13 years ago
Closed 13 years ago
Long taps broken on Top Sites tab
Categories
(Firefox for Android Graveyard :: General, defect)
Firefox for Android Graveyard
General
Tracking
(firefox15+ fixed, firefox16 verified, firefox17 verified)
People
(Reporter: wesj, Assigned: wesj)
References
Details
(Keywords: regression)
Attachments
(1 file)
4.01 KB,
patch
|
mbrubeck
:
review+
bnicholson
:
review+
akeybl
:
approval-mozilla-beta+
|
Details | Diff | Splinter Review |
Search suggestions changed something so that now we return an AwesomeBarItem rather than a Cursor when we call getItemFromPosition(). I'm filing this to track, but with the refactoring in bug 759041 this will be cleaner to fix, so I'm planning to just fix it there.
Comment 2•13 years ago
|
||
When I long tap on an entry from Top Sites I get errors like this (one error for each long tap):
E/GeckoAwesomeBar( 9831): item at 0 is not a Cursor
E/GeckoAwesomeBar( 9831): item at 1 is not a Cursor
E/GeckoAwesomeBar( 9831): item at 2 is not a Cursor
E/GeckoAwesomeBar( 9831): item at 3 is not a Cursor
--
Firefox 16.0a1 (2012-06-19)
Device: Galaxy Nexus
OS: Android 4.0.2
Summary: Long taps broken on All Pages tab → Long taps broken on Top Sites tab
Comment 3•13 years ago
|
||
The respective patches of bug 759041 were already landed some time ago, even though that bug itself is not fixed yet. Therefore this works now.
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → WORKSFORME
Comment 4•13 years ago
|
||
This is still an issue on Firefox Mobile 15.0b2 build 1 on HTC Desire running Android 2.2
Comment 5•13 years ago
|
||
Right, the patches landed on m-c when it was 16 and were not uplifted to 15.
Wes, is it possible to uplift those patches to Beta?
If we do that though, we probably also want to fix bug 777649 for Beta.
Status: RESOLVED → REOPENED
tracking-fennec: --- → ?
status-firefox15:
--- → affected
status-firefox16:
--- → fixed
status-firefox17:
--- → fixed
Resolution: WORKSFORME → ---
Assignee | ||
Comment 6•13 years ago
|
||
The fix in 16 isn't something I want to push forward. I'll put together a separate patch for 15 if we want it?
Comment 7•13 years ago
|
||
I think this is definitely something we need to fix for 15, since it's a regression.
tracking-firefox15:
--- → ?
Keywords: regression
Assignee | ||
Comment 8•13 years ago
|
||
Since this will go straight to beta with no testing on central, thought more eyes would be better. This is pretty similar to what's on central though, without the refactor. Tested locally tapping on different entries in all three panes with and without search suggestions.
I'll ask for approval once the reviews are done.
Assignee: nobody → wjohnston
Attachment #646191 -
Flags: review?(mbrubeck)
Attachment #646191 -
Flags: review?(bnicholson)
Attachment #646191 -
Flags: approval-mozilla-beta?
Assignee | ||
Updated•13 years ago
|
Attachment #646191 -
Flags: approval-mozilla-beta?
Updated•13 years ago
|
Attachment #646191 -
Flags: review?(mbrubeck) → review+
Comment 9•13 years ago
|
||
Wes, is bug 777649 also present in 15? Did you fix it with your patch here? If yes please remember that the other channels need a fix for it as well.
Updated•13 years ago
|
Attachment #646191 -
Flags: review?(bnicholson) → review+
Assignee | ||
Comment 10•13 years ago
|
||
Comment on attachment 646191 [details] [diff] [review]
Patch for beta
[Approval Request Comment]
Bug caused by (feature/regressing bug #): Search suggestions
User impact if declined: No context menu on awesomescreen
Testing completed (on m-c, etc.): Patch on m-c is very similar, but not exactly the same. Tested locally.
Risk to taking this patch (and alternatives if risky): Low risk. Basically adds a new path and a new getter method.
String or UUID changes made by this patch: None.
Attachment #646191 -
Flags: approval-mozilla-beta?
Updated•13 years ago
|
Updated•13 years ago
|
Attachment #646191 -
Flags: approval-mozilla-beta? → approval-mozilla-beta+
Assignee | ||
Comment 11•13 years ago
|
||
Status: REOPENED → RESOLVED
Closed: 13 years ago → 13 years ago
Resolution: --- → FIXED
Updated•13 years ago
|
Comment 12•12 years ago
|
||
Tcs added in MozTrap for Firefox 15 and later versions both phone and tablet:
https://moztrap.mozilla.org/manage/case/6030/ Awesomescreen tablet suite -
Long tap in Top Sites
https://moztrap.mozilla.org/manage/case/6031/ Awesomescreen suite -
Long tap in Top Sites
Flags: in-moztrap+
Updated•11 years ago
|
tracking-fennec: ? → ---
Updated•4 years ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•