Last Comment Bug 648929 - frequently getting is null error in tabview.js
: frequently getting is null error in tabview.js
Product: Firefox Graveyard
Classification: Graveyard
Component: Panorama (show other bugs)
: Trunk
: All All
: -- normal
: Firefox 6
Assigned To: Tim Taubert [:ttaubert]
Depends on:
  Show dependency treegraph
Reported: 2011-04-10 18:48 PDT by Frank Yan (:fryn)
Modified: 2016-04-12 14:00 PDT (History)
5 users (show)
See Also:
QA Whiteboard:
Iteration: ---
Points: ---

patch v1 (792 bytes, patch)
2011-04-12 16:14 PDT, Tim Taubert [:ttaubert]
ian: review+
raymond: feedback+
Details | Diff | Splinter Review
patch for checkin (816 bytes, patch)
2011-04-14 13:56 PDT, Tim Taubert [:ttaubert]
no flags Details | Diff | Splinter Review

Description Frank Yan (:fryn) 2011-04-10 18:48:51 PDT
Upon selecting a group, I get the following error what seems to be every time:

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

at the line:
this.add($el, options);
in the method:
this.dropOptions.drop = function GroupItem_dropOptions_drop(event) { ... }
Comment 1 Frank Yan (:fryn) 2011-04-12 02:04:34 PDT
CC'ing mitcho, since he wrote the code.
Comment 2 Tim Taubert [:ttaubert] 2011-04-12 04:36:53 PDT
Do you have an exact STR? I tried to reproduce but couldn't.
Comment 3 Tim Taubert [:ttaubert] 2011-04-12 14:50:36 PDT
Can reproduce now. This happens every time you click on a tab to zoom into it.
Comment 4 Frank Yan (:fryn) 2011-04-12 14:55:16 PDT
(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. :)
Comment 5 Tim Taubert [:ttaubert] 2011-04-12 16:14:50 PDT
Created attachment 525541 [details] [diff] [review]
patch v1
Comment 6 Tim Taubert [:ttaubert] 2011-04-13 13:43:25 PDT
Comment on attachment 525541 [details] [diff] [review]
patch v1

Passed try (kind of):
Comment 7 Frank Yan (:fryn) 2011-04-13 13:45:20 PDT
(In reply to comment #6)
> Passed try (kind of):

Yeah, tryserver results are weird. I'd ignore the Android red though. :P
Comment 8 Ian Gilman [:iangilman] 2011-04-14 09:41:17 PDT
Comment on attachment 525541 [details] [diff] [review]
patch v1

Nice catch
Comment 9 Tim Taubert [:ttaubert] 2011-04-14 13:56:39 PDT
Created attachment 526101 [details] [diff] [review]
patch for checkin
Comment 10 :Ms2ger (⌚ UTC+1/+2) 2011-04-17 05:54:00 PDT
Comment 11 George Carstoiu 2011-04-22 02:43:51 PDT
Verified on Mozilla/5.0 (Windows NT 5.1; rv:6.0a1) Gecko/20110421 Firefox/6.0a1

Note You need to log in before you can comment on or make changes to this bug.