[chromium-dev] Re: Question about V8 bindings

2009-07-23 Thread Mads Sig Ager
Ah, yes of course. The generated code for the base class itself needs it. I had also forgotten that we use the ID to get the superclass template to inherit from. Don't listen to me. :-) -- Mads On Wed, Jul 22, 2009 at 10:15 PM, Drew Wilsonatwil...@chromium.org wrote: Digging further through

[chromium-dev] Re: FYI: Upcoming O3D integration changes.

2009-07-23 Thread Dean McNamee
Any idea on how much this increase the size of chrome.dll? On Wed, Jul 22, 2009 at 9:51 PM, Nicolas Sylvainnsylv...@chromium.org wrote: On Wed, Jul 22, 2009 at 11:47 AM, Greg Spencer gspen...@google.com wrote: On Wed, Jul 22, 2009 at 11:27 AM, Nicolas Sylvain nsylv...@chromium.org wrote:

[chromium-dev] Re: avoiding compile failures on buildbot

2009-07-23 Thread Thomas Van Lenten
Just to be complete, linux can have the same issue, and I'd expect Windows also to be able too. This is one class of things the try server doesn't catch because it is building debug/unoptimized. TVL On Wed, Jul 22, 2009 at 10:09 PM, Andrew Scherkus scher...@chromium.orgwrote: On Wed, Jul 22,

[chromium-dev] Re: Generating files in source tree considered harmful

2009-07-23 Thread Dan Kegel
On Thu, Jul 23, 2009 at 4:56 AM, Mark Mentovaim...@chromium.org wrote: An objdir-ish solution would make sense, except the native build systems we use don't really work in terms of objdirs, so we'd just wind up generating a parallel directory structure with nothing but xcode projects,

[chromium-dev] Re: Generating files in source tree considered harmful

2009-07-23 Thread Darin Fisher
On Wed, Jul 22, 2009 at 9:56 PM, Mark Mentovai m...@chromium.org wrote: Dan Kegel wrote: No. Generating files is the point of gyp. Nothing says they have to be in the source tree. Out of all of the proposals out there, the only ones that make any sense to my software-developin' mind

[chromium-dev] Re: Generating files in source tree considered harmful

2009-07-23 Thread Darin Fisher
On Thu, Jul 23, 2009 at 7:26 AM, Darin Fisher da...@chromium.org wrote: On Wed, Jul 22, 2009 at 9:56 PM, Mark Mentovai m...@chromium.org wrote: Dan Kegel wrote: No. Generating files is the point of gyp. Nothing says they have to be in the source tree. Out of all of the proposals out

[chromium-dev] Re: Generating files in source tree considered harmful

2009-07-23 Thread Nicolas Sylvain
On Wed, Jul 22, 2009 at 9:15 PM, Mark Larson (Google) m...@chromium.orgwrote: On Wed, Jul 22, 2009 at 20:27, Dan Kegel d...@kegel.com wrote: Stop me if you've heard this one before. Today, a new directory was added to the source tree, and shortly thereafter was reverted. Should have

[chromium-dev] Re: Generating files in source tree considered harmful

2009-07-23 Thread Dan Kegel
On Thu, Jul 23, 2009 at 2:28 PM, Darin Fisherda...@chromium.org wrote: An objdir-ish solution would make sense, except the native build systems we use don't really work in terms of objdirs, so we'd just wind up generating a parallel directory structure with nothing but xcode projects,

[chromium-dev] Re: Generating files in source tree considered harmful

2009-07-23 Thread Nicolas Sylvain
On Thu, Jul 23, 2009 at 8:53 AM, Evan Martin e...@chromium.org wrote: On Thu, Jul 23, 2009 at 7:47 AM, Nicolas Sylvainnsylv...@chromium.org wrote: gclient has nothing to do with this case. svn update src/ was trying to add a directory called src/bleh, but src/bleh already existed, so svn

[chromium-dev] Re: Generating files in source tree considered harmful

