[Webkit-unassigned] [Bug 62346] [EFL][WK2] Add dummy files in WebProcess/WebCoreSupport/efl

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=62346





--- Comment #2 from Leandro Pereira <leandro at profusion.mobi>  2011-06-09 06:48:49 PST ---
(From update of attachment 96528)
View in context: https://bugs.webkit.org/attachment.cgi?id=96528&action=review

As with other WebKit2 patches, I'm mostly reviewing for style issues, as I'm not familiar with WebKit2 yet.

WebEditorClientEfl.cpp doesn't use anything EFL-related (and I couldn't see any EFL-specific behaviour there, too). Is it possible to share this code with other WebKit2 ports?

> Source/WebKit2/WebProcess/WebCoreSupport/efl/WebEditorClientEfl.cpp:44
> +bool WebEditorClient::executePendingEditorCommands(Frame* frame, Vector<WTF::String> pendingEditorCommands, bool allowTextInsertion)
> +{
> +    Vector<Editor::Command> commands;
> +    for (size_t i = 0; i < pendingEditorCommands.size(); i++) {

I'd use Vector::iterator here (Vector::begin(), Vector::end(), etc), for both pendingEditorCommands and commands.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.