Bug 115309

Summary:

Replace uses of StorageTask with StorageThread::dispatch and WTF::bind

Product:

WebKit

Reporter:

Anders Carlsson <andersca>

Component:

New Bugs

Assignee:

Anders Carlsson <andersca>

Status:

RESOLVED FIXED

Severity:

Normal

Priority:

Version:

528+ (Nightly build)

Hardware:

Unspecified

OS:

Unspecified

Attachments:

Description	Flags
Patch	kling: review+

Anders Carlsson

Reported 2013-04-27 13:07:11 PDT

Replace uses of StorageTask with StorageThread::dispatch and WTF::bind

Attachments
Patch (10.45 KB, patch) 2013-04-27 13:07 PDT, Anders Carlsson	kling: review+	Details Formatted Diff Diff
View All Add attachment proposed patch, testcase, etc.

Anders Carlsson

Comment 1 2013-04-27 13:07:51 PDT

Created attachment 199910 [details] Patch

Andreas Kling

Comment 2 2013-04-27 13:25:24 PDT

Comment on attachment 199910 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=199910&action=review r=me > Source/WebCore/storage/StorageTask.cpp:94 > break; This line should also be removed. > Source/WebCore/storage/StorageTracker.cpp:391 > + m_thread->dispatch(bind(&StorageTracker::syncDeleteAllOrigins, this)); Dispatching a "sync" method sure looks odd.

Anders Carlsson

Comment 3 2013-04-27 13:27:27 PDT

Committed r149242: <http://trac.webkit.org/changeset/149242>

Note You need to log in before you can comment on or make changes to this bug.