On Tue, Nov 3, 2009 at 06:50, John Abd-El-Malek j...@chromium.org wrote:
*The solution*
1+2: Use ChromeThread::PostTask and friends (i.e. PostDelayedTask,
DeleteSoon, ReleaseSoon) which are safe and efficient: no locks are grabbed
if the target thread is known to outlive the current thread.
Yeah, I understood :) I haven't seen much discussion of these issues
so figured I'd try and start some - perhaps a lurker would be
motivated to work on it. Or maybe the Chrome team in a later release.
The force-ssl stuff seems like good progress. Still, Chrome takes a
less aggressive stance than
Hi Adam/Evan/Pawell, thanks for the useful feedback.
This gold sounds good, although if I want to install it on Fedora I note
that there isn't package (yet) and the src/build/install-build-deps.sh
script is apparently only for Ubuntu 8/9, so I will have to go the 'hard
way' mentioned on the
On Tue, Nov 3, 2009 at 12:54 AM, Mike Hearn mh.in.engl...@gmail.com wrote:
Yeah, I understood :) I haven't seen much discussion of these issues
so figured I'd try and start some - perhaps a lurker would be
motivated to work on it. Or maybe the Chrome team in a later release.
We had a lot of
Is it feasible to get an easy way to initiate a hard refresh
(bypassing the cache)? There's a key-combo (CTRL + Refresh) in other
browsers, not sure if it works in Chrome, but has anyone ever thought
of building this functionality right into the button? Would be helpful
if you could simply
Can you put this information, as well as some examples for proper
usage, on dev.chromium.org. I doubt this stuff is very well documented
from the perspective of someone trying to use it, and this looks like
a great start!!!
On Tue, Nov 3, 2009 at 12:50 AM, John Abd-El-Malek j...@chromium.org
Dear chromiumers,
I've got a change (see https://bugs.webkit.org/show_bug.cgi?id=31051
solution for crbug.com/16821) which might have performance
implications (it invokes WinAPI via couple of levels of indirection
and might trigger V8's GC).
There are various ways to make this check faster, but
On Tue, Nov 3, 2009 at 1:01 AM, 黒鉄章 ak...@yayakoshi.net wrote:
This gold sounds good, although if I want to install it on Fedora I note
that there isn't package (yet) and the src/build/install-build-deps.sh
script is apparently only for Ubuntu 8/9
Patches to fix this would be welcome.
so I
I'm not sure which news you're referring to. :)
To answer Jeremy's request: briefly, git doesn't track renames at all
(which are conceptually differences between versions) which makes
sense when you consider what it does track (conceptually, only
collections of files and an ordering between
On Tue, Nov 3, 2009 at 16:58, Evan Martin e...@chromium.org wrote:
I'm not sure which news you're referring to. :)
To the one that the files are supposed to by svn-copied by git, that there
is a command to check for that, and that you offered to help in case of
problems. :)
I get
NameError: name '_toolset' is not defined while loading dependencies
of base/base.gyp while loading dependencies of app/app.gyp while
loading dependencies of build/all.gyp while trying to load
build/all.gyp
when building or even just running 'gclient runhooks --force'.
Also, when I do
*Chrome UI Jank Task Force Status*
The Chrome UI Jank Task Force is out to fix UI Jank and slowness in the
browser. A list of open Jank bugs is here:
http://code.google.com/p/chromium/issues/list?q=label:Jank (feel free to
take one!)
*Updates*
*
*
Chase: working on disk access on startup, has a
Are there best practices we can follow that will help the scripts be
more accurate? For example, if we move the file, make sure that we
commit it in the new location before making any text changes? If so,
we could at least document that.
Erik
On Tue, Nov 3, 2009 at 7:58 AM, Evan Martin
Generally it should Just Work. If it doesn't, I'd like to hear about it.
http://code.google.com/p/chromium/wiki/UsingGit#Renames_and_Copies
On Tue, Nov 3, 2009 at 9:16 AM, Erik Kay erik...@chromium.org wrote:
Are there best practices we can follow that will help the scripts be
more accurate?
On Tue, Nov 3, 2009 at 8:06 AM, Dan Kegel d...@kegel.com wrote:
I get
NameError: name '_toolset' is not defined while loading dependencies
of base/base.gyp while loading dependencies of app/app.gyp while
loading dependencies of build/all.gyp while trying to load
build/all.gyp
when
On Tue, Nov 3, 2009 at 19:08, Drew Wilson atwil...@chromium.org wrote:
I've also been known to use the built-in GNU emacs from a terminal shell
(especially when working remotely/SSH-ing). I see no reason why you can't
have the same experience that you do with your Linux box, although you'd
I've been using Aquamacs, although I don't like the fact that it has *so
many* modes built-in and turned-on by default, and I also don't like the
fact that it doesn't lend itself well to customization (it has its own set
of initialization files it keeps hidden off). It's decent, though (but
again,
I heartily recommend MacVim ( http://code.google.com/p/macvim/ ). I
hear its fullscreen mode is pretty rad :-P
Having said that, XCode – like most OS X apps – support some emacs
keybindings. Your Remove entire line example is ctrl-a ctrl-k ctrl-k
in XCode.
I usually use some mix of MacVim and
TextMate is pretty solid and very customizable. I wonder if we can get gyp
to generate .tmproj files :P
I also like it's command line tool mate for opening files from the
Terminal.
On Tue, Nov 3, 2009 at 9:56 AM, Paweł Hajdan Jr. phajdan...@chromium.orgwrote:
Recently I started working more
Another option is to do one TBR commit where you just fork the file
with no changes, then work again as normal.
May I suggest using gcl instead for this changelist... Would that be
possible?
I don't like the idea of losing all the commit history of the file he's
working on, just because git
On Tue, Nov 3, 2009 at 7:30 AM, Anton Muhin ant...@chromium.org wrote:
So the question: how could I check that I won't (notably) regress
Chromium performance with this change?
We have perf bots; you can check in and look at the effects there, and there
might even be a way to get perf results
2009/11/3 Paweł Hajdan Jr. phajdan...@chromium.org:
Recently I started working more and more on Mac OS X, and I'm trying to find
an editor that would work the best for me.
On Linux I used GNU Emacs with many customizations. Some of the most handy
ones for working on Chromium were Google Style
On Tue, Nov 3, 2009 at 9:21 AM, Evan Martin e...@chromium.org wrote:
On Tue, Nov 3, 2009 at 8:06 AM, Dan Kegel d...@kegel.com wrote:
NameError: name '_toolset' is not defined while loading dependencies
when building or even just running 'gclient runhooks --force'.
Also, when I do 'gclient
In my case, git didn't quite figure out that my refactoring (splitting a
file into two) should also link up the new file to the history of the old.
There are magic flags I could pass (-Cn) to make git more aggressive about
matching up files to histories, but it turns out that passing those flags
Recently I started working more and more on Mac OS X, and I'm trying to find
an editor that would work the best for me.
On Linux I used GNU Emacs with many customizations. Some of the most handy
ones for working on Chromium were Google Style script and launching the
compilation from the editor. I
There's a Carbon-based, non-Aquamacs Emacs available for Mac OS X. Recommended.
http://www.emacswiki.org/emacs/EmacsForMacOS
http://www.emacswiki.org/emacs/CarbonEmacsPackage
-Ken
On Tue, Nov 3, 2009 at 10:08 AM, Drew Wilson atwil...@chromium.org wrote:
I've been using Aquamacs, although I
But this means that the person didn't use the trybot.
I think we need to be harsher on people who commit with changes that didn't
complete or failed on the trybot. They need to have a really good reason as
to why they want to try their change on the buildbot and possibly delay many
other
On Tue, Nov 3, 2009 at 9:18 AM, Evan Martin e...@chromium.org wrote:
Generally it should Just Work. If it doesn't, I'd like to hear about it.
http://code.google.com/p/chromium/wiki/UsingGit#Renames_and_Copies
FWIW, it never used to work at all for me, but it recently started
working. Did
On Tue, Nov 3, 2009 at 6:01 AM, Mike Pinkerton pinker...@chromium.org wrote:
Can you put this information, as well as some examples for proper
usage, on dev.chromium.org. I doubt this stuff is very well documented
from the perspective of someone trying to use it, and this looks like
a great
Another option is to do one TBR commit where you just fork the file
with no changes, then work again as normal.
Pretty ugly though. :)
On Tue, Nov 3, 2009 at 9:29 AM, Drew Wilson atwil...@chromium.org wrote:
In my case, git didn't quite figure out that my refactoring (splitting a
file into
Thanks a lot, Peter.
Yes, I am aware of those. Just wanted to run something before committing :)
I actually ran some DOM related tests. Adam (on IM) suggested to run
SunSpider, V8 and page load cyclers. I am going to do it and if
numbers are fine, then would try to commit my change.
yours,
On Tue, Nov 3, 2009 at 8:06 AM, Dan Kegel d...@kegel.com wrote:
I get
NameError: name '_toolset' is not defined while loading dependencies
of base/base.gyp while loading dependencies of app/app.gyp while
loading dependencies of build/all.gyp while trying to load
build/all.gyp
when
On Tue, Nov 3, 2009 at 7:08 PM, Ben Goodger (Google) b...@chromium.orgwrote:
I am supportive of auto-revert as long as we apply it universally. So many
times the tree has been busted forever because of a vacuum of action by the
sheriff.
Also FYI - the trybots never work for me on my home
The most common case of 5 minute bustage fix is file was omitted
from changelist.
-Ben
On Tue, Nov 3, 2009 at 2:34 PM, Peter Kasting pkast...@google.com wrote:
On Tue, Nov 3, 2009 at 2:08 PM, Ojan Vafai o...@chromium.org wrote:
To be clear, here's the proposed policy: Any change that would
Drover is actually an order of magnitude faster than a standard svn
merge/reverse, merge since it only affects the files that are involved with
the CL (as opposed to trying to descending the entire trunk). A normal
merge would take in the space of 8 minutes on my box (which required an
existing
Hi
One of the goal of the Green Tree Task Force was to make reverting a change
easy and fast.
This is really important to keep the tree green and open. The old saying
is Revert now, think later... If a change broke the build and the fix
would
take more than 1 or 2 minutes to be committed, or if
Thanks Kenneth.
I tried this. But I get
I/Users/samuel/chromium3/src/third_party/WebKit/WebCore/WebCore.gyp/../../../../xcodebuild/WebCore.build/Debug/webcore_bindings.build/DerivedSources/i386
I'd be willing to consider alternatives. The key thing here is that we need
a way to generate multiple configurations within a single expansion,
currently at least.
I suppose an alternative would be to have gyp expand things multiple times
with CONF or somesuch being each value from a list of
On Tue, Nov 3, 2009 at 12:55 PM, John Munro ghost...@gmail.com wrote:
Lifehacker posted a series of browser benchmarks here
http://lifehacker.com/539/browser-speed-tests-the-windows-7-results
and in their memory use test Chrome came out using the most.
It's entirely possible their
Thanks. Kenneth,
I fixed the ENABLE_3D_CANVAS redefined error. But I still get the
following compilation error:
In file included from
/Users/samuel/chromium3/src/third_party/WebKit/WebCore/WebCore.gyp/../bindings/v8/DerivedSourcesAllInOne.cpp:65:
On Tue, Nov 3, 2009 at 6:05 PM, John Abd-El-Malek j...@chromium.org wrote:
But this means that the person didn't use the trybot.
I think we need to be harsher on people who commit with changes that didn't
complete or failed on the trybot. They need to have a really good reason as
to why they
One trick that I've regularly done when concerned about the perf impact of
a patch is to:
a) Wait till late at night pacific time, when very few checkins are landing
(often there are multi-hour gaps).
b) Land your change, and wait till the build bots all start building (about
a minute later).
I've heard this suggested a few times (not just in this thread) - is there
an advantage to running Carbon Emacs instead of
plain-old-GNU-Emacs-in-a-terminal?
-atw
On Tue, Nov 3, 2009 at 1:14 PM, Scott Hess sh...@chromium.org wrote:
+1. I found Aquamacs weird/annoying. Carbon Emacs seemed a
On Tue, Nov 3, 2009 at 1:53 PM, Nicolas Sylvain nsylv...@chromium.orgwrote:
This is really important to keep the tree green and open. The old saying
is Revert now, think later... If a change broke the build and the fix
would
take more than 1 or 2 minutes to be committed, or if the committer
On Tue, Nov 3, 2009 at 2:34 PM, Peter Kasting pkast...@google.com wrote:
If a change closes the tree, the change author has 1 or 2 minutes to respond
to a ping. The change should be reverted if the author doesn't respond, if
he says to revert, or if he does not say he has a fix within the
Carbon Emacs makes the Alt key work right, and copy/paste to the
system pasteboard works like I expect w/in emacs. I find using emacs
w/in Terminal.app to be annoying enough that I use emacsclient to
throw edits out to Emacs.app.
-scott
On Tue, Nov 3, 2009 at 2:21 PM, Drew Wilson
FYI: http://codereview.chromium.org/341078
http://codereview.chromium.org/341078Linux interactive tests have finally
gone green.
On Mon, Nov 2, 2009 at 10:08 AM, John Abd-El-Malek j...@chromium.org wrote:
(CCing today's sheriffs + Jay)
I was thinking of marking the bug to P1, except that
Hi all,
I have just checked in a new set of baselines that should allow you to
run the layout tests on Win 7 as well as Vista and XP.
For those of you playing along at home, this means that if you have a
baseline that is windows 7 specific, or is the same across 7, Vista,
and XP, check it into
Ha.
http://tech.slashdot.org/story/09/11/03/1334203/X11-Chrome-Reportedly-Outperforms-Windows-and-Mac-Versions
On Fri, Oct 30, 2009 at 9:44 AM, Linus Upson li...@google.com wrote:
scrolling jank in gmail
http://crbug.com/25741
Linus
On Wed, Oct 28, 2009 at 12:05 PM, Adam Barth
In Carbon emacs, the command key works as Meta, and copying/pasting is
integrated with the system clipboard. There are probably other
advantages.
-Ken
On Tue, Nov 3, 2009 at 1:21 PM, Drew Wilson atwil...@chromium.org wrote:
I've heard this suggested a few times (not just in this thread) - is
Interesting. WebKit has similar functionality as:
bugzilla-tool rollout 12345
We've found that git reverts are at least an order of magnitude faster
than SVN reverse merges.
I wonder what bugzilla-tool/drover can learn from one another.
-eric
On Tue, Nov 3, 2009 at 1:53 PM, Nicolas Sylvain
IMO, I wouldn't mind draconian reverts in the interest of keeping the
tree open and allowing the sheriffs some semblance of productivity.
OTOH, git makes it really easy for me to un-revert and try again, so
maybe I'm biased there.
- a
On Tue, Nov 3, 2009 at 3:03 PM, Eric Seidel
Thanks, James.
So I ran SunSpider, V8 and page_cyclers. SunSpider and V8 are roughly the same.
page_cyclers trickier. First of all, I failed to run some tests on my
box (for both clean and patched builds)---all *Http tests just hang.
I hope it's missetup on my part, but hopefully not crucial.
On Tue, Nov 3, 2009 at 12:03 AM, Paweł Hajdan Jr.
phajdan...@chromium.orgwrote:
On Tue, Nov 3, 2009 at 06:50, John Abd-El-Malek j...@chromium.org wrote:
*The solution*
1+2: Use ChromeThread::PostTask and friends (i.e. PostDelayedTask,
DeleteSoon, ReleaseSoon) which are safe and efficient: no
On Tue, Nov 3, 2009 at 10:12 AM, Paweł Hajdan Jr.
phajdan...@chromium.orgwrote:
On Tue, Nov 3, 2009 at 19:08, Drew Wilson atwil...@chromium.org wrote:
I've also been known to use the built-in GNU emacs from a terminal shell
(especially when working remotely/SSH-ing). I see no reason why you
I tried that a few days ago, and drover died with me with something
along the lines of Can't talk to chrome-svn (which as far as I
understand is some internal svn repo?). A quick glance at the source
confirms that this is still the case.
On Tue, Nov 3, 2009 at 1:53 PM, Nicolas Sylvain
On Mon, Nov 2, 2009 at 3:37 PM, Bradley Nelson bradnel...@google.comwrote:
S,o as it happens, I just had someone on nacl make the first use of
multiple inheritance this morning.
He hasn't checked it in yet, but the use case is:
'Common': {
# bunch of global stuff
'defines': [
#
On Tue, Nov 3, 2009 at 4:21 AM, Meok meok...@gmail.com wrote:
Is it feasible to get an easy way to initiate a hard refresh
(bypassing the cache)? There's a key-combo (CTRL + Refresh) in other
browsers, not sure if it works in Chrome, but has anyone ever thought
of building this functionality
On Tue, Nov 3, 2009 at 3:01 PM, Nico Weber tha...@chromium.org wrote:
I tried that a few days ago, and drover died with me with something
along the lines of Can't talk to chrome-svn (which as far as I
understand is some internal svn repo?). A quick glance at the source
confirms that this is
On Tue, Nov 3, 2009 at 7:46 PM, Kenneth Russell k...@chromium.org wrote:
On Tue, Nov 3, 2009 at 6:05 PM, John Abd-El-Malek j...@chromium.org
wrote:
But this means that the person didn't use the trybot.
I think we need to be harsher on people who commit with changes that
didn't
complete
I'm OK with that.
Just make it clear that the sheriff does have authority. One time when I was
sheriff I wanted to revert a broken patch. The author insisted on patching
it over and over. He finally got it working about about seven patches and
nearly three hours or so, when I was insisting on
+100.
This is very similar to getting paged about a production problem.
Sometimes you get sucked into wasting an hour on easy fixes which
don't fix anything. That sets you up for stupid mistakes.
So, you broke the build. Take it like a man/woman, revert your
change, and land it again when
It has been working fine for me. Is it possible you have the wrong
version of WebKit? What does tools/sync-webkit-git.py say?
On Tue, Nov 3, 2009 at 6:46 PM, n179911 n179...@gmail.com wrote:
Thanks. Kenneth,
I fixed the ENABLE_3D_CANVAS redefined error. But I still get the
following
Anyone who wants to follow along on this, I've filed
http://code.google.com/p/chromium/issues/detail?id=26659 to track it.
-- Dirk
On Sat, Oct 24, 2009 at 5:01 PM, Dirk Pranke dpra...@chromium.org wrote:
Sure. I was floating the idea first before doing any work, but I'll
just grab an existing
Forgive me if I shouldn't be posting this kind of thing to this group,
but I've seen similar things posted here before.
Lifehacker posted a series of browser benchmarks here
http://lifehacker.com/539/browser-speed-tests-the-windows-7-results
and in their memory use test Chrome came out using
I've updated the wiki. I was wary of putting this stuff up because of bit
rot, but since there's already documentation we won't be worse off.
As for the renderer process: this object would have to be recreated. I'd
prefer to keep ChromeThread just for one process to keep things clearer. I
Thanks for your help.
It seems everything is in sync, but I still get the same compiling errors.
~/chromium3/src(trunk) $ git pull
Already up-to-date.
~/chromium3/src(trunk) $ tools/sync-webkit-git.py
Desired revision: r50487.
Already on correct revision.
~/chromium3/src(trunk) $ gclient sync
On Tue, Nov 3, 2009 at 8:36 PM, n179911 n179...@gmail.com wrote:
And when i got to a different directory trying to use git to get
Webkit code, I get this error possible repository corruption on the
remote side, not sure if these 2 are related.
git clone git://git.webkit.org/WebKit.git .
On Tue, Nov 3, 2009 at 8:39 PM, Evan Martin e...@chromium.org wrote:
On Tue, Nov 3, 2009 at 8:36 PM, n179911 n179...@gmail.com wrote:
And when i got to a different directory trying to use git to get
Webkit code, I get this error possible repository corruption on the
remote side, not sure if
Hi,
I've experienced the same problem before. As far as I tested, building
Release mode binary (hammer --mode=Release) would require much less memory
than Debug binary. If it's sufficient for you, you may want to try this.
Thanks,
Yuta
On Tue, Nov 3, 2009 at 6:01 PM, 黒鉄章 ak...@yayakoshi.net
Do the trybots build the release version? Because I had a build break last
week that passed the 3 basic trybots, but failed to compile on the release
buildbots because of a missing include which was apparently pulled in
through other means in the debug version.
-atw
On Tue, Nov 3, 2009 at 7:30
Yeah, does anyone know why SVN does that? It seems like it could easily
inspect the revision to see what files changed, and then only merge those
files :-/
depot_tools/revert also works this way. Should we remove
depot_tools/revert, or re-implement it on top of drover?
-Darin
On Tue, Nov 3,
Great to see this! Thanks,
-Darin
On Tue, Nov 3, 2009 at 6:31 PM, Dirk Pranke dpra...@chromium.org wrote:
Hi all,
I have just checked in a new set of baselines that should allow you to
run the layout tests on Win 7 as well as Vista and XP.
For those of you playing along at home, this
Great stuff! Thanks for working on this. No longer do I have to add
--nocheck-sys-deps to all of my commands.
fetch me a cofee no fetch me a coffee --nocheck-sys-deps ok.
:DG
On Tue, Nov 3, 2009 at 6:31 PM, Dirk Pranke dpra...@chromium.org wrote:
Hi all,
I have just checked in a new set of
Very cool!
Could this idea be done in the render process? We try to keep the media
processing code off the render thread but we've been bitten using cached
MessageLoops which have been destructed (usually on tab close when the
render thread goes away).
On Mon, Nov 2, 2009 at 9:50 PM, John
I encountered another problem related to Singletons in unit tests.
PluginService is a Singleton, and it listens to extensions notifications. In
one of my tests when I was using the extensions notifications the
PluginService crashed because I passed NULL as the Extension* - because the
listener I
Why don't you pass an Extension instance? It seems like you are
violating the contract of that notification by not doing so, and even
if you solve this problem, other similar problems would be likely to
happen again in the future.
- a
On Tue, Nov 3, 2009 at 11:49 PM, Paweł Hajdan Jr.
77 matches
Mail list logo