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

Issue 629021 link

Starred by 5 users

Issue metadata

Status: Duplicate
Merged: issue 627623
Owner:
Last visit > 30 days ago
Closed: Jul 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug



Sign in to add a comment

Hangouts App: sad mac when two-finger scrolling chat window

Project Member Reported by djkurtz@chromium.org, Jul 18 2016

Issue description

Chrome Version: 54.0.2792.0
Chrome OS Version: 8577.0.0
Chrome OS Platform: veyron_minnie

Also repro'ed on: elm 8589.0

Steps To Reproduce:
(1) Open Hangouts App
(2) Click a contact to load chat history
(3) Two finger scroll chat pane

Expected Result:

Chat content scrolls.

Actual Result:

Content turns black with unhappy mac face.

How frequently does this problem reproduce? (Always, sometimes, hard to
reproduce?)

Sometimes.

What is the impact to the user, and is there a workaround? If so, what is
it?

Scrolling in Hangouts is unreliable.

Please provide any additional information below. Attach a screen shot or
log if possible.

On veyron_minnie 8577.0.0 /var/log/ui/ui.LATEST has:

** Signal 11 from trusted code: pc=b6e0e88e
NaCl process exited with status 62720 (0xf500)
_sys_cr_finished
System crash-reporter failed to process crash report.
Report Id:


On self-built elm 8589.0 w/ USE="-gn" chromeos-chrome, /var/log/ui/ui.LATEST has:

==> /var/log/ui/ui.LATEST <==
[1:3:0718/173324:FATAL:fling_curve.cc(48)] Check failed: max_start_velocity > 0 (0 vs. 0)
#0 0x0000f18b42ee <unknown>
#1 0x0000f18c6cda <unknown>
#2 0x0000f618bcd4 <unknown>
#3 0x0000f5ddd02c <unknown>
#4 0x0000f5d8fbd4 <unknown>
#5 0x0000f5e0c840 <unknown>
#6 0x0000f5e0d14e <unknown>
#7 0x0000f5e0e228 <unknown>
#8 0x0000f446c9b6 <unknown>
#9 0x0000f446b8f0 <unknown>
#10 0x0000f19246cc <unknown>
#11 0x0000f18cd474 <unknown>
#12 0x0000f18cd74e <unknown>
#13 0x0000f18ceb32 <unknown>
#14 0x0000f18cee78 <unknown>
#15 0x0000f18f6230 <unknown>
#16 0x0000f18cc96c <unknown>
#17 0x0000f190d7ac <unknown>
#18 0x0000f190a75e <unknown>

Received signal 6
#0 0x0000f18b42ee <unknown>
#1 0x0000f18b4680 <unknown>
#2 0x0000f082c1b0 <unknown>
#3 0x0000f081b636 <unknown>
#4 0x0000f082b324 gsignal
[end of stack trace]
 
Cc: mtomasz@chromium.org khmel@chromium.org
Labels: -Pri-3 M-54 Pri-1
Owner: xiy...@chromium.org
Last time ( issue 615512 ), I think xiyuan fixed a similar issue.  Perhaps this one is related.

Comment 2 by xiy...@chromium.org, Jul 18 2016

Mergedinto: 628525
Status: Duplicate (was: Available)
The FATAL line is indeed the problem.

Comment 3 by sahel@chromium.org, Jul 19 2016

This stack trace might help:

Happens on ChromeOS too with touch pad two-finger scrolling on hangout window.

Program received signal SIGILL, Illegal instruction.
[Switching to Thread 13819]
ui::FlingCurve::FlingCurve (this=0x30c3d4ab92c0, velocity=..., start_timestamp=...) at ../../ui/events/gestures/fling_curve.cc:48
48        CHECK_GT(max_start_velocity, 0);
(gdb) bt
#0  ui::FlingCurve::FlingCurve (this=0x30c3d4ab92c0, velocity=..., start_timestamp=...) at ../../ui/events/gestures/fling_curve.cc:48
#1  0x00007f9563307d3c in CreateDefaultPlatformCurve (initial_velocity=...) at ../../ui/events/gestures/blink/web_gesture_curve_impl.cc:46
#2  ui::WebGestureCurveImpl::CreateFromDefaultPlatformCurve (initial_velocity=..., initial_offset=..., on_main_thread=<optimized out>)
    at ../../ui/events/gestures/blink/web_gesture_curve_impl.cc:60
#3  0x00007f95632a221c in content::BlinkPlatformImpl::createFlingAnimationCurve (this=<optimized out>, device_source=<optimized out>, velocity=..., cumulative_scroll=...)
    at ../../content/child/blink_platform_impl.cc:846
