Bug 123139 - Remote Layer Tree: Support hardware accelerated filters
Summary: Remote Layer Tree: Support hardware accelerated filters
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit2 (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Tim Horton
URL:
Keywords:
Depends on: 123330
Blocks:
  Show dependency treegraph
 
Reported: 2013-10-21 23:29 PDT by Tim Horton
Modified: 2013-10-25 02:49 PDT (History)
7 users (show)

See Also:


Attachments
patch (27.18 KB, patch)
2013-10-22 00:07 PDT, Tim Horton
andersca: review+
eflews.bot: commit-queue-
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Tim Horton 2013-10-21 23:29:13 PDT
I took the approach of serializing FilterOperation and FilterOperations and using existing PlatformCAFiltersMac code to apply them on the UI process side.

I haven't added custom filter support yet, that might be slightly more tricky.
Comment 1 Tim Horton 2013-10-22 00:07:10 PDT
Created attachment 214823 [details]
patch
Comment 2 Tim Horton 2013-10-22 00:07:46 PDT
(In reply to comment #0)
> I took the approach of serializing FilterOperation and FilterOperations and using existing PlatformCAFiltersMac code to apply them on the UI process side.
> 
> I haven't added custom filter support yet, that might be slightly more tricky.

Actually it looks like we don't have accelerated custom filters so we don't need this.
Comment 3 EFL EWS Bot 2013-10-22 00:12:59 PDT
Comment on attachment 214823 [details]
patch

Attachment 214823 [details] did not pass efl-wk2-ews (efl-wk2):
Output: http://webkit-queues.appspot.com/results/8848227
Comment 4 Anders Carlsson 2013-10-22 07:06:51 PDT
Comment on attachment 214823 [details]
patch

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

> Source/WebKit2/Shared/WebCoreArgumentCoders.cpp:1183
> +static void encodeFilterOperation(ArgumentEncoder& encoder, FilterOperation* filter)

I think FilterOperation should be a const reference.

> Source/WebKit2/Shared/WebCoreArgumentCoders.cpp:1318
> +        filters.operations().append(filter);

Can use std::move(filter) here to avoid ref churn.
Comment 5 Tim Horton 2013-10-22 12:24:01 PDT
http://trac.webkit.org/changeset/157803