Closed Bug 720408 Opened 13 years ago Closed 13 years ago

Exception handling message "Tab:ScreenshotData": java.lang.StringIndexOutOfBoundsException

Categories

(Firefox for Android Graveyard :: General, defect, P2)

ARM
Android
defect

Tracking

(firefox11 fixed, firefox12 fixed, fennec11+)

RESOLVED FIXED
Firefox 13
Tracking Status
firefox11 --- fixed
firefox12 --- fixed
fennec 11+ ---

People

(Reporter: aaronmt, Assigned: blassey)

Details

(Keywords: regression)

Attachments

(2 files)

I/GeckoApp( 3910): Got message: Tab:ScreenshotData
E/GeckoApp( 3910): Exception handling message "Tab:ScreenshotData":
E/GeckoApp( 3910): java.lang.StringIndexOutOfBoundsException: length=6; index=22
E/GeckoApp( 3910): 	at java.lang.String.indexAndLength(String.java:589)
E/GeckoApp( 3910): 	at java.lang.String.substring(String.java:1448)
E/GeckoApp( 3910): 	at org.mozilla.gecko.GeckoApp.handleMessage(GeckoApp.java:924)
E/GeckoApp( 3910): 	at org.mozilla.gecko.GeckoAppShell.handleGeckoMessage(GeckoAppShell.java:1616)
E/GeckoApp( 3910): 	at org.mozilla.gecko.GeckoAppShell.nativeRun(Native Method)
E/GeckoApp( 3910): 	at org.mozilla.gecko.GeckoAppShell.nativeRun(Native Method)
E/GeckoApp( 3910): 	at org.mozilla.gecko.GeckoAppShell.runGecko(GeckoAppShell.java:483)
E/GeckoApp( 3910): 	at org.mozilla.gecko.GeckoThread.run(GeckoThread.java:104)


--
Samsung Nexus S (Android 4.0.3)
Mozilla/5.0 (Android; Linux armv7l; rv:12.0a1) Gecko/20120123 Firefox/12.0a1 Fennec/12.0a1
Assignee: nobody → blassey.bugs
tracking-fennec: --- → 11+
Priority: -- → P2
Attached patch patchSplinter Review
Attachment #592984 - Flags: review?(mark.finkle)
Comment on attachment 592984 [details] [diff] [review]
patch

Why would we get into this state? Is an empty string coming back?

If you know, can you add a comment above the new if check?
Attachment #592984 - Flags: review?(mark.finkle) → review+
(In reply to Mark Finkle (:mfinkle) from comment #2)
> Comment on attachment 592984 [details] [diff] [review]
> patch
> 
> Why would we get into this state? Is an empty string coming back?
> 
> If you know, can you add a comment above the new if check?

not an empty string, but looks like empty data from the log. I suspect that getting the data from the canvas failed (possibly because drawing to it failed) and we're just getting a string with "data:".
Target Milestone: --- → Firefox 13
https://hg.mozilla.org/mozilla-central/rev/396945d35979
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Comment on attachment 592984 [details] [diff] [review]
patch

[Approval Request Comment]
Regression caused by (bug #): 
User impact if declined: 
Low volume crash from unknown cause. We weren't able to reproduce
Testing completed (on m-c, etc.): 
Risk to taking this patch (and alternatives if risky):
String changes made by this patch:
Attachment #592984 - Flags: approval-mozilla-beta?
Attachment #592984 - Flags: approval-mozilla-aurora?
Comment on attachment 592984 [details] [diff] [review]
patch

[Triage Comment]
Mobile only - approved for Aurora 12 and Beta 11.
Attachment #592984 - Flags: approval-mozilla-beta?
Attachment #592984 - Flags: approval-mozilla-beta+
Attachment #592984 - Flags: approval-mozilla-aurora?
Attachment #592984 - Flags: approval-mozilla-aurora+
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: