The default bug view has changed. See this FAQ.

frequently getting drag.info is null error in tabview.js

VERIFIED FIXED in Firefox 6

Status

Firefox Graveyard
Panorama
VERIFIED FIXED
6 years ago
a year ago

People

(Reporter: fryn, Assigned: ttaubert)

Tracking

Trunk
Firefox 6

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Reporter)

Description

6 years ago
Upon selecting a group, I get the following error what seems to be every time:

Error: drag.info is null
Source File: chrome://browser/content/tabview.js
Line: 4118

at the line:
this.add(drag.info.$el, options);
in the method:
this.dropOptions.drop = function GroupItem_dropOptions_drop(event) { ... }
(Reporter)

Comment 1

6 years ago
CC'ing mitcho, since he wrote the code.
(Assignee)

Comment 2

6 years ago
Do you have an exact STR? I tried to reproduce but couldn't.
(Assignee)

Comment 3

6 years ago
Can reproduce now. This happens every time you click on a tab to zoom into it.
Assignee: nobody → tim.taubert
Status: NEW → ASSIGNED
(Reporter)

Comment 4

6 years ago
(In reply to comment #3)
> This happens every time you click on a tab to zoom into it.

That's actually what I meant. I must have been half-asleep when filing this.
Thanks for investigating and taking this bug. :)
(Assignee)

Comment 5

6 years ago
Created attachment 525541 [details] [diff] [review]
patch v1
Attachment #525541 - Flags: feedback?(raymond)
Attachment #525541 - Flags: feedback?(raymond) → feedback+
(Assignee)

Updated

6 years ago
Attachment #525541 - Flags: review?(ian)
(Assignee)

Comment 6

6 years ago
Comment on attachment 525541 [details] [diff] [review]
patch v1

Passed try (kind of):

http://tbpl.mozilla.org/?tree=MozillaTry&pusher=tim.taubert@gmx.de&rev=a2426c7ed09f
(Reporter)

Comment 7

6 years ago
(In reply to comment #6)
> Passed try (kind of):
> 
> http://tbpl.mozilla.org/?tree=MozillaTry&pusher=tim.taubert@gmx.de&rev=a2426c7ed09f

Yeah, tryserver results are weird. I'd ignore the Android red though. :P
Comment on attachment 525541 [details] [diff] [review]
patch v1

Nice catch
Attachment #525541 - Flags: review?(ian) → review+
(Assignee)

Comment 9

6 years ago
Created attachment 526101 [details] [diff] [review]
patch for checkin
Attachment #525541 - Attachment is obsolete: true
(Assignee)

Updated

6 years ago
Keywords: checkin-needed
http://hg.mozilla.org/mozilla-central/rev/84b1567baa32
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 6

Comment 11

6 years ago
Verified on Mozilla/5.0 (Windows NT 5.1; rv:6.0a1) Gecko/20110421 Firefox/6.0a1
Status: RESOLVED → VERIFIED
Product: Firefox → Firefox Graveyard
You need to log in before you can comment on or make changes to this bug.