[chromium-dev] Re: Throttling URLRequestFileJobs rate?

2009-06-11 Thread Stuart Morgan
On Thu, Jun 11, 2009 at 8:03 PM, Michael Nordman wrote: > Sounds like the underlying issue is not the number of requests (or > type of request), but the number of SharedMemory instances in use on > behalf of request handling at any one time. True; I'll take a look at how else SharedMemory is used

[chromium-dev] Re: Throttling URLRequestFileJobs rate?

2009-06-11 Thread Stuart Morgan
On Thu, Jun 11, 2009 at 7:45 PM, Darin Fisher wrote: > It seems like this issue, since it is about the shared memory used for > streaming resources to a renderer, is not particular to file://.  It could > happen with http:// as well assuming we had a fast enough network or a janky > enough local s

[chromium-dev] Re: Throttling URLRequestFileJobs rate?

2009-06-11 Thread Michael Nordman
Sounds like the underlying issue is not the number of requests (or type of request), but the number of SharedMemory instances in use on behalf of request handling at any one time. Maybe put some limits on directly on that. When a request gets to the point where it needs a SharedMemory block it may

[chromium-dev] Re: Throttling URLRequestFileJobs rate?

2009-06-11 Thread Darin Fisher
On Thu, Jun 11, 2009 at 7:45 PM, Darin Fisher wrote: > On Thu, Jun 11, 2009 at 5:13 PM, Stuart Morgan > wrote: > >> >> I'm working on getting the Intl2 test set from the page cycler up and >> running on the Mac, which currently crashes very quickly. It turns out >> that one of the test pages has

[chromium-dev] Re: Throttling URLRequestFileJobs rate?

2009-06-11 Thread Darin Fisher
On Thu, Jun 11, 2009 at 5:13 PM, Stuart Morgan wrote: > > I'm working on getting the Intl2 test set from the page cycler up and > running on the Mac, which currently crashes very quickly. It turns out > that one of the test pages has hundreds of images on it, and we > simultaneously make hundreds

[chromium-dev] Re: Throttling URLRequestFileJobs rate?

2009-06-11 Thread Aaron Boodman
On Thu, Jun 11, 2009 at 5:13 PM, Stuart Morgan wrote: > > I'm working on getting the Intl2 test set from the page cycler up and > running on the Mac, which currently crashes very quickly. It turns out > that one of the test pages has hundreds of images on it, and we > simultaneously make hundreds

[chromium-dev] Re: Throttling URLRequestFileJobs rate?

2009-06-11 Thread Scott Hess
In my experience, taking code which assumes a low number of file descriptors and just ramping up the file descriptor limits to accommodate a particular case doesn't work out well. You end up finding out that there are three or four other edge cases which cause problems, things like O(N^2) code pa

[chromium-dev] Re: Throttling URLRequestFileJobs rate?

2009-06-11 Thread Jeremy Orlow
On Thu, Jun 11, 2009 at 5:13 PM, Stuart Morgan wrote: > > I'm working on getting the Intl2 test set from the page cycler up and > running on the Mac, which currently crashes very quickly. It turns out > that one of the test pages has hundreds of images on it, and we > simultaneously make hundreds

[chromium-dev] Throttling URLRequestFileJobs rate?

2009-06-11 Thread Stuart Morgan
I'm working on getting the Intl2 test set from the page cycler up and running on the Mac, which currently crashes very quickly. It turns out that one of the test pages has hundreds of images on it, and we simultaneously make hundreds of URLRequestFileJobs to load them. Each of those uses a SharedM

[chromium-dev] Re: Resurrecting the JSC build.

2009-06-11 Thread Eric Seidel
I think such a build would be useful for performance testing of the rest of our infrastructure. It also makes having a webkit.org chromium-webkit build easier. I'm not sure how hard maintaining such a build would be with the new gyp goodness. -eric On Thu, Jun 11, 2009 at 4:06 PM, Dimitri Glaz

