New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 22913 link

Starred by 3 users

Issue metadata

Status: Fixed
Owner: ----
Closed: Oct 2009
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 1
Type: Bug
M-3

Restricted
  • Only users with EditIssue permission may comment.



Sign in to add a comment

Site crashes Chrome

Reported by remy.ra...@gmail.com, Sep 24 2009

Issue description

Chrome Version       : 4.0.211.7 (Build officiel 26751), on Windows XP
URLs (if applicable) : http://james.padolsey.com/demos/grayscale/
Other browsers tested:
     Safari 4: OK
         IE 8: OK

What steps will reproduce the problem?
1. Open http://james.padolsey.com/demos/grayscale/
2. The Chrome tab crashes and shows the familiar dead folder

What is the expected result?
Not crashing

What happens instead?
Crashing


 
Labels: -Area-Misc Area-WebKit Crash
Status: Untriaged
thanks for the report!

"aw snap" confirmed using chrome 4.0.219.4 (Developer Build 27360) under windows
doesn't crash with Safari using latest WebKit nightly

Comment 2 by karen@chromium.org, Sep 28 2009

what a neat demo! yes the site says " It only works slightly in Safari<4 and Chrome 
because of their lacking support of CanvasContext.getImageData (because of this, 
images cannot be grayscaled)." dimitri any chance this is relatively easy to add? It 
doesn't seem urgent to me, more like a nice to have. an appears to be an issue with our 
port of webkit since it works on the most recent Safari.
Labels: -Pri-2 Pri-1 JavaScript
Status: Assigned
There are really two unrelated issues here:

1) the crash and
2) CanvasContext.getImageData support

While we may not implement a feature, we should never crash. A crash is always a P1. Unimplemented feature can be 
P2 or P3.

Looking at the stack trace of the crash, this site provides an excellent repro of this: http://crash/reportdetail?
reportid=1c0593f757c40cde. Thanks for the report, remy.rakic.

The stack trace is in V8, so assigning to ager.

As for the getImageData support, we do indeed support it. Author's note is out of date, probably referring to Chrome 
1.

Comment 4 by karen@chromium.org, Sep 29 2009

thanks! once the crash is fixed, i will test the site and email the author so he can 
change the note.

Comment 5 by ager@chromium.org, Sep 29 2009

Fixed in V8 r2985.

Merged to V8 version 1.2 branch in r2986 and to V8 trunk in r2988.

Comment 7 by bugdro...@gmail.com, Sep 30 2009

The following revision refers to this bug:
    http://src.chromium.org/viewvc/chrome?view=rev&revision=27604 

------------------------------------------------------------------------
r27604 | sgjesse@chromium.org | 2009-09-30 04:17:18 -0700 (Wed, 30 Sep 2009) | 7 lines
Changed paths:
   M http://src.chromium.org/viewvc/chrome/trunk/src/DEPS?r1=27604&r2=27603

Update V8 to version 1.3.13.4.

This version contains a fix for a V8 bug (actually in version 1.3.13.3) and adds some missing handle scopes in the API.

BUG= 22913 
TEST=none
Review URL: http://codereview.chromium.org/243044
------------------------------------------------------------------------

Labels: Mstone-3
This site immediately causes my version of Chrome to silently Crash.
which version of chrome are you using? It actually works perfectly for me. I was 
planning on contacting the site owner to ask him to remove the warning that it doesn't 
work in chrome.
This is currently only fixed in 4.0 trunk. For stable 3.0 it requires that V8 is 
updated to 1.2.14.20.
Add label: Fixed.

Comment 14 by mal@google.com, Oct 21 2009

Status: Fixed
V8 r2986 updates 1.2 to 1.2.14.20. That is the version shipping with Chrome 3.0.195.27.
Project Member

Comment 15 by bugdroid1@chromium.org, Oct 12 2012

Labels: Restrict-AddIssueComment-Commit
Owner: ----
This issue has been closed for some time. No one will pay attention to new comments.
If you are seeing this bug or have new data, please click New Issue to start a new bug.
Project Member

Comment 16 by bugdroid1@chromium.org, Mar 10 2013

Labels: -Area-WebKit -Crash -Mstone-3 Cr-Content Cr-Stability M-3
Project Member

Comment 17 by bugdroid1@chromium.org, Mar 13 2013

Labels: -Restrict-AddIssueComment-Commit Restrict-AddIssueComment-EditIssue
Project Member

Comment 18 by bugdroid1@chromium.org, Apr 6 2013

Labels: -Cr-Content Cr-Blink

Sign in to add a comment