Don't restrict supported media types in test_shell when running interactively

Don't restrict supported media types in test_shell when running interactively.

BUG=62828
TEST=See bug report


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=65945

[Marshall, 2010-11-11 18:52:41]

Please review.

[scherkus (not reviewing), 2010-11-11 19:27:00]

http://codereview.chromium.org/4705007/diff/6001/webkit/tools/test_shell/test...
File webkit/tools/test_shell/test_shell_webkit_init.cc (right):

http://codereview.chromium.org/4705007/diff/6001/webkit/tools/test_shell/test...
webkit/tools/test_shell/test_shell_webkit_init.cc:24: :
mime_registry_(layout_test_mode) {
what about creating a webkit_glue::SimpleWebMimeRegistryImpl instead?

[Marshall, 2010-11-11 19:41:12]

On 2010/11/11 19:27:00, scherkus wrote:
> what about creating a webkit_glue::SimpleWebMimeRegistryImpl instead?

Good idea, that's a simpler way to do it.  Please review.

[scherkus (not reviewing), 2010-11-11 19:47:51]

ahh much simpler :)

LGTM and have fun watching videos!

[Marshall, 2010-11-11 19:51:27]

On 2010/11/11 19:47:51, scherkus wrote:
> ahh much simpler :)
> 
> LGTM and have fun watching videos!

Thanks :-) I'll commit once the tries come up clean.

[Timur Iskhodzhanov, 2010-11-12 17:36:53]

Looks like Memory Waterfall didn't like this change - all WebKit bots gone red.
Can you please take a look?

[Marshall Greenblatt, 2010-11-12 18:02:47]

Looking...

On Fri, Nov 12, 2010 at 12:36 PM, <timurrrr@chromium.org> wrote:

> Looks like Memory Waterfall didn't like this change - all WebKit bots gone
> red.
> Can you please take a look?
>
>
> http://codereview.chromium.org/4705007/
>

[Marshall Greenblatt, 2010-11-12 18:28:00]

SimpleWebMimeRegistryImpl needs a virtual destructor, committing now.

On Fri, Nov 12, 2010 at 1:02 PM, Marshall Greenblatt <magreenblatt@gmail.com
> wrote:

> Looking...
>
>
> On Fri, Nov 12, 2010 at 12:36 PM, <timurrrr@chromium.org> wrote:
>
>> Looks like Memory Waterfall didn't like this change - all WebKit bots gone
>> red.
>> Can you please take a look?
>>
>>
>> http://codereview.chromium.org/4705007/
>>
>
>