Bug 150592

Summary:

[GTK] Use RunLoop::Timer instead of GMainLoopSource

Product:

WebKit

Reporter:

Carlos Garcia Campos <cgarcia>

Component:

WebKitGTK

Assignee:

Nobody <webkit-unassigned>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

bugs-noreply, zan

Priority:

Keywords:

Gtk

Version:

WebKit Local Build

Hardware:

Unspecified

OS:

Unspecified

Bug Depends on:

150590

Bug Blocks:

Attachments:

Description	Flags
Patch	zan: review+

Description Carlos Garcia Campos 2015-10-27 10:14:51 PDT

For now only in the cases where the default priority is used.

Comment 1 Carlos Garcia Campos 2015-10-27 10:17:09 PDT

Created attachment 264135 [details]
Patch

Comment 2 Zan Dobersek 2015-10-28 00:57:18 PDT

Comment on attachment 264135 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=264135&action=review

> Source/WebKit2/UIProcess/gtk/GestureController.cpp:130
> +    dragGesture->m_longPressTimeout.startOneShot(delay / 1000.0);

Not in the scope of this patch, but RunLoop::Timer::startOneShot() sure could use an overload that takes in std::chrono::something.

Comment 3 Carlos Garcia Campos 2015-10-30 05:13:05 PDT

Committed r191787: <http://trac.webkit.org/changeset/191787>