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

Issue 599378 link

Starred by 6 users
Status: Fixed
Owner:
pdr@chromium.org
Closed: Feb 2017
Cc:
pdr@chromium.org
█ amineer@chromium.org
wangxianzhu@chromium.org
e...@chromium.org
changwan@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 1
Type: Bug



Sign in to add a comment

Crash in debug build - Assertion Failure

Project Member Reported by ramy...@samsung.com, Mar 31 2016 
Version: <Chromuim 51.0.2695.0, always?>
OS: <Android 5.1.1>

What steps will reproduce the problem?
(1)Open www.kill.com in landscape mode.
(2)Once the page is loaded, change orientation to portrait.
(3)Browser crash is observed

Failing in below assert.
ASSERT(!m_knownToHaveNoOverflowAndNoFallbackFonts || (fallbackFonts.isEmpty() && glyphOverflow.isApproximatelyZero()));

What is the expected output?
Browser should not crash

What do you see instead?
Browser crashes due to assert failure.

Please use labels and text to provide additional information.

Comment 1 by pdr@chromium.org, Mar 31 2016

Components: Blink>TextAutosize
I can reproduce this in devtools with emulation enabled.

Looks like it is caused by text autosizing increasing the font size. It would be good to fix this, but it's correctly filed as a P3.

Comment 2 by pdr@chromium.org, Mar 31 2016

Status: Available (was: Untriaged)

Comment 3 by wangxianzhu@chromium.org, May 13 2016

Cc: wangxianzhu@chromium.org
 Issue 611914  has been merged into this issue.

Comment 4 by pdr@chromium.org, Aug 19 2016

Cc: changwan@chromium.org neis@chromium.org hablich@chromium.org amineer@chromium.org
 Issue 638841  has been merged into this issue.

Comment 5 by hablich@chromium.org, Aug 19 2016

Cc: -hablich@chromium.org -neis@chromium.org
Not V8 related, so removing us from CC.

Comment 6 by changwan@chromium.org, Aug 19 2016

Labels: -Pri-3 Pri-1
Copying priority=1 from  issue 638841 .

Comment 7 by w...@chromium.org, Oct 7 2016 

Ping. This crash is affecting my ability to debug another issue. Can we remove the assert until it's fixed?

Comment 8 by skobes@chromium.org, Feb 14 2017

Owner: fmalita@chromium.org
Status: Assigned (was: Available)
Still repros.

It looks like this assertion was first added in http://crrev.com/416623002.  @fmalita do you mind taking a look?

Comment 9 by fmalita@chromium.org, Feb 17 2017

Owner: pdr@chromium.org
Status: Fixed (was: Assigned)
Sounds like https://codereview.chromium.org/2683553002/ addressed this issue, and I can no longer repro.

Marking as fixed.
►

Sign in to add a comment