#4  0x00007f9563343e33 in ui::InputHandlerProxy::HandleGestureFlingStart (this=this@entry=0x30c3cf1cf918, gesture_event=...) at ../../ui/events/blink/input_handler_proxy.cc:733
#5  0x00007f9563344a9b in ui::InputHandlerProxy::HandleInputEvent (this=0x30c3cf1cf918, event=...) at ../../ui/events/blink/input_handler_proxy.cc:346
#6  0x00007f9563346293 in ui::InputHandlerProxy::HandleInputEventWithLatencyInfo (this=0x30c3cf1cf918, event=..., latency_info=latency_info@entry=0x7f955436d250)
    at ../../ui/events/blink/input_handler_proxy.cc:278
#7  0x00007f9561191568 in content::InputHandlerManager::HandleInputEvent (this=0x30c3ceea8c00, routing_id=13, input_event=0x30c3cfede638, latency_info=0x7f955436d250)
    at ../../content/renderer/input/input_handler_manager.cc:231
#8  0x00007f956118fea4 in Run (args#2=0x7f955436d250, args#1=0x30c3cfede638, args#0=13, this=0x30c3cee18a50) at ../../base/callback.h:389
#9  content::InputEventFilter::ForwardToHandler (this=0x30c3cee18a00, message=...) at ../../content/renderer/input/input_event_filter.cc:194
#10 0x00007f955d70ad41 in Run (this=0x7f955436db88) at ../../base/callback.h:389
#11 base::debug::TaskAnnotator::RunTask (this=0x30c3cee20180, queue_function=<optimized out>, pending_task=...) at ../../base/debug/task_annotator.cc:51
#12 0x00007f955d6b0454 in base::MessageLoop::RunTask (this=0x30c3cee20000, pending_task=...) at ../../base/message_loop/message_loop.cc:494
#13 0x00007f955d6b085e in base::MessageLoop::DeferOrRunPendingTask (this=0x30c3cee20000, pending_task=...) at ../../base/message_loop/message_loop.cc:503
#14 0x00007f955d6b2a58 in base::MessageLoop::DoWork (this=0x30c3cee20000) at ../../base/message_loop/message_loop.cc:627
#15 0x00007f955d6b2e09 in base::MessagePumpDefault::Run (this=0x30c3ceefcc60, delegate=0x30c3cee20000) at ../../base/message_loop/message_pump_default.cc:35
#16 0x00007f955d6cdbca in base::RunLoop::Run (this=0x7f955436dcc0) at ../../base/run_loop.cc:35
#17 0x00007f955d6afb49 in base::MessageLoop::Run (this=<optimized out>) at ../../base/message_loop/message_loop.cc:295
#18 0x00007f955d6ed142 in base::Thread::ThreadMain (this=0x30c3cee488c0) at ../../base/threading/thread.cc:256
#19 0x00007f955d6e9666 in base::(anonymous namespace)::ThreadFunc (params=<optimized out>) at ../../base/threading/platform_thread_posix.cc:70
#20 0x00007f955c0d2307 in start_thread (arg=0x7f955436e700) at pthread_create.c:309
#21 0x00007f955ab905cd in clone () at ../sysdeps/unix/sysv/linux/x86_64/clone.S:111
(gdb) p max_start_velocity 
$1 = 0

Comment 4 by sahel@chromium.org, Jul 19 2016

Status: Assigned (was: Duplicate)

Comment 5 by xiy...@chromium.org, Jul 19 2016

Cc: xiy...@chromium.org
Owner: asimjour@chromium.org
De-duped from  issue 628525  since sahel@ wants to focus on event_sender.cc validation for the test and does not think that would fix the issue on chromeos.

Pass to asimjour@ per aldr@:
Adding Amir who may have been looking into fling generation code on Chrome OS recently

fyi, I have added some log in MakeWebGestureEventFromAuraEvent [1] and confirmed that x_offset() and y_offset() of ET_SCROLL_FLING_START typed ui::ScrollEvent are both 0.

https://cs.chromium.org/chromium/src/content/browser/renderer_host/web_input_event_aura.cc?rcl=0&l=165-167

Comment 6 by khmel@chromium.org, Jul 19 2016

I am currently fixing 627623 and it seems these bugs are dup. I was able to repro crash in Hangout as described in https://chromium-review.googlesource.com/#/c/361789. With that CL no more crash is observed for both bugs (the same for Files.app).
Cc: warx@chromium.org derat@chromium.org puneetster@chromium.org w...@chromium.org mark@chromium.org
 Issue 628829  has been merged into this issue.
Can we confirm that the fix for  bug 627623  fixes this as well?
Labels: -M-54 ReleaseBlock-Beta M-53

Comment 10 by khmel@chromium.org, Jul 20 2016

I tested and CL for  bug 627623   did fix. However CL was banned since it affects 2-finger Chrome navigation. Need alternative fix that is safe for Chrome navigation and for our bugs. I can confirm only that source of this problems is the same (Chrome OS Scroll manager issues empty fling event that is not compatible with all Chrome parts).
Mergedinto: -628525 627623
Status: Duplicate (was: Assigned)

Sign in to add a comment