Issue 7054002: Temporary fix until WebKit::initialize does this for us.

Issue 7054002: Temporary fix until WebKit::initialize does this for us. (Closed)

Created:
9 years, 7 months ago by michaeln

Modified:
9 years, 7 months ago

Reviewers:
darin (slow to review), jochen (gone - plz use gerrit), PhistucK

CC:
chromium-reviews, jam, darin-cc_chromium.org

Base URL:
svn://chrome-svn/chrome/trunk/src/

Visibility:
Public.

Description

Temporary fix until WebKit::initialize does this for us. Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=86722

Patch Set 3 : '' #

Created: 9 years, 7 months ago

Download [raw] [tar.bz2]

		Unified diffs	Side-by-side diffs	Delta from patch set	Stats (+5 lines, -0 lines)			Patch
	M	content/browser/in_process_webkit/webkit_thread.cc	View	1 2	2 chunks	+5 lines, -0 lines	0 comments	Download

Messages

Total messages: 7 (0 generated)

Expand Messages | Collapse Messages

michaeln

In the absence of a better answer for M13, should we do this now?

9 years, 7 months ago (2011-05-24 19:26:33 UTC) #2

jochen (gone - plz use gerrit)

On 2011/05/24 19:26:33, michaeln wrote: > In the absence of a better answer for M13, ...

9 years, 7 months ago (2011-05-24 19:36:22 UTC) #3

PhistucK

http://codereview.chromium.org/7054002/diff/2001/content/browser/in_process_webkit/webkit_thread.cc File content/browser/in_process_webkit/webkit_thread.cc (right): http://codereview.chromium.org/7054002/diff/2001/content/browser/in_process_webkit/webkit_thread.cc#newcode54 content/browser/in_process_webkit/webkit_thread.cc:54: // Exercise WebSecurityOrigin to get it's underlying statics initialized. ...

9 years, 7 months ago (2011-05-25 06:38:49 UTC) #5

michaeln

9 years, 7 months ago (2011-05-25 18:29:42 UTC) #6

darin (slow to review)

9 years, 7 months ago (2011-05-25 21:47:07 UTC) #7

LGTM

On Wed, May 25, 2011 at 11:29 AM, <michaeln@chromium.org> wrote:

>
>
>
http://codereview.chromium.org/7054002/diff/2001/content/browser/in_process_w...
> File content/browser/in_process_webkit/webkit_thread.cc (right):
>
>
>
http://codereview.chromium.org/7054002/diff/2001/content/browser/in_process_w...
> content/browser/in_process_webkit/webkit_thread.cc:54: // Exercise
> WebSecurityOrigin to get it's underlying statics initialized.
> On 2011/05/25 06:38:49, PhistucK wrote:
>
>> s/it's/its
>>
>
> Done.
>
>
> http://codereview.chromium.org/7054002/
>

Expand Messages | Collapse Messages