Re: [chromium-dev] class has virtual method but non-virtual destructor

2009-11-23 Thread Darin Fisher
On Fri, Nov 20, 2009 at 3:00 PM, Mark Mentovai m...@chromium.org wrote: James Robinson wrote: What's the benefit of omitting the virtual destructor? The benefit is that the destructor stays out of the vtable, which will potentially reduce the vtable size and save a layer of indirection. I

Re: [chromium-dev] Anyone understand how V8 GC works?

2009-11-23 Thread Anton Muhin
Now from the right account. yours, anton. On Mon, Nov 23, 2009 at 3:00 PM, Anton Muhin ant...@google.com wrote: On Sun, Nov 22, 2009 at 1:45 AM, Drew Wilson atwil...@chromium.org wrote: I guess I don't really understand why gcPrologue has that ASSERT(wrapper.IsWeak()) then, unless there's

Re: [chromium-dev] workflow for cross-platform development

2009-11-23 Thread Marc-Antoine Ruel
On Sat, Nov 21, 2009 at 1:52 PM, Chris Bentzel cbent...@google.com wrote: Have there been any thoughts about adding gcl patch and unpatch commands which will grab the file diffs as well as duplicate the CL metadata in src/.svn/gcl_info? There's the gcl metadata and the svn metadata. Webkit

Re: [chromium-dev] Singleton shenanigans in base/time_win.cc causing problems (not the first time)

2009-11-23 Thread Paweł Hajdan Jr .
On Fri, Nov 20, 2009 at 23:34, Darin Fisher da...@chromium.org wrote: We could define a function that must be called before you can use code in base/. You could add a call to this everywhere that we currently create the AtExitManager. Or, maybe we could combine those somehow. Thanks for

Re: [chromium-dev] Link error when compiling chromium on ubuntu 9.10

2009-11-23 Thread Michael Moss
You could also try adding swap. It isn't ideal, but it might at least let you build until you can add more RAM. It can take about 4GB to link a debug build. Another option is to build in release mode, which should require less RAM. Michael On Mon, Nov 23, 2009 at 12:12 AM, Craig Schlenter

[chromium-dev] Re: Problem during compilation about gen_x86_insn.py

2009-11-23 Thread Dakota Smith
I would like to add that I am having the same problem when I try to build Chromium from the tarball. I have searched through my documents, the un-extracted tarball, and even the online source code and could not find the file. On the same note, if you just want to run Chromium OS (like I do), the

[chromium-dev] Re: [Chrome-team] Re: [MEMORY] Counting committed bytes

2009-11-23 Thread Anton Muhin
Just an update. The guy to blame was CreateDIBSection used in skia/ext/bitmap_platform_win.cc According to http://msdn.microsoft.com/en-us/library/dd183494(VS.85).aspx : If hSection is NULL, the system allocates memory for the DIB. In this case, the CreateDIBSection function ignores the

Re: [chromium-dev] Linux: gold linker users should upgrade to 2.20 soon.

2009-11-23 Thread Lei Zhang
If you are just testing the Google Chrome build and you don't need symbols, you can always comment out the -gstabs line in build/common.gypi. On Sat, Nov 21, 2009 at 9:10 PM, David Moore davemo...@google.com wrote: Problem solved. Coincident with using the new linker I started building the

[chromium-dev] Core Text

2009-11-23 Thread Jeremy Moskovich
Re http://crbug.com/27195 https://bugs.webkit.org/show_bug.cgi?id=31802 : Dan Bernstein says that Core Text on Leopard has performance issues vs ATSUI so I'm going to look into switching APIs at runtime rather than compile time. So we'd use ATSUI 10.6 Core Text = 10.6 . Best regards,

Re: [chromium-dev] Core Text

2009-11-23 Thread Jeremy Moskovich
Thanks Nico, I'll run some numbers. Best regards, Jeremy On Mon, Nov 23, 2009 at 11:15 PM, Nico Weber tha...@chromium.org wrote: Did you do measuring if it's actually slower on 10.5? The CoreText backend for MacVim is much faster than the ATSUI backend from what I've heard (then again,

Re: [chromium-dev] Any interest in command line tools for network stack?