[chromium-dev] Re: Resurrecting the JSC build.

2009-06-11 Thread Tony Chang
Kind of tangential, but having a JSC build today would mean that on linux we could build a 64bit version of chromium so we can test IMEs sooner on our 64bit workstations. On Thu, Jun 11, 2009 at 4:06 PM, Dimitri Glazkov wrote: > > Team, > > Now that we're unforked, we want to concentrate on elimi

[chromium-dev] Re: Resurrecting the JSC build.

2009-06-11 Thread Mike Pinkerton
IIRC from the Mac porting effort (a year ago now), the JSC build relies significantly on objective-C dom bindings which we neither want to use nor are easy to get working in our build system. We jettisoned them for a reason and my guess is that it would be a non-trivial effort to re-instate them (

[chromium-dev] Resurrecting the JSC build.

2009-06-11 Thread Dimitri Glazkov
Team, Now that we're unforked, we want to concentrate on eliminating layout test failures. Through the magic of the WebKit merge, we've accumulated quite a few. Today, we expect around 400 failures, which is not a good number by any stretch. As one of the ways to help determine the source of the

[chromium-dev] Re: Naming things _views

2009-06-11 Thread Mike Pinkerton
FWIW, we don't use _cocoa or _mac in browser/cocoa at all, assuming it was implied. I assumed gtk was doing the same. On Thu, Jun 11, 2009 at 11:53 AM, Ben Goodger (Google) wrote: > > Well, class name is supposed to match the file name in Google style. > > -Ben > > On Thu, Jun 11, 2009 at 11:22 A

[chromium-dev] Re: How to change the cursor smoothly for a view

2009-06-11 Thread Ben Goodger (Google)
It's in the class registration function in widget_win.cc -Ben On Thu, Jun 11, 2009 at 2:52 PM, Peter Kasting wrote: > On Thu, Jun 11, 2009 at 2:48 PM, Ben Goodger (Google) > wrote: >> >> Daniel, do you want to try making this change instead? It'll be pretty >> obvious to you if it doesn't work.

[chromium-dev] Re: How to change the cursor smoothly for a view

2009-06-11 Thread Peter Kasting
On Thu, Jun 11, 2009 at 2:48 PM, Ben Goodger (Google) wrote: > Daniel, do you want to try making this change instead? It'll be pretty > obvious to you if it doesn't work. Filed as crbug.com/13926 . We should try and find if we set a value in the WindowWin or WidgetWin (or somewhere) window crea

[chromium-dev] Re: How to change the cursor smoothly for a view

2009-06-11 Thread Ben Goodger (Google)
Daniel, do you want to try making this change instead? It'll be pretty obvious to you if it doesn't work. -Ben On Thu, Jun 11, 2009 at 2:43 PM, Ben Goodger (Google) wrote: > It's worth a shot. We should make sure this is one of the WNDCLASSEX > fields that when not set causes the window not to b

[chromium-dev] Re: How to change the cursor smoothly for a view

2009-06-11 Thread Ben Goodger (Google)
It's worth a shot. We should make sure this is one of the WNDCLASSEX fields that when not set causes the window not to be created, obviously! -Ben On Thu, Jun 11, 2009 at 2:40 PM, Peter Kasting wrote: > On Thu, Jun 11, 2009 at 2:38 PM, Ben Goodger (Google) > wrote: >> >> Is the default cursor c

[chromium-dev] Re: How to change the cursor smoothly for a view

2009-06-11 Thread Peter Kasting
On Thu, Jun 11, 2009 at 2:38 PM, Ben Goodger (Google) wrote: > Is the default cursor config in the window class even necessary? Isn't > the default pointer the arrow anyway? Maybe we could just remove it > from WindowWin. As long as Windows doesn't make the mouse cursor disappear by default when

[chromium-dev] Re: How to change the cursor smoothly for a view

2009-06-11 Thread Ben Goodger (Google)
I see a very faint flicker on XP. It may be exaggerated with different hardware/driver configs I suppose. Is the default cursor config in the window class even necessary? Isn't the default pointer the arrow anyway? Maybe we could just remove it from WindowWin. -Ben On Thu, Jun 11, 2009 at 2:16

[chromium-dev] Re: Text editing freezing jank?

2009-06-11 Thread Julie Parent
If anyone sees this somewhere other than Gmail replies, please comment on the bug. On Thu, Jun 11, 2009 at 1:00 PM, Nick Baum wrote: > Doh, thanks! > -Nick > > > On Thu, Jun 11, 2009 at 12:18 PM, Peter Kasting wrote: > >> On Thu, Jun 11, 2009 at 12:17 PM, Nick Baum wrote: >> >>> Has anyone else

[chromium-dev] Re: How to change the cursor smoothly for a view

2009-06-11 Thread Peter Kasting
On Thu, Jun 11, 2009 at 9:58 AM, Peter Kasting wrote: > Also if there are problems like this with existing classes please file > bugs, and since you have some idea of the fix consider fixing them. > It looks like maybe this problem occurs when mousing over text in the Omnibox -- on Vista my curs

[chromium-dev] Re: 2 problems with extension development

2009-06-11 Thread Aaron Boodman
fwiw, I created a bug on this: http://code.google.com/p/chromium/issues/detail?id=13912 I suspect this will be a fun one to fix :) - a 2009/6/11 Aaron Boodman : > I think views are going to have to sometimes "check in" with the > background page on creation. But since the background page is mea

[chromium-dev] Re: Text editing freezing jank?

2009-06-11 Thread Nick Baum
Doh, thanks! -Nick On Thu, Jun 11, 2009 at 12:18 PM, Peter Kasting wrote: > On Thu, Jun 11, 2009 at 12:17 PM, Nick Baum wrote: > >> Has anyone else run into this behavior in Chrome, in Gmail or elsewhere? >> Any suggestions for next steps to track this down? >> > > I suggest searching the bug d

[chromium-dev] Re: Exclude Tab Bar Area

2009-06-11 Thread Stefan Nuxoll
I have the same issue with the Windows language bar and the SCIM toolbar, I don't think there's really any way to fix this though. On Thu, Jun 11, 2009 at 1:52 PM, Peter wrote: > > Is this possible in chrome currently (perhaps some theme hack would > allow this??)  or through and extension? > > I

[chromium-dev] Re: Exclude Tab Bar Area

2009-06-11 Thread Erik Kay
There's no easy way to do this today. Erik On Thu, Jun 11, 2009 at 12:52 PM, Peter wrote: > > Is this possible in chrome currently (perhaps some theme hack would > allow this??) or through and extension? > > I want to exclude the tab bar from using some space near the window > controls in the

[chromium-dev] Exclude Tab Bar Area

2009-06-11 Thread Peter
Is this possible in chrome currently (perhaps some theme hack would allow this??) or through and extension? I want to exclude the tab bar from using some space near the window controls in the upper left corner of the window. This space is permanently occupied by winamp on my system (set to alway

[chromium-dev] Re: Text editing freezing jank?

2009-06-11 Thread Peter Kasting
On Thu, Jun 11, 2009 at 12:17 PM, Nick Baum wrote: > Has anyone else run into this behavior in Chrome, in Gmail or elsewhere? > Any suggestions for next steps to track this down? > I suggest searching the bug database. You will find this: http://crbug.com/12785 PK --~--~-~--~~

[chromium-dev] Text editing freezing jank?

2009-06-11 Thread Nick Baum
Hi, I've been running into a really annoying freeze when typing messages in Gmail, and I'm trying to figure out if it's Chrome or Gmail. I'll be happily typing away, and the tab will freeze for several seconds. It keeps accepting keyboard input and eventually catches up, but in the meantime it's co

[chromium-dev] Re: Makefile build broken

2009-06-11 Thread Adam Langley
On Thu, Jun 11, 2009 at 11:49 AM, Albert J. Wong (王重傑) wrote: > I'll probably just submit the generated files as tony suggested. Check with Tony, but I think I just reviewed a GYP change which will make (forgive the pun) it work again. AGL --~--~-~--~~~---~--~~ C

[chromium-dev] Re: Naming things _views

2009-06-11 Thread Ben Goodger (Google)
Well, class name is supposed to match the file name in Google style. -Ben On Thu, Jun 11, 2009 at 11:22 AM, Erik Kay wrote: > So the class name is the same too in this case?  (InfoBubble and InfoBubble) > Erik > > On Thu, Jun 11, 2009 at 10:46 AM, Ben Goodger (Google) > wrote: >> >> sure why no

[chromium-dev] Re: Makefile build broken

2009-06-11 Thread 王重傑
sorry for the bustage. I had assumed the make file was still a toy with only 2-3 people using it (not many linux chrome folks where I am). Thanks for rolling it back Adam. I'll probably just submit the generated files as tony suggested. -Albert On Thu, Jun 11, 2009 at 9:51 AM, Thomas Van Lent

[chromium-dev] Re: Any way to compile webkit "faster"

2009-06-11 Thread Sverrir Á . Berg
Hi Brad,It started approximately 2 weeks ago so I guess its more likely the chrome.sln conversion that did it. Sverrir On Thu, Jun 11, 2009 at 2:20 PM, Bradley Nelson wrote: > By last gyp Sverrir, do you mean the modules in chrome.sln switching over, > or the ones in webkit?There was one genera

[chromium-dev] Re: Renderer Process and Browser Process

2009-06-11 Thread John Abd-El-Malek
On Wed, Jun 10, 2009 at 10:19 PM, Meryl Silverburgh < silverburgh.me...@gmail.com> wrote: > On Wed, Jun 10, 2009 at 12:18 PM, John Abd-El-Malek > wrote: > > > > > > On Wed, Jun 10, 2009 at 11:48 AM, Meryl Silverburgh > > wrote: > >> > >> From this document, it talks about Messaging between > >> >

[chromium-dev] Re: Naming things _views

2009-06-11 Thread Erik Kay
So the class name is the same too in this case? (InfoBubble and InfoBubble) Erik On Thu, Jun 11, 2009 at 10:46 AM, Ben Goodger (Google) wrote: > > sure why not... the _platform suffix is supposed to be for files in a > location where multiple platfomrs are built. > > -Ben > > On Thu, Jun 11, 20

[chromium-dev] Re: Any way to compile webkit "faster"

2009-06-11 Thread Bradley Nelson
By last gyp Sverrir, do you mean the modules in chrome.sln switching over, or the ones in webkit?There was one generated set of targets in webkit.gyp, that we ended up adding a makefile emission option for. It was just too slow with rules (all the overhead of crossing over into cygwin per file). Th

[chromium-dev] Re: Adding Mac OS X Spellchecker to Chromium

2009-06-11 Thread sidchat
Use of OS X spellchecker integration in our multi process model probably should be the primary goal here to begin with. If that is achieved, the other stuff (auto spell correct, automatically setting the dictionary etc) should not be too problematic. Note that the actual replacement for auto-corr

[chromium-dev] Re: 2 problems with extension development

2009-06-11 Thread Aaron Boodman
I think views are going to have to sometimes "check in" with the background page on creation. But since the background page is meant to hold on the "core" logic of your extension, it makes much more sense to me have its lifetime bookend all other parts of the extension. - a On Thu, Jun 11, 2009

[chromium-dev] Re: Naming things _views

2009-06-11 Thread Ben Goodger (Google)
sure why not... the _platform suffix is supposed to be for files in a location where multiple platfomrs are built. -Ben On Thu, Jun 11, 2009 at 2:47 AM, Dean McNamee wrote: > Does this mean we can do something similar for GTK? > > It feels a bit unfair that we have to name everything > browser/g

[chromium-dev] Re: How to change the cursor smoothly for a view

2009-06-11 Thread Peter Kasting
On Thu, Jun 11, 2009 at 12:26 AM, Daniel Cowx wrote: > I've fixed this flickering problem by calling ::SetClassLong(hwnd, > GCL_HCURSOR, NULL) within my OnMouseEntered and then I reset it back > to the original class cursor in OnMouseExited. Not sure if this is a > reasonable solution or if you e

[chromium-dev] Re: Makefile build broken

2009-06-11 Thread Thomas Van Lenten
On Thu, Jun 11, 2009 at 12:48 PM, Adam Langley wrote: > > On Thu, Jun 11, 2009 at 8:52 AM, Tony Chang wrote: > > Albert, what do you think? I estimate that about 5-7 people on the > > linux team use the make build these days. > > I reverted in r18168. It sucks that there's no try or builders for

[chromium-dev] Re: Makefile build broken

2009-06-11 Thread Adam Langley
On Thu, Jun 11, 2009 at 8:52 AM, Tony Chang wrote: > Albert, what do you think?  I estimate that about 5-7 people on the > linux team use the make build these days. I reverted in r18168. It sucks that there's no try or builders for the make build, but it's what everyone uses these days. AGL --

[chromium-dev] Re: 2 problems with extension development

2009-06-11 Thread Nick Baum
Well, the issue with loading views first is that when you open a new window, that would create new views. In the extension we've implemented, the background page iterates through the views whenever it needs to update them. Alternatively, we may want to implement onViewCreated events so it's easier

[chromium-dev] Re: 2 problems with extension development

2009-06-11 Thread Nebojša Ćirić
I am somewhat confused wrt. loading sequence. If background page main task is to load data once for all views, and then to update them, shouldn't then views be loaded first (toolstrips etc.)? *background_page*: ... 1. gather data once ... 2. update all views // if loads first, no other views are go

[chromium-dev] Re: Makefile build broken

2009-06-11 Thread Tony Chang
The make file generator has a bug. I started to work on fixing the build last night but it's not finished and I'm not sure how long it will take me this morning. I think it's a little unfair to revert even though all the buildbots are green (since there's no make buildbot and we can use the scon

[chromium-dev] Poting Chromium and security

2009-06-11 Thread Berend-Jan Wever
Hey all, Dean reminded me that because we are developing Chrome for multiple platforms, there will be differences between the various ports that may affect the security of our code. E.g. if we use different implementations of image decoders on different ports, Chromium as a whole will have more "

[chromium-dev] Re: Linux build totally busted.

2009-06-11 Thread Marc-Andre Decoste
Yes, it was my patch, it is completely my fault, I take all the blame.. I'm very sorry about that... The patch had been reviewed but through another revision that was mentioned in the comments... I should have put a link to it as opposed to only the revision ID (I'm used to internal tools that inse

[chromium-dev] Re: Makefile build broken

2009-06-11 Thread Adam Langley
On Thu, Jun 11, 2009 at 2:39 AM, Dean McNamee wrote: > At least for me, I'm hitting an error with generate_stubs.py.  Will > try to figure out the proper fix, in the mean time I fixed up the > paths manually and ran the following from my source root.  I was able > to successfully build.  This is f

[chromium-dev] Re: Any way to compile webkit "faster"

2009-06-11 Thread Sverrir Á . Berg
Just wanted to echo Mohamed's findings. My build has been very slow the past two weeks. I'm using VS2008 and I have not used /MP recently. I have a top of the line machine (xw6600 Quad Core 8G mem, Intel SSD) and building chrome.exe after a sync/clean takes approx 40 minutes.I think this started

[chromium-dev] Re: Linux build totally busted.

2009-06-11 Thread Dean McNamee
Reoccuring lesson, when you're looking for a shotty change, look for the ones with the worst commit messages: Of course I overlooked the change 50 times because the message tells you absolutely nothing, doesn't have a review URL, etc. I don't feel like figuring out what is wrong with this patch,

[chromium-dev] Re: Linux build totally busted.

2009-06-11 Thread Dean McNamee
One of the ideas would be to bisect the archived builds (http://build.chromium.org/buildbot/continuous/linux/LATEST/). I am also working on hunting this down. I'll let you know when/if I find anything. Thanks for the great help -- dean On Thu, Jun 11, 2009 at 1:15 PM, Craig Schlenter wrote: >

[chromium-dev] Re: Linux build totally busted.

2009-06-11 Thread Craig Schlenter
It might not be r18131 ... I compiled head without 18131 and even though it looked like that helped earlier, it's broken again now. Interestingly when opening new tabs, only some of them are broken which is probably why I thought it was ok earlier ... will leave something bisecting in the backroun

[chromium-dev] Re: Linux build totally busted.

2009-06-11 Thread Craig Schlenter
Hi Dean I am seeing the same thing ... it seems to be r18131 that is at fault. I have not tracked it down further than that ... Shared build is also bust. See attached patch ... dunno if that's the right fix. --Craig On Thu, Jun 11, 2009 at 11:58 AM, Dean McNamee wrote: > Something related to i

[chromium-dev] Re: Any way to compile webkit "faster"

2009-06-11 Thread nakro
most of it will, but i never tried it however, i saw here in this forum ppl complaining that /MP on 2005 creates corrupted .PDB (the debug info) files so i don't know, but the incremental linking should work if you wish you can PM me and i will send you whatever you want (even the tool i wrote t

[chromium-dev] Re: Any way to compile webkit "faster"

2009-06-11 Thread PhistucK
Do you think it will also work for Visual Studio 2005 and Windows Vista? ☆PhistucK On Thu, Jun 11, 2009 at 12:39, nakro wrote: > > Mohamed, if you wish i can share with you the "tricks" i do on my > machine > they make the difference of even building chrome.exe from 10 mins to > less than 1 (af

[chromium-dev] Re: Naming things _views

2009-06-11 Thread Dean McNamee
Does this mean we can do something similar for GTK? It feels a bit unfair that we have to name everything browser/gtk/blah_gtk.cc and BrowserToolbarGtk, etc, while views gets the short name. For example views: views/info_bubble.cc and class name InfoBubble gtk: gtk/info_bubble_gtk.cc and class

[chromium-dev] Makefile build broken

2009-06-11 Thread Dean McNamee
At least for me, I'm hitting an error with generate_stubs.py. Will try to figure out the proper fix, in the mean time I fixed up the paths manually and ran the following from my source root. I was able to successfully build. This is for a debug build, replace Debug w/ Release for a release buil

[chromium-dev] Re: Any way to compile webkit "faster"

2009-06-11 Thread nakro
Mohamed, if you wish i can share with you the "tricks" i do on my machine they make the difference of even building chrome.exe from 10 mins to less than 1 (after a small source code change) all this on my very very slow machine i wrote a small exe program which parses the .vsprops and ..vcproj so

[chromium-dev] Re: How to change the cursor smoothly for a view

2009-06-11 Thread Daniel Cowx
I've fixed this flickering problem by calling ::SetClassLong(hwnd, GCL_HCURSOR, NULL) within my OnMouseEntered and then I reset it back to the original class cursor in OnMouseExited. Not sure if this is a reasonable solution or if you even think this is a problem. If so, I can file a bug report an

[chromium-dev] Re: Is there any logging to print out messaging between Browser and Renderer

2009-06-11 Thread Jeremy Moskovich
You can set the environmental variable in XCode by double clicking on the executable target. I'm not sure the logging code is enabled on OSX, the right place to look would be in chrome/common/ipc_channel_posix.cc, search for IPC_MESSAGE_LOG_ENABLED. If that's compiled in, then if the env variable