Bug 31427 - Add postTaskToMainThread to ScriptExecutionContext
Summary: Add postTaskToMainThread to ScriptExecutionContext
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebCore Misc. (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC OS X 10.5
: P2 Normal
Assignee: Dmitry Titov
URL:
Keywords:
Depends on:
Blocks: 31317
  Show dependency treegraph
 
Reported: 2009-11-12 12:13 PST by Dmitry Titov
Modified: 2009-11-18 00:41 PST (History)
1 user (show)

See Also:


Attachments
Proposed patch. (5.19 KB, patch)
2009-11-12 12:21 PST, Dmitry Titov
ap: review+
dimich: commit-queue-
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Dmitry Titov 2009-11-12 12:13:17 PST
Moving the code from Document to the protected member of the base class. It can be used as helper for the derived contexts that live on the main thread.

In preparation for GlobalScript (bug 31317)
Comment 1 Dmitry Titov 2009-11-12 12:21:43 PST
Created attachment 43090 [details]
Proposed patch.
Comment 2 Dmitry Titov 2009-11-12 17:18:11 PST
Landed: http://trac.webkit.org/changeset/50919
Comment 3 Eric Seidel (no email) 2009-11-17 21:26:16 PST
There was a style error in this patch, btw.  Guess we need Adam Barth's style bot. :)
Comment 4 Eric Seidel (no email) 2009-11-17 21:26:48 PST
{} around single line if was the style error:
62	    } else { 
 	263	        callOnMainThread(PerformTaskData::performTask, new PerformTaskData(this, task)); 
 	264	    }
Comment 5 Dmitry Titov 2009-11-18 00:41:26 PST
> {} around single line if was the style error:

Thanks!
Will fix while landing a patch for regression (bug 31615)