2009-07-23 Thread Thomas Van Lenten
On Thu, Jul 23, 2009 at 12:00 PM, Dan Kegel d...@kegel.com wrote: On Thu, Jul 23, 2009 at 2:28 PM, Darin Fisherda...@chromium.org wrote: An objdir-ish solution would make sense, except the native build systems we use don't really work in terms of objdirs, so we'd just wind up generating a

[chromium-dev] Re: FYI: Upcoming O3D integration changes.

2009-07-23 Thread Greg Spencer
On Thu, Jul 23, 2009 at 2:46 AM, Dean McNamee de...@chromium.org wrote: Any idea on how much this increase the size of chrome.dll? Not yet - I'll let you know as soon as we're ready (some symbols are still not being referenced, so the count probably isn't accurate yet). -Greg.

[chromium-dev] Re: Generating files in source tree considered harmful

2009-07-23 Thread Dan Kegel
On Thu, Jul 23, 2009 at 9:07 AM, Thomas Van Lententhoma...@chromium.org wrote: Here's my christmas wish: I'd like gyp and chrome to support cross-compilation, so that I could (on my Linux box) kick off distcc-accelerated builds for all three platforms, each one going into a separate objdir.

[chromium-dev] Re: Generating files in source tree considered harmful

2009-07-23 Thread Thomas Van Lenten
The other thing to remember is the buildbot scripts and a bunch of build scripts on all platforms are full of assumptions about the relationships between projects and the tree. :( TVL On Thu, Jul 23, 2009 at 12:11 PM, Dan Kegel d...@kegel.com wrote: On Thu, Jul 23, 2009 at 9:07 AM, Thomas Van

[chromium-dev] Re: Generating files in source tree considered harmful

2009-07-23 Thread Dan Kegel
On Thu, Jul 23, 2009 at 9:15 AM, Thomas Van Lententhoma...@chromium.org wrote: The other thing to remember is the buildbot scripts and a bunch of build scripts on all platforms are full of assumptions about the relationships between projects and the tree.  :( Well, yes. I didn't say it would

[chromium-dev] Re: Generating files in source tree considered harmful

2009-07-23 Thread Darin Fisher
On Thu, Jul 23, 2009 at 8:53 AM, Evan Martin e...@chromium.org wrote: On Thu, Jul 23, 2009 at 7:47 AM, Nicolas Sylvainnsylv...@chromium.org wrote: gclient has nothing to do with this case. svn update src/ was trying to add a directory called src/bleh, but src/bleh already existed, so svn

[chromium-dev] Re: Generating files in source tree considered harmful

2009-07-23 Thread Scott Hess
On Thu, Jul 23, 2009 at 9:01 AM, Nicolas Sylvainnsylv...@chromium.org wrote: On Thu, Jul 23, 2009 at 8:53 AM, Evan Martin e...@chromium.org wrote: On Thu, Jul 23, 2009 at 7:47 AM, Nicolas Sylvainnsylv...@chromium.org wrote: gclient has nothing to do with this case. svn update src/ was

[chromium-dev] Re: Design Doc: Adaptive spell checking for multilingual users

2009-07-23 Thread sidchat
Thank you Brian and Paul for your comments and insights. Yes, the plan is to expand to multi-spellcheck simultaneously when the user types in some other language other than the default one, and avoid changing on the fly too often. Paul, I have this feature on hold right now to discuss more

[chromium-dev] Dumping rendered contents to image and identifying objects

2009-07-23 Thread Grace
Hi, I'm trying to figure out a way to get the rendered contents of a page (such as amazon.com) to dump to an image file (PNG, bitmap or similar), and I also want to be able to identify what objects are being rendered (flash etc.). What would be the best way to approach this? Thanks for your

[chromium-dev] Re: FF's search engine keyword

2009-07-23 Thread Dirk Pranke
(cc'ing chromium-discuss, bcc'ing chromium-dev) It is? How do you specify keywords in Chrome's Bookmarks editor? -- Dirk On Wed, Jul 22, 2009 at 12:49 PM, Peter Kastingpkast...@google.com wrote: On Wed, Jul 22, 2009 at 12:46 PM, Igor Gatis igorga...@gmail.com wrote: (please forgive me if

[chromium-dev] Mini Installer Build issues

2009-07-23 Thread Dan Duong
Hi All, My name is Dan here goes my first post to chromium-dev! Anyone else seeing problems building the mini_installer vcproj? I get the following error: 5fatal error RC1110: could not open .\$(IntDir) \mini_installer_exe_version.rc I'm guessing $(IntDir) is an absolute path and its trying to

[chromium-dev] Re: Mini Installer Build issues

2009-07-23 Thread Bradley Nelson
This kind of error tends to occur if gyp regenerates the project while you are in visual studio.Very often if you close and reopen the solution the problem will go away. Unfortunately hitting yes to project/sln reload does not seem to be equivalent in all cases to a full close and reopen. -BradN

[chromium-dev] REMINDER: STRING AND FEATURE FREEZE EOD TOMORROW

2009-07-23 Thread Anthony LaForge
Please have any strings you want in the next release in by Friday evening. Kind Regards, Anthony Laforge Technical Program Manager Mountain View, CA External Phone: 1-650-214-4055 --~--~-~--~~~---~--~~ Chromium Developers mailing list:

[chromium-dev] Re: fighting the flakiness - resource bundle issues?

2009-07-23 Thread Peter Kasting
On Thu, Jul 23, 2009 at 4:31 PM, Paweł Hajdan Jr. phajdan...@chromium.orgwrote: Some of the flaky failures are caused by resource bundle issues. If you are familiar with the build process, or the resource bundle, please take a look. It looks like something needed a manual clobber and didn't

[chromium-dev] Re: fighting the flakiness - resource bundle issues?

2009-07-23 Thread Tony Chang
To elaborate on Peter's comment. IncrediBuild (which the buildbots use) get confused by changes to our grd files. Our grd files generate headers, which should then cause lots of cc files to get rebuilt. Visual Studio seems to always get this right, but IncrediBuild gets this wrong and cc files

[chromium-dev] Re: Compiler warnings?

2009-07-23 Thread Ben Laurie
On Thu, Jul 23, 2009 at 4:30 PM, Wan-Teh Changw...@google.com wrote: Ben, I believe the GCC warning you want is -Wreturn-type, which is enabled if we specify -Wall: http://gcc.gnu.org/onlinedocs/gcc-4.3.3/gcc/Warning-Options.html#Warning-Options Are we not compiling with -Wall? Apparently

[chromium-dev] Re: fighting the flakiness - resource bundle issues?

2009-07-23 Thread Paweł Hajdan Jr .
Is it possible to force it to rebuild some files, or... I don't know, do you see some real way to fix this problem? On Thu, Jul 23, 2009 at 16:41, Tony Chang t...@chromium.org wrote: To elaborate on Peter's comment. IncrediBuild (which the buildbots use) get confused by changes to our grd

[chromium-dev] Re: Compiler warnings?

2009-07-23 Thread Ben Laurie
On Thu, Jul 23, 2009 at 4:44 PM, Darin Fisherda...@chromium.org wrote: On Thu, Jul 23, 2009 at 4:42 PM, Ben Laurie b...@google.com wrote: On Thu, Jul 23, 2009 at 4:30 PM, Wan-Teh Changw...@google.com wrote: Ben, I believe the GCC warning you want is -Wreturn-type, which is enabled if

[chromium-dev] Re: Compiler warnings?

2009-07-23 Thread Evan Martin
On Thu, Jul 23, 2009 at 4:50 PM, Ben Laurieb...@google.com wrote: On Thu, Jul 23, 2009 at 4:44 PM, Darin Fisherda...@chromium.org wrote: On Thu, Jul 23, 2009 at 4:42 PM, Ben Laurie b...@google.com wrote: On Thu, Jul 23, 2009 at 4:30 PM, Wan-Teh Changw...@google.com wrote: Ben, I

[chromium-dev] Re: Compiler warnings?

2009-07-23 Thread Wan-Teh Chang
Ben, I believe the GCC warning you want is -Wreturn-type, which is enabled if we specify -Wall: http://gcc.gnu.org/onlinedocs/gcc-4.3.3/gcc/Warning-Options.html#Warning-Options Are we not compiling with -Wall? Wan-Teh --~--~-~--~~~---~--~~ Chromium Developers

[chromium-dev] Re: Compiler warnings?

2009-07-23 Thread Mike Mammarella
gcc/g++ have __attribute__((warn_unused_result)) that you can specify on a per-function basis: http://www.ohse.de/uwe/articles/gcc-attributes.html#func-warn_unused_result Or do you mean warnings when a function is supposed to return a value but does not have a return statement at the end?

[chromium-dev] new valgrind mandatory if you use chrome_tests.sh

2009-07-23 Thread Dan Kegel
If you don't run tools/valgrind/chrome_tests.sh, you can ignore this message. tools/valgrind/chrome_tests.sh now uses the --show-possible option to valgrind. This option isn't in standard valgrind, so bad things will happen if you haven't run tools/valgrind/build-valgrind-for-chrome.sh to

[chromium-dev] Re: new valgrind mandatory if you use chrome_tests.sh

2009-07-23 Thread Dan Kegel
On Thu, Jul 23, 2009 at 10:55 PM, Dan Kegeld...@kegel.com wrote: If you don't run tools/valgrind/chrome_tests.sh, you can ignore this message. tools/valgrind/chrome_tests.sh now uses the --show-possible option to valgrind.  This option isn't in standard valgrind, so bad things will happen if

[chromium-dev] Re: Compiler warnings?

2009-07-23 Thread Ben Laurie
On Thu, Jul 23, 2009 at 4:54 PM, Evan Martine...@chromium.org wrote: On Thu, Jul 23, 2009 at 4:50 PM, Ben Laurieb...@google.com wrote: On Thu, Jul 23, 2009 at 4:44 PM, Darin Fisherda...@chromium.org wrote: On Thu, Jul 23, 2009 at 4:42 PM, Ben Laurie b...@google.com wrote: On Thu, Jul 23,

[chromium-dev] Re: Compiler warnings?

2009-07-23 Thread Ben Laurie
On Thu, Jul 23, 2009 at 4:17 PM, Mike Mammarellam...@chromium.org wrote: gcc/g++ have __attribute__((warn_unused_result)) that you can specify on a per-function basis: http://www.ohse.de/uwe/articles/gcc-attributes.html#func-warn_unused_result Or do you mean warnings when a function is

[chromium-dev] fighting the flakiness - resource bundle issues?

2009-07-23 Thread Paweł Hajdan Jr .
Some of the flaky failures are caused by resource bundle issues. If you are familiar with the build process, or the resource bundle, please take a look. I'm pasting relevant parts of the logs: C:\b\slave\chromium-dbg-builder\build\src\chrome\test\ui\ui_test.cc(227): error: Value of:

[chromium-dev] Re: fighting the flakiness - resource bundle issues?

2009-07-23 Thread Tony Chang
Look at how the current gyp hook works. It looks for changes to .gyp files and only runs if a .gyp (and maybe gypi?) file has changed. You can find what header it generates by opening the grd file and parsing the XML (the XML lists the output files). You'll need to build the base directory

[chromium-dev] Compiler warnings?

2009-07-23 Thread Ben Laurie
I just fixed a bug that wouldn't have happened at all if missing return values were flagged ... is there a way to turn on compiler warnings (building on Linux using make)? Is there some reason they're not on by default? --~--~-~--~~~---~--~~ Chromium Developers

[chromium-dev] Re: fighting the flakiness - resource bundle issues?

2009-07-23 Thread Paweł Hajdan Jr .
I think that this workaround may be worth it. I'm not familiar with the IncrediBuild, but I can help making the hook (and we can run it only on Windows). How do I make a hook know which grd files changed? And also know which headers it generates? Alternatively, maybe this Windows-only hook would

[chromium-dev] Re: fighting the flakiness - resource bundle issues?

2009-07-23 Thread Tony Chang
Here's a crappy work around: Add a gclient hook that checks for grd file changes. When a grd file changes, force delete the header it would generate. I'm pretty sure this would prevent bad builds from IncrediBuild. Alternately, maybe we can make a reduced test case and file a bug against