Bug 100558 - [CSS Shaders] Add the ValidatedCustomFilterOperation class
Summary: [CSS Shaders] Add the ValidatedCustomFilterOperation class
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Layout and Rendering (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Alexandru Chiculita
URL:
Keywords:
Depends on:
Blocks: 100533
  Show dependency treegraph
 
Reported: 2012-10-26 13:54 PDT by Alexandru Chiculita
Modified: 2012-10-30 16:34 PDT (History)
13 users (show)

See Also:


Attachments
Patch V1 (21.28 KB, patch)
2012-10-26 14:44 PDT, Alexandru Chiculita
buildbot: commit-queue-
Details | Formatted Diff | Diff
Patch V2 (21.39 KB, patch)
2012-10-26 15:47 PDT, Alexandru Chiculita
dino: review+
webkit.review.bot: commit-queue-
Details | Formatted Diff | Diff
Patch V3 (21.57 KB, patch)
2012-10-30 16:28 PDT, Alexandru Chiculita
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Alexandru Chiculita 2012-10-26 13:54:49 PDT
Add the ValidatedCustomFilterOperation to the project files.

Look in the parent bug for more details: https://bugs.webkit.org/show_bug.cgi?id=100533
Comment 1 Alexandru Chiculita 2012-10-26 14:44:23 PDT
Created attachment 171017 [details]
Patch V1
Comment 2 Build Bot 2012-10-26 15:04:03 PDT
Comment on attachment 171017 [details]
Patch V1

Attachment 171017 [details] did not pass mac-ews (mac):
Output: http://queues.webkit.org/results/14608218
Comment 3 Alexandru Chiculita 2012-10-26 15:47:28 PDT
Created attachment 171034 [details]
Patch V2
Comment 4 Dean Jackson 2012-10-30 03:46:00 PDT
Comment on attachment 171034 [details]
Patch V2

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

> Source/WebCore/platform/graphics/filters/ValidatedCustomFilterOperation.h:60
> +    virtual bool operator==(const FilterOperation& o) const
> +    {
> +        if (!isSameType(o))
> +            return false;
> +
> +        return true;
> +    }

While this is unimportant, I think we should return false while this remains unimplemented. I'm happy for you to ignore me here.
Comment 5 Alexandru Chiculita 2012-10-30 09:15:26 PDT
Comment on attachment 171034 [details]
Patch V2

Thanks Dean! I will implement that function in the following patch anyway, so I'm just adding cq+ for this patch.
Comment 6 WebKit Review Bot 2012-10-30 09:19:30 PDT
Comment on attachment 171034 [details]
Patch V2

Rejecting attachment 171034 [details] from commit-queue.

Failed to run "['/mnt/git/webkit-commit-queue/Tools/Scripts/webkit-patch', '--status-host=queues.webkit.org', '-..." exit_code: 2

Last 500 characters of output:
ium.cpp
patching file Source/WebCore/platform/graphics/filters/FilterOperation.h
patching file Source/WebCore/platform/graphics/filters/FilterOperations.cpp
patching file Source/WebCore/platform/graphics/filters/ValidatedCustomFilterOperation.cpp
patching file Source/WebCore/platform/graphics/filters/ValidatedCustomFilterOperation.h

Failed to run "[u'/mnt/git/webkit-commit-queue/Tools/Scripts/svn-apply', u'--force', u'--reviewer', u'Dean Jacks..." exit_code: 1 cwd: /mnt/git/webkit-commit-queue

Full output: http://queues.webkit.org/results/14620878
Comment 7 Alexandru Chiculita 2012-10-30 11:15:36 PDT
Landed manually in http://trac.webkit.org/changeset/132927 .
Comment 8 Vincent Scheib 2012-10-30 14:55:50 PDT
Error:
..\platform\graphics\filters\ValidatedCustomFilterOperation.cpp(51):error C2027: use of undefined type 'WebCore::FractionalLayoutSize'
         C:\b\build\slave\webkit-win-latest-rel\build\src\third_party\WebKit\Source\WebCore\rendering\LayoutTypes.h(44) : see declaration of 'WebCore::FractionalLayoutSize'

build:
http://build.chromium.org/p/chromium.webkit/builders/WebKit%20Win%20Builder/builds/32028
and other windows builders.
Comment 9 Vincent Scheib 2012-10-30 14:57:31 PDT
Rolled out in:
http://trac.webkit.org/changeset/132947
Comment 10 Alexandru Chiculita 2012-10-30 16:28:58 PDT
Created attachment 171545 [details]
Patch V3

Adding the FractionalLayoutSize.h in the ValidatedCustomFilterOperation
Comment 11 Alexandru Chiculita 2012-10-30 16:34:26 PDT
(In reply to comment #10)
> Created an attachment (id=171545) [details]
> Patch V3
> 
> Adding the FractionalLayoutSize.h in the ValidatedCustomFilterOperation

Landed again in http://trac.webkit.org/changeset/132960 .