Make TestWebIDBFactory allocate a separate WebIDBFactory per thread.

Make TestWebIDBFactory allocate a separate WebIDBFactory per thread.

This allows the main WebKit thread and web worker threads to each get their own 
WebIDBFactory instance, which mimics the behavior when we let the WebIDBFactory 
be instantiated via the WebKitPlatformSupport::idbFactory() method. 

Avoid storing WebString on class that will be reached by multiple threads.

This is a slight variation on the patch attempted here:
https://chromiumcodereview.appspot.com/12181010

R=alecflett@chromium.org

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

[darin (slow to review), 2013-02-15 20:16:10]

One thing I realized is that webkit_support::GetAbsoluteWebStringFromUTF8Path is
not necessary.  ScopedTempDir already produces an absolute FilePath, so we just
need to convert that to a WebString.  This is what the GetDataDir function does.

[alecflett, 2013-02-15 20:24:06]

On 2013/02/15 20:16:10, darin wrote:
> One thing I realized is that webkit_support::GetAbsoluteWebStringFromUTF8Path
is
> not necessary.  ScopedTempDir already produces an absolute FilePath, so we
just
> need to convert that to a WebString.  This is what the GetDataDir function
does.

lgtm - I don't know enough about FilePaths so I'm assuming you've got this
right...

[darin (slow to review), 2013-02-15 21:19:11]

Thanks.  FilePath just uses std::{w}string, so it is thread-safe.

-Darin


On Fri, Feb 15, 2013 at 12:24 PM, <alecflett@chromium.org> wrote:

> On 2013/02/15 20:16:10, darin wrote:
>
>> One thing I realized is that webkit_support::**
>> GetAbsoluteWebStringFromUTF8Pa**th
>>
> is
>
>> not necessary.  ScopedTempDir already produces an absolute FilePath, so we
>>
> just
>
>> need to convert that to a WebString.  This is what the GetDataDir function
>>
> does.
>
> lgtm - I don't know enough about FilePaths so I'm assuming you've got this
> right...
>
>
https://chromiumcodereview.**appspot.com/12230054/<https://chromiumcodereview...
>