Closed Bug 446509 Opened 12 years ago Closed 12 years ago

assertions while panning on cnet site

Categories

(Firefox for Android Graveyard :: Panning/Zooming, defect)

defect
Not set

Tracking

(Not tracked)

VERIFIED FIXED
fennec1.0m6

People

(Reporter: dougt, Assigned: Gavin)

References

()

Details

Attachments

(1 file)

1) load http://news.cnet.com/8301-1023_3-9991979-93.html,
2) try to pan.

I get this on the console:

************************************************************
* Call to xpconnect wrapped JSObject produced this error:  *
[Exception... "'[JavaScript Error: "aElement is null" {file: "chrome://browser/content/deckbrowser.xml" line: 211}]' when calling method: [nsIDOMEventListener::handleEvent]"  nsresult: "0x80570021 (NS_ERROR_XPC_JAVASCRIPT_ERROR_WITH_DETAILS)"  location: "<unknown>"  data: yes]
************************************************************
************************************************************
* Call to xpconnect wrapped JSObject produced this error:  *
[Exception... "'[JavaScript Error: "aElement is null" {file: "chrome://browser/content/deckbrowser.xml" line: 211}]' when calling method: [nsIDOMEventListener::handleEvent]"  nsresult: "0x80570021 (NS_ERROR_XPC_JAVASCRIPT_ERROR_WITH_DETAILS)"  location: "<unknown>"  data: yes]
************************************************************

I tried to reproduce, and couldn't.
Attached patch patchSplinter Review
I'm not sure how this can happen, but I guess we should protect against it...
Attachment #330747 - Flags: review?(mark.finkle)
Status: NEW → ASSIGNED
OS: Mac OS X → All
Hardware: PC → All
Target Milestone: --- → Fennec M6
Comment on attachment 330747 [details] [diff] [review]
patch

Maybe you could fallback to the <body>?
Attachment #330747 - Flags: review?(mark.finkle) → review+
Zooming to the <body> wouldn't be very useful in the general case, would it? hopefully this is pretty exceptional so I don't think we need to worry about it too much.
http://hg.mozilla.org/index.cgi/mobile-browser/rev/9359ccb169ba
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
verified with beta3
Status: RESOLVED → VERIFIED
Component: General → Panning/Zooming
You need to log in before you can comment on or make changes to this bug.