Add temporary FilterType default branches in preparation of FilterTypeSaturatingBrightness

Add temporary FilterType default branches in preparation of FilterTypeSaturatingBrightness

To prevent compile warnings (treated as errors) when the new FilterType lands in WK, this patch
adds temporary default branches to ParamTraits<WebKit::WebFilterOperation>::*.

R=senorblanco@google.com

BUG=

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=181742

[Stephen White, 2013-02-08 20:53:02]

On 2013/02/08 20:51:23, fmalita wrote:

LGTM (although you'll probably need an OWNER).

[commit-bot: I haz the power, 2013-02-08 20:53:54]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/fmalita@google.com/12209071/1

[commit-bot: I haz the power, 2013-02-08 20:53:56]

Presubmit check for 12209071-1 failed and returned exit status 1.

INFO:root:Found 1 file(s).

Running presubmit commit checks ...
Running /b/commit-queue/workdir/chromium/PRESUBMIT.py

** Presubmit Messages **
If this change has an associated bug, add BUG=[bug number].

** Presubmit ERRORS **
Missing LGTM from an OWNER for files in these directories:
    content/common

Presubmit checks took 1.0s to calculate.

[fmalita_google_do_not_use, 2013-02-08 22:02:40]

Chris, Julien - can you please take a look at this?

[jln (very slow on Chromium), 2013-02-09 04:52:19]

Hi,

we're owner in content/common for the Linux sandbox. Please, get an OWNER of
content/ to do this review.

[piman, 2013-02-11 19:58:16]

LGTM, just make sure to revert (TBR=me) after the WK change has rolled.

[danakj, 2013-02-11 21:34:33]

You'll need a security review for *_messages.cc file changes (see the OWNERS
file)

[fmalita_google_do_not_use, 2013-02-11 22:58:09]

On 2013/02/11 21:34:33, danakj wrote:
> You'll need a security review for *_messages.cc file changes (see the OWNERS
> file)

Hmm, I can only see *_messages.h entries in the OWNERS file (for IPC message
changes) - am I missing something?

Either way, it would make sense to get such a review for the final patch, not
this temp build fix (which should get reverted as soon as WK r142496 rolls in
and https://codereview.chromium.org/12033075/ lands).

[danakj, 2013-02-11 23:14:41]

On Mon, Feb 11, 2013 at 2:58 PM, <fmalita@google.com> wrote:

> On 2013/02/11 21:34:33, danakj wrote:
>
>> You'll need a security review for *_messages.cc file changes (see the
>> OWNERS
>> file)
>>
>
> Hmm, I can only see *_messages.h entries in the OWNERS file (for IPC
> message
> changes) - am I missing something?
>

Oh okay, I must have misremembered, please ignore :) The bot will warn with
more accuracy than I will heh.