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

RESOLVED FIXED in Firefox 11

Status

()

Firefox for Android
General
P2
normal
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: aaronmt, Assigned: blassey)

Tracking

({regression})

unspecified
Firefox 13
ARM
Android
regression
Points:
---

Firefox Tracking Flags

(firefox11 fixed, firefox12 fixed, fennec11+)

Details

Attachments

(2 attachments)

(Reporter)

Description

5 years ago
Created attachment 590773 [details]
Raw log (Nightly 01/23)

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)

Updated

5 years ago
Assignee: nobody → blassey.bugs
tracking-fennec: --- → 11+
Priority: -- → P2
Created attachment 592984 [details] [diff] [review]
patch
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:".

Updated

5 years ago
Target Milestone: --- → Firefox 13

Comment 4

5 years ago
https://hg.mozilla.org/mozilla-central/rev/396945d35979
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
(Reporter)

Updated

5 years ago
Keywords: regressionwindow-wanted
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 6

5 years ago
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+

Comment 7

5 years ago
https://hg.mozilla.org/releases/mozilla-beta/rev/51389da3984a
status-firefox11: --- → fixed
https://hg.mozilla.org/releases/mozilla-aurora/rev/cc433916cd1d
status-firefox12: affected → fixed
You need to log in before you can comment on or make changes to this bug.