2009-11-23 Thread Eric Roman
One command line tool I have sometimes wanted when debugging, is a way to dump out a cached response from the disk cache. The only way I know to do this currently is create a profile and load it in chrome, then use about:cache to print the response as hex. (At which point you still need to

[chromium-dev] Extensions gallery open for uploads

2009-11-23 Thread Aaron Boodman
Hello all, I'm happy to announce that the developer area of our extensions gallery is now open: https://chrome.google.com/extensions You can upload your extensions, edit their details, add screenshots and videos, test the autoupdate system, and begin publishing your download URL. We are

Re: [chromium-dev] Core Text

2009-11-23 Thread Dirk Pranke
As an aside, have we looked at using DirectWrite() on Windows? -- Dirk On Mon, Nov 23, 2009 at 12:58 PM, Jeremy Moskovich jer...@chromium.org wrote: Re http://crbug.com/27195  https://bugs.webkit.org/show_bug.cgi?id=31802 : Dan Bernstein says that Core Text on Leopard has performance issues

Re: [chromium-dev] Any interest in command line tools for network stack?

2009-11-23 Thread Mike Belshe
On Mon, Nov 23, 2009 at 2:26 PM, Eric Roman ero...@chromium.org wrote: One command line tool I have sometimes wanted when debugging, is a way to dump out a cached response from the disk cache. There is a dump_cache.exe utility; it can copy a cache or copy the whole thing to a set of disk

[chromium-dev] UI Jank Task Force Status Update

2009-11-23 Thread Glenn Wilson
*UI Jank Task Force Status Update* * * *Agenda* * * Excellent progress last week - Spellchecking now in done in the renderer; moved a lot of disk access off I/O thread. - Launching child processes now done in background thread; off of UI / IO threads. - 30% improvement on Linux

[chromium-dev] Request for a reviewer

2009-11-23 Thread James Hawkins
Hi, Any kind soul willing to review a simple regexp parser? 211 lines. http://codereview.chromium.org/439008 Thanks, James Hawkins -- Chromium Developers mailing list: chromium-dev@googlegroups.com View archives, change email options, or unsubscribe:

Re: [chromium-dev] UI Jank Task Force Status Update

2009-11-23 Thread Peter Kasting
On Mon, Nov 23, 2009 at 2:58 PM, Glenn Wilson gwil...@chromium.org wrote: Investigating using a flat file for safebrowsing instead of SQLite (need to recreate bloom filter and watch disk I/O) jam crunched some numbers here. During the bloom filter recreation process we read about 50 MB and

Re: [chromium-dev] Core Text

2009-11-23 Thread Peter Kasting
On Mon, Nov 23, 2009 at 2:51 PM, Dirk Pranke dpra...@chromium.org wrote: As an aside, have we looked at using DirectWrite() on Windows? crbug.com/25541 (No) PK -- Chromium Developers mailing list: chromium-dev@googlegroups.com View archives, change email options, or unsubscribe:

[chromium-dev] Re: UI Jank Task Force Status Update

2009-11-23 Thread Glenn Wilson
Sorry, two corrections: - The promo cookie was removed from the new tab page startup, but local tests show cookies are still being loaded. Investigation is ongoing. - The disk access run time check is actually a script that uses Windbg to detect these types of accesses. The next

[chromium-dev] Re: [chromium-extensions] Extensions gallery open for uploads

2009-11-23 Thread Antony Sargent
Also: will you be allowing extensions hosted on Google to have a custom update URL? I'm guessing no, but thought I'd find out for sure. Our plan has been for the gallery to just handle autoupdate for you. Any reason you'd want to do it yourself? If it's for usage tracking purposes, one

[chromium-dev] buildbot failure in Chromium on Modules XP (dbg), revision 32899

2009-11-23 Thread buildbot
Automatically closing tree for net_unittests on Modules XP (dbg) http://build.chromium.org/buildbot/waterfall/builders/Modules%20XP%20%28dbg%29/builds/19523 http://build.chromium.org/buildbot/waterfall/waterfall?builder=Modules%20XP%20%28dbg%29 --= Automatically closing tree for net_unittests

Re: [chromium-dev] Request for a reviewer

2009-11-23 Thread Darin Fisher
Don't we already have a real regexp engine in WebKit. Why add another? Did you consider wrapping WebCore::RegularExpression with a WebKit API? -Darin On Mon, Nov 23, 2009 at 3:09 PM, James Hawkins jhawk...@chromium.orgwrote: Hi, Any kind soul willing to review a simple regexp parser? 211

[chromium-dev] buildbot failure in Chromium on Webkit Mac10.5 (dbg)(3), revision 32913

2009-11-23 Thread buildbot
Automatically closing tree for test_shell_tests on Webkit Mac10.5 (dbg)(3) http://build.chromium.org/buildbot/waterfall/builders/Webkit%20Mac10.5%20%28dbg%29%283%29/builds/7458 http://build.chromium.org/buildbot/waterfall/waterfall?builder=Webkit%20Mac10.5%20%28dbg%29%283%29 --= Automatically

[chromium-dev] Re: buildbot failure in Chromium on Webkit Mac10.5 (dbg)(3), revision 32913

2009-11-23 Thread Mike Belshe
I'm looking at this; not sure that it is anything I caused Mike On Mon, Nov 23, 2009 at 8:18 PM, build...@chromium.org wrote: http://build.chromium.org/buildbot/waterfall/ Automatically closing tree for test_shell_tests on Webkit Mac10.5 (dbg)(3)

[chromium-dev] Re: buildbot failure in Chromium on Webkit Mac10.5 (dbg)(3), revision 32913

2009-11-23 Thread Mike Belshe
Dirk suspects this was caused by his: http://chrome-buildbot.corp.google.com:8010/builders/Webkit%20(dbg)(3)/builds/10311 http://chrome-buildbot.corp.google.com:8010/builders/Webkit%20(dbg)(3)/builds/10311 Mike On Mon, Nov 23, 2009 at 8:23 PM, Mike Belshe mbel...@google.com wrote: I'm

Re: [chromium-dev] Core Text

2009-11-23 Thread 坊野 博典
Greetings Jeremy, This is just for your information. It seems WebCore/platform/graphics/mac/ComplexTextControllerCoreText.cpp uses CTRunGetAdvancesPtr() and CTRunGetAdvances(), which are available only on 10.6 or later. (This might be a reason why WebKit doesn't use Core Text for Leopard?) (*1)

Re: [chromium-dev] Core Text

2009-11-23 Thread Jeremy Moskovich
Thanks Hironori, I thought so too at the beginning but it turns out that they are available, just not declared in public headers. The WebKit bug I linked to has a patch that switches us to Core Text on 10.5 10.6. I'll do some perf tests and then we can make a decision based on that. Best