Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(18)

Issue 765053003: Windows-specific bits for loading V8's initial snapshot from external files. (Closed)

Created:
6 years ago by baixo
Modified:
5 years, 11 months ago
CC:
chromium-reviews, grt+watch_chromium.org, wfh+watch_chromium.org, grt (UTC plus 2)
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Project:
chromium
Visibility:
Public.

Description

Windows-specific bits for loading V8's initial snapshot from external files. BUG=421063

Patch Set 1 #

Patch Set 2 : Enable the feature in a separate CL. #

Patch Set 3 : Move FILES.cfg to a separate CL and assume blobs are in versions folder #

Unified diffs Side-by-side diffs Delta from patch set Stats (+16 lines, -0 lines) Patch
M chrome/installer/mini_installer.gypi View 1 chunk +6 lines, -0 lines 0 comments Download
M gin/isolate_holder.cc View 1 2 3 chunks +10 lines, -0 lines 0 comments Download

Messages

Total messages: 10 (2 generated)
baixo1
Adding OWNERS: gab: chrome/installer mmoss: win/FILES.cfg jochen: gin/ Could you guys take a look, please? ...
6 years ago (2014-12-01 14:43:07 UTC) #2
gab
Do you need those .bin's to be dropped on actual Chrome installs? And if so ...
6 years ago (2014-12-01 16:02:24 UTC) #3
baixo1
On 2014/12/01 16:02:24, gab wrote: > Do you need those .bin's to be dropped on ...
6 years ago (2014-12-01 16:12:21 UTC) #4
gab
On 2014/12/01 16:12:21, baixo1 wrote: > On 2014/12/01 16:02:24, gab wrote: > > Do you ...
6 years ago (2014-12-01 16:20:15 UTC) #5
jochen (gone - plz use gerrit)
On 2014/12/01 at 16:20:15, gab wrote: > On 2014/12/01 16:12:21, baixo1 wrote: > > On ...
6 years ago (2014-12-02 10:27:06 UTC) #6
baixo1
On 2014/12/02 10:27:06, jochen (slow) wrote: > On 2014/12/01 at 16:20:15, gab wrote: > > ...
6 years ago (2014-12-02 11:20:13 UTC) #7
rmcilroy
On 2014/12/02 11:20:13, baixo1 wrote: > On 2014/12/02 10:27:06, jochen (slow) wrote: > > On ...
5 years, 11 months ago (2015-01-09 13:39:56 UTC) #9
gab
5 years, 11 months ago (2015-01-12 19:19:14 UTC) #10
On 2015/01/09 13:39:56, rmcilroy wrote:
> On 2014/12/02 11:20:13, baixo1 wrote:
> > On 2014/12/02 10:27:06, jochen (slow) wrote:
> > > On 2014/12/01 at 16:20:15, gab wrote:
> > > > On 2014/12/01 16:12:21, baixo1 wrote:
> > > > > On 2014/12/01 16:02:24, gab wrote:
> > > > > > Do you need those .bin's to be dropped on actual Chrome installs?
And
> if
> > > so
> > > > > > where (i.e. in the version directory?)?
> > > > > 
> > > > > Yes. V8 is now initialized from external files (the .bin's I added)
and
> > > Chrome
> > > > > will crash if such files are not present. This is already the way it
> works
> > > > > on Android, Linux and Mac.
> > > > > 
> > > > > I need them alongside the exe.
> > > > 
> > > > Having them alongside the exe is VERY hard and problematic on Windows.
> > > > 
> > > > To allow for in-use updates we put everything but chrome.exe itself in
the
> > > version folder besides chrome.exe.
> > > > 
> > > > In order to land in the version folder you'll need to update
> chrome.release
> > as
> > > well as the google3 signing script IIRC (+grt@ who knows signing better
than
> > I).
> > > > 
> > > > Is there a specific reason you think you need to be besides chrome.exe
> > itself?
> > > We have many ways (manifests, etc.) to help keep everything, even load
time
> > > dependencies, in the version directory; but I need to fully understand
your
> > use
> > > case before I can help you further.
> > > 
> > > where's the icudtl.dat file? It should probably be right next to it.
> > 
> > Yes, exactly. I thought it was alongside the exe, but it's actually
> > in the version folder.
> 
> Moving this to https://codereview.chromium.org/819203003/

SG, closing this CL then.

Powered by Google App Engine
This is Rietveld 408576698