Re: [chromium-dev] Interesting performance characteristics of http://a.qoid.us/google.html

2010-01-12 Thread Brett Wilson
On Mon, Jan 11, 2010 at 9:04 PM, Chris Evans cev...@chromium.org wrote: Hi, I found this interesting: http://a.qoid.us/google.html Because it's clearly very non-JavaScript focused (just a CSS rotation) and yet the browsers exhibit very different performance characteristics. All on my

Re: [chromium-dev] Interesting performance characteristics of http://a.qoid.us/google.html

2010-01-12 Thread Brett Wilson
On Tue, Jan 12, 2010 at 11:33 AM, Evan Martin e...@chromium.org wrote: On Tue, Jan 12, 2010 at 10:17 AM, Brett Wilson bre...@chromium.org wrote: I imagine the problems on Linux are the same as they were on Windows, where the underlying graphics layer wants to place glyphs aligned on pixel

Re: [chromium-dev] How can I tell if a Renderer process finishes loading a page

2010-01-11 Thread Brett Wilson
On Mon, Jan 11, 2010 at 12:59 AM, hap 497 hap...@gmail.com wrote: Thank you. But when I load www.cnn.com, I see something strange: First the throbber is spinning, and then the cnn fave icon is shown and then the throbber is spinning again, and then fave icon is shown. And then the fave icon

Re: [chromium-dev] Splitting off some pieces of chrome.gyp...

2009-12-10 Thread Brett Wilson
On Mon, Dec 7, 2009 at 1:43 PM, Bradley Nelson bradnel...@google.com wrote: Hello All, Last week I re-landed a change to split off parts of chrome.gyp into .gypi's in the same directory. I had done something similar a couple weeks back, but took it out because concern was raised about merge

Re: [chromium-dev] Thoughts on // NOLINT?

2009-12-09 Thread Brett Wilson
On Wed, Dec 9, 2009 at 4:24 PM, John Abd-El-Malek j...@chromium.org wrote: btw I searched the code, almost all the instances are in code from different repositories, like v8, gtest, gmock.  I counted only 17 instances in Chrome's code. Most of the Chrome NOLINTs look like the're around

[chromium-dev] Re: Changes to using threads in the browser process

2009-11-03 Thread Brett Wilson
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

[chromium-dev] Re: Tab Thumbnails and Aero Peek (of Windows 7)

2009-10-23 Thread Brett Wilson
2009/10/23 Hironori Bono (坊野 博典) hb...@google.com: Hi Brett, Thank you so much for noticing this. I'm integrating the ThumbnailGenertor class into my prototype now. :) By the way, I'm a little wondering if there is a function that changes the width and the height of a thumbnail image

[chromium-dev] Re: Tab Thumbnails and Aero Peek (of Windows 7)

2009-10-22 Thread Brett Wilson
Hi Hironori, Thanks for researching this topic. On generating thumbnails, I wrote chrome/browser/tab_contents/thumbnail_generator which tries to get the most up-to-date thumbnail images for all tabs (except ones that haven't ever been shown) with minimal performance overhead. The code hasn't

[chromium-dev] Re: Spellchecker and memory-mapped dicts

2009-10-22 Thread Brett Wilson
On Thu, Oct 22, 2009 at 1:39 PM, Evan Stade est...@chromium.org wrote: Hi all, At its last meeting the jank task force discussed improving responsiveness of the spellchecker but we didn't come to a solid conclusion so I thought I'd bring it up here to see if anyone else has opinions. The

[chromium-dev] Re: Brent's Windows port of WebKit moving to Skia

2009-10-19 Thread Brett Wilson
On Mon, Oct 19, 2009 at 7:27 AM, Evan Martin e...@chromium.org wrote: On Sun, Oct 18, 2009 at 11:58 PM, Adam Barth aba...@chromium.org wrote: It looks like Brent Fulgham is strongly considering moving his Windows WebKit port to Skia:

[chromium-dev] Re: Running applications in chrome with javascript

2009-10-19 Thread Brett Wilson
On Mon, Oct 19, 2009 at 1:19 AM, David palm...@gmail.com wrote: Someone knows where is a good guide for make an npapi plugin? and how to call function to execute process... Your best bet is probably starting with the layout test plugin:

[chromium-dev] String handling in base moved around, and where you should put your shared code

2009-10-10 Thread Brett Wilson
I finished moving the last bit of string stuff in preparation for separating ICU from base. All ICU and related utilities have moved to src/base/i18n. The rest of base should not depend on base/i18n because it will be a separate library, but base/i18n can depend on the rest of base. The core

[chromium-dev] Re: Local State / Preferences distinction

2009-10-07 Thread Brett Wilson
On Wed, Oct 7, 2009 at 12:52 PM, Tony Chang t...@chromium.org wrote: Actually, the original distinction was stuff that could be shared across computers and stuff that couldn't be shared (i.e., local state).  I think originally this was for the difference between stuff that a mounted home

[chromium-dev] Re: Local State / Preferences distinction

2009-10-07 Thread Brett Wilson
On Wed, Oct 7, 2009 at 1:26 PM, Tony Chang t...@chromium.org wrote: The quasi-defunct profile system that uses the enable-udd-profiles still has a separate Local State file and safe browsing databases because it used different user data directory (udd). This change seems orthogonal to the

[chromium-dev] Re: Unimplemented GraphicsContextSkia functions

2009-09-30 Thread Brett Wilson
On Wed, Sep 30, 2009 at 12:18 AM, Andrew Scherkus scher...@chromium.org wrote: I'm doing some video UI refactoring to use GraphicsContext but noticed drawPath() is unimplemented (causes linker error). What's a good practice to implementing something no one has ever used before?  Do we strive

[chromium-dev] Re: Kiosk Mode for Chrome

2009-09-29 Thread Brett Wilson
On Tue, Sep 29, 2009 at 4:15 PM, Jeremy Orlow jor...@chromium.org wrote: I'm guessing different people/companies will have different needs for a kiosk mode. Maybe all of these should be separate flags rather than one kiosk flag?  We could then offer recommendations in a Chromium for kiosks

[chromium-dev] Re: Kiosk Mode for Chrome

2009-09-29 Thread Brett Wilson
On Tue, Sep 29, 2009 at 4:37 PM, Jeremy Orlow jor...@chromium.org wrote: On Tue, Sep 29, 2009 at 4:20 PM, Brett Wilson bre...@chromium.org wrote: On Tue, Sep 29, 2009 at 4:15 PM, Jeremy Orlow jor...@chromium.org wrote: I'm guessing different people/companies will have different needs

[chromium-dev] Re: Kiosk Mode for Chrome

2009-09-24 Thread Brett Wilson
On Thu, Sep 24, 2009 at 10:32 PM, Adam Barth aba...@chromium.org wrote: On Thu, Sep 24, 2009 at 10:07 PM, Darin Fisher da...@chromium.org wrote: Maybe I'm in the minority, but it doesn't sound that unreasonable to support command line options for disabling the status bubble and starting in

[chromium-dev] Re: Clear Strict-Transport-Security state checkbox added

2009-09-17 Thread Brett Wilson
On Thu, Sep 17, 2009 at 3:31 PM, Adam Langley a...@chromium.org wrote: On Thu, Sep 17, 2009 at 3:03 PM, Peter Kasting pkast...@google.com wrote: It's disappointing to me that this change was made without any bug in the bug database linked, and without any input from a member of the UI team,

[chromium-dev] Re: Need help on how to send data from RenderView to a custom DOMUI instantly

2009-09-15 Thread Brett Wilson
On Tue, Sep 15, 2009 at 7:31 PM, Mohamed Mansour m0.interact...@gmail.com wrote: Hi all, I am having trouble trying to figure out how I could replace the contents of a specific WebFrame while I am in RenderView with a custom DOMUI page that I have created. Here is a scenario: User visits

[chromium-dev] Re: linux startup regression

2009-09-03 Thread Brett Wilson
On Thu, Sep 3, 2009 at 9:26 AM, Evan Martine...@chromium.org wrote: Ok, I'll mark that I own this here and unassign it if I can't figure it out: http://code.google.com/p/chromium/issues/detail?id=20991 On Thu, Sep 3, 2009 at 2:41 AM, Dean McNameede...@chromium.org wrote: Ug, I can't

[chromium-dev] Re: Long (exciting?) writeup on improving Mac New Tab Performance (not entirely Mac-specific)

2009-08-31 Thread Brett Wilson
On Mon, Aug 31, 2009 at 8:56 AM, Mark Mentovaim...@chromium.org wrote: I spent a chunk of last week looking at the new tab page performance on startup on the Mac.  I found that the renderer was waiting on data from the browser process for what seemed like far too long.  The key to this

[chromium-dev] Re: Long (exciting?) writeup on improving Mac New Tab Performance (not entirely Mac-specific)

2009-08-31 Thread Brett Wilson
On Mon, Aug 31, 2009 at 9:17 AM, Brett Wilsonbre...@chromium.org wrote: On Mon, Aug 31, 2009 at 8:56 AM, Mark Mentovaim...@chromium.org wrote: I spent a chunk of last week looking at the new tab page performance on startup on the Mac.  I found that the renderer was waiting on data from the

[chromium-dev] Re: Layout try slaves

2009-08-31 Thread Brett Wilson
On Mon, Aug 31, 2009 at 10:13 AM, Marc-Antoine Ruelmar...@chromium.org wrote: If you are not a committer, you can skip this message. If you want to run layout tests as a try job, you can use the layout try slaves. They are not in the default pool so you need to reference them manually.

[chromium-dev] Re: Copying of profiles across systems

2009-08-26 Thread Brett Wilson
On Wed, Aug 26, 2009 at 10:23 AM, Avi Drissmana...@google.com wrote: I've heard people proclaim the principle of being able to copy a profile across systems as being a deciding factor for certain changes (e.g. the history epoch change). However, it doesn't seem to be universally held or

[chromium-dev] Re: Formatting substrings in a views::Label

2009-08-26 Thread Brett Wilson
+finnur who did this for the about box. Brett On Wed, Aug 26, 2009 at 2:15 PM, Aaron Boodmana...@chromium.org wrote: Argh. Forgot attachment. On Wed, Aug 26, 2009 at 2:15 PM, Aaron Boodmana...@chromium.org wrote: I'm playing with different ideas for the extension install dialog, and

[chromium-dev] Linux/Mac time epoch change

2009-08-25 Thread Brett Wilson
I just checked in r24417 which changes the 0 for times on Mac Linux (used to mean 1970) to match Windows (1601). This means that the profiles can be copied between systems without getting the dates all wrong, and should also fix some bugs related to cookie expiration (

[chromium-dev] Re: private VM field on the about:memory page

2009-08-24 Thread Brett Wilson
On Mon, Aug 24, 2009 at 6:18 AM, Anand Mistryakmis...@gmail.com wrote: I'm looking at the about:memory page and am wondering how useful is the private VM field?  Would it be just as good to have a total VM instead?  The reason I ask is because private VM doesn't map easily to Linux where

[chromium-dev] Re: Question about resource_dispatcher_host.h

2009-08-24 Thread Brett Wilson
On Mon, Aug 24, 2009 at 12:49 PM, hap 497hap...@gmail.com wrote: Thanks. But the picture in the document shows there is only 1 ResourceDispatcherHost and there are 2 Renderer Processes: http://dev.chromium.org/developers/design-documents/multi-process-architecture And the

[chromium-dev] Re: FreeBSD port and ifdefs

2009-08-19 Thread Brett Wilson
On Wed, Aug 19, 2009 at 1:23 PM, Darin Fisherda...@chromium.org wrote: On Wed, Aug 19, 2009 at 1:14 PM, Evan Martin e...@chromium.org wrote: Rad that you're doing this! I think that we shouldn't have any linux-specific directories except for one or two cases where it's *really*

[chromium-dev] Re: FreeBSD port and ifdefs

2009-08-19 Thread Brett Wilson
On Wed, Aug 19, 2009 at 6:00 PM, Dean McNameede...@chromium.org wrote: I kinda feel like this is one of those things you can try hard to premeditate, but in the end you'll just have to deal with it being ugly for a while and hope it eventually converges to something better. The changes in

[chromium-dev] Re: FreeBSD port and ifdefs

2009-08-19 Thread Brett Wilson
On Wed, Aug 19, 2009 at 9:49 PM, Brett Wilsonbre...@chromium.org wrote: On Wed, Aug 19, 2009 at 6:00 PM, Dean McNameede...@chromium.org wrote: I kinda feel like this is one of those things you can try hard to premeditate, but in the end you'll just have to deal with it being ugly for a while

[chromium-dev] Re: browser/sync is moving in

2009-08-18 Thread Brett Wilson
On Mon, Aug 17, 2009 at 11:19 PM, estelectronix...@gmail.com wrote: Thanks for the explaination, Nick Carter. I understand now, Chrome is a Google product while Chromium is a open- source project. Any thing else differs between Google Chrome and Chromium? I found dictionaries missing from

[chromium-dev] Re: RenderView page ID

2009-08-17 Thread Brett Wilson
On Mon, Aug 17, 2009 at 2:59 PM, Evan Stadeest...@chromium.org wrote: I believe Brett meant to say chrome/renderer/render_{view,widget}.{h,cc} They're both correct, depending on whether you want the browser or the renderer. Brett --~--~-~--~~~---~--~~ Chromium

[chromium-dev] Re: Mac History Menu

2009-08-12 Thread Brett Wilson
On Wed, Aug 12, 2009 at 11:15 AM, Robert Sesekrse...@gmail.com wrote: Two things about the Mac history menu that I'd like people to weigh in on: 1. The Show All History command should have a keyboard shortcut. We can't use the logical Cmd+H because it's bound by the system. Stuart suggested

[chromium-dev] Re: Mac History Menu

2009-08-12 Thread Brett Wilson
On Wed, Aug 12, 2009 at 12:18 PM, Avi Drissmana...@chromium.org wrote: Brett— Are we talking about the history page, or history items? The history page gets its own tab, sure. But when someone picks an item from the history menu, where does it go? I think current foreground tab is right,

[chromium-dev] Re: Do we have any existing code for reading/writing INI files?

2009-08-10 Thread Brett Wilson
On Mon, Aug 10, 2009 at 3:28 PM, Daniel Cowxdaniel.c...@gmail.com wrote: Just wondering if there's any code kicking around somewhere in the codebase for reading/writing INI files? No, we don't deal with ini files at all to my knowledge. We use JSON for that type of thing. What do you need it

[chromium-dev] Re: How do I filter CC'd code reviews in Gmail?

2009-08-03 Thread Brett Wilson
On Fri, Jul 31, 2009 at 5:44 PM, Rahul Kuchhalkuch...@chromium.org wrote: I thought to:user and -cc:user would work in this case? Have you tried using that in Has the words on Filter edit screen? One problem with this is that it doesn't fix the case where people respond to the review mail via

[chromium-dev] Re: new hunspell has tons of valgrind warnings... revert?

2009-08-02 Thread Brett Wilson
On Sat, Aug 1, 2009 at 4:34 PM, Dan Kegeld...@kegel.com wrote: I suppose you could try running the hunspell test suite itself under valgrind.  Their README tells how to do it, but when I tried, I couldn't get it to work.  (Wonder if that means they haven't run it, either?) Hi Dan, Purify

[chromium-dev] How do I filter CC'd code reviews in Gmail?

2009-07-31 Thread Brett Wilson
I added myself to some watchlists and I get a bunch of email about a lot of reviews But this has confused my Gmail filters, which automatic ally label and archive code reviews not addressed directly to me, and all of my CCd code reviews show up in my inbox. Anybody know how to reliably detect

[chromium-dev] Re: WebContents class in chromium?

2009-07-25 Thread Brett Wilson
On Sat, Jul 25, 2009 at 4:27 PM, n179911n179...@gmail.com wrote: Hi, For http://dev.chromium.org/developers/how-tos/getting-around-the-chrome-source-code, it said For web content, this specialization is WebContents in chrome/browser/web_contents.cc. There are also a few other types for

[chromium-dev] Re: [linux] Using system libraries for Chromium

2009-07-22 Thread Brett Wilson
On Wed, Jul 22, 2009 at 2:58 PM, Adam Langleya...@chromium.org wrote: On Wed, Jul 22, 2009 at 9:50 PM, Darin Fisherda...@google.com wrote: Personally, I much prefer the #include png.h approach.  Is it a problem to wait for the GYP change that makes this possible? Well, I could make it

[chromium-dev] Re: Please keep TOOLKIT_VIEWS green!

2009-07-21 Thread Brett Wilson
On Tue, Jul 21, 2009 at 8:42 AM, Nicolas Sylvainnsylv...@chromium.org wrote: On Tue, Jul 21, 2009 at 8:31 AM, Paweł Hajdan Jr. phajdan...@chromium.org wrote: Can we have a trybot with that configuration, which would just compile the code? I think it would really save people's time. I never

[chromium-dev] Re: base::string16 / WebCore::String incompatibility

2009-07-13 Thread Brett Wilson
On Fri, Jul 10, 2009 at 5:04 PM, Jeremy Orlowjor...@chromium.org wrote: WebCore::String has the interesting property of differentiating between an empty string and a null string. In string16, however, there is no such thing as null. The LocalStorage implementation I'm working on proxies data

[chromium-dev] Re: base::string16 / WebCore::String incompatibility

2009-07-13 Thread Brett Wilson
On Mon, Jul 13, 2009 at 10:05 AM, Jeremy Orlowjor...@chromium.org wrote: On Mon, Jul 13, 2009 at 8:59 AM, Darin Fisher da...@chromium.org wrote: On Mon, Jul 13, 2009 at 8:20 AM, Brett Wilson bre...@chromium.org wrote: On Fri, Jul 10, 2009 at 5:04 PM, Jeremy Orlowjor...@chromium.org wrote

[chromium-dev] Re: Watchlist matching any part of a path

2009-07-07 Thread Brett Wilson
On Tue, Jul 7, 2009 at 6:20 PM, Andrew Scherkusscher...@google.com wrote: According to the watchlist file, brettw is watching base from /src kylep sent me a code review that only modifies touches files in /src/media/base, however brettw got added to the cc list. Is this intended behavior for

[chromium-dev] Re: About hunspell

2009-07-03 Thread Brett Wilson
On Fri, Jul 3, 2009 at 12:00 AM, Dmitry.Skibadmitry.sk...@gmail.com wrote: Recently I was playing with hunspell and got very displeased with its code (which claims to be cpp, but in fact 'classified' c). I even started to think about rewriting it in, you know, correct cpp. Then I realized

[chromium-dev] Conventions and patterns for multi-platform development

2009-06-19 Thread Brett Wilson
Our team has had somewhat of an ad-hoc approach to organizing code that's different across platforms. In many cases our approach has been quite good. In others, less so, and there have also been questions about what the preferred method for writing a certain component in a cross-platform way.

[chromium-dev] Re: Icelandic dictionary for Chrome

2009-06-19 Thread Brett Wilson
On Fri, Jun 19, 2009 at 1:15 PM, Sverrir Á. Bergsver...@chromium.org wrote: I'm hoping to be able to add Icelandic spelling check to Chrome  Before I invest a lot of time I have some quick questions/observations: I found an myspell/aspell dictionary for Open Office here: 

[chromium-dev] Re: How Render process send Image IO request to Browser process

2009-06-16 Thread Brett Wilson
There are a surprising number of comic characters on here. I bet you know Meryl Silverburgh! Generally this type of thing is on the dev site like here: http://dev.chromium.org/developers/design-documents/multi-process-resource-loading Brett --~--~-~--~~~---~--~~

[chromium-dev] Mac painting problems

2009-06-14 Thread Brett Wilson
I was informed that I may have broken mac tab contents painting in 18363. I have to leave now, so if this is causing you problems you can back me out. If somebody wants to fix it instead, I'll make you chocolate chip cookies :) Brett --~--~-~--~~~---~--~~

[chromium-dev] Re: Question about BrowserRenderProcessHost::OnPageContents() method

2009-06-07 Thread Brett Wilson
On Sat, Jun 6, 2009 at 11:13 PM, meryl silverburgh.me...@gmail.com wrote: Hi, I add debug print status in  BrowserRenderProcessHost::OnPageContents () to print out the value of 'contents' void BrowserRenderProcessHost::OnPageContents(const GURL url,                                      

[chromium-dev] Re: [extensions] Gleam API proposal

2009-06-03 Thread Brett Wilson
On Wed, Jun 3, 2009 at 1:00 PM, Aaron Boodman a...@chromium.org wrote: [I actually created this awhile ago, but never sent it 'round] It is really common for extensions to want to offer additional contextual options for content elements. For example Download this movie, edit this image,

[chromium-dev] Re: Question about chromium architecture

2009-06-02 Thread Brett Wilson
On Tue, Jun 2, 2009 at 4:57 PM, Meryl Silverburgh silverburgh.me...@gmail.com wrote: I am reading this document http://dev.chromium.org/developers/how-tos/getting-around-the-chrome-source-code; about chromium source code: It said: renderer: Code for the subprocess in each tab. This embeds

[chromium-dev] Clobber if you get V8 linker errors in the next sync

2009-05-29 Thread Brett Wilson
When I landed the WebKIt integration this morning a bunch of the bots needed clobbering due to linker errors. The errors were missing V8 handles while linking generate_profile. I haven't seen any of the commonly-compiled projects fail because of this, so it's possible you don't have to clobber

[chromium-dev] Re: reminder: don't use CHECK()

2009-05-27 Thread Brett Wilson
On Wed, May 27, 2009 at 11:56 AM, Amanda Walker ama...@chromium.org wrote: On Tue, May 26, 2009 at 11:31 PM, Brett Wilson bre...@chromium.org wrote: Don't bother doing an assertion when the next line will crash anyway:  DCHECK(foo);  foo-DoSomething(); will normally crash pretty obviously

[chromium-dev] Re: Skia is moving

2009-05-13 Thread Brett Wilson
On Wed, May 13, 2009 at 12:39 PM, Darin Fisher da...@chromium.org wrote: That is what I meant by pain It only applies to third party code that conforms to the google style guide, which says that all include paths must be relative to the root.  Come to think of it, I think this could cause

[chromium-dev] Re: Changes to FilePath?

2009-05-13 Thread Brett Wilson
On Wed, May 13, 2009 at 3:51 PM, Amanda Walker ama...@chromium.org wrote: Perhaps what we need is a companion to FilePath.  For example: FilePath: much as it is now, lightweight, alternative to string manipulation. FileReference: heavierweight, can talk to the file system and have carnal

[chromium-dev] Re: Changes to FilePath?

2009-05-13 Thread Brett Wilson
On Wed, May 13, 2009 at 4:34 PM, Amanda Walker ama...@chromium.org wrote: On Wed, May 13, 2009 at 7:07 PM, Brett Wilson bre...@chromium.org wrote: On Wed, May 13, 2009 at 3:51 PM, Amanda Walker ama...@chromium.org wrote: Perhaps what we need is a companion to FilePath.  For example

[chromium-dev] Re: Changes to FilePath?

2009-05-13 Thread Brett Wilson
On Wed, May 13, 2009 at 6:12 PM, Greg Spencer gspen...@google.com wrote: On Wed, May 13, 2009 at 4:07 PM, Brett Wilson bre...@chromium.org wrote: On Wed, May 13, 2009 at 3:51 PM, Amanda Walker ama...@chromium.org wrote: Perhaps what we need is a companion to FilePath.  For example

[chromium-dev] Re: Changes to FilePath?

2009-05-13 Thread Brett Wilson
On Wed, May 13, 2009 at 7:24 PM, Brett Wilson bre...@chromium.org wrote: On Wed, May 13, 2009 at 6:12 PM, Greg Spencer gspen...@google.com wrote: On Wed, May 13, 2009 at 4:07 PM, Brett Wilson bre...@chromium.org wrote: On Wed, May 13, 2009 at 3:51 PM, Amanda Walker ama...@chromium.org wrote

[chromium-dev] Re: Skia is moving

2009-05-12 Thread Brett Wilson
On Tue, May 12, 2009 at 2:48 PM, Nicolas Sylvain nsylv...@chromium.org wrote: On Tue, May 12, 2009 at 2:22 PM, Stephen White senorbla...@chromium.org wrote: I'm in the process of updating chromium to use tip-of-tree skia, and in the same CL, moving skia to a third_party directory, retrieved

[chromium-dev] Re: Passing PlatformGraphicsContext from WebKit to Chrome code

2009-05-06 Thread Brett Wilson
2009/5/6 Andrew Scherkus scher...@chromium.org: On Tue, May 5, 2009 at 9:49 PM, Albert J. Wong (王重傑) ajw...@chromium.org wrote: On Tue, May 5, 2009 at 9:38 PM, Amanda Walker ama...@chromium.org wrote: Ah, I see. Hmm, going in that direction (from a GraphicsContext back up to the

[chromium-dev] Re: Passing PlatformGraphicsContext from WebKit to Chrome code

2009-05-05 Thread Brett Wilson
On Wed, May 6, 2009 at 8:40 AM, Andrew Scherkus scher...@chromium.org wrote: WebKit's MediaPlayerPrivate interface is a bit backwards where they pass in a GraphicsContext: http://src.chromium.org/viewvc/chrome/trunk/deps/third_party/WebKit/WebCore/platform/graphics/MediaPlayer.h?view=markup

[chromium-dev] Re: unforking: 20% perf hit for international page cycler expected soon

2009-05-01 Thread Brett Wilson
On Sat, May 2, 2009 at 6:09 AM, Evan Martin e...@chromium.org wrote: The suggestions on that code review are good: we ought to measure how many fonts normal users see, and then pick the cache tuning parameter accordingly. Adam Barth is a good person to ask about how to do this, since he

[chromium-dev] Re: Why are pref keys wchar_t's?

2009-05-01 Thread Brett Wilson
2009/5/1 Mohamed Mansour m0.interact...@gmail.com: Why wouldn't we just use std::string ? Many places in the code uses std::string. DictionaryValue needs to be converted as well as many others. So what do we finally decide, go what Pink stated and use char* or use std::string. I believe the

[chromium-dev] Re: Suggestion for crossplatform ProcessSingleton and ChromeBrowserProcessId()

2009-04-27 Thread Brett Wilson
On Mon, Apr 27, 2009 at 8:05 AM, Mike Pinkerton pinker...@chromium.org wrote: On Sun, Apr 26, 2009 at 5:50 PM, Amanda Walker ama...@chromium.org wrote: Application startup is one of the areas where we count every millisecond, and try to touch the disk as little as possible.  I don't think

[chromium-dev] WebContents is dead, long live TabContents!

2009-04-24 Thread Brett Wilson
I just moved the remaining methods from WebContents to TabContents. I kept the methods and data members the same, just merged the classes, so it should be easy to find what you're looking for (or to resolve conflicts if you have local changes to web_contents.cc). Hopefully this will make the

[chromium-dev] Re: DO NOT ADD DEPENDENCIES FROM CHROME/VIEWS on CHROME/COMMON

2009-04-24 Thread Brett Wilson
On Fri, Apr 24, 2009 at 10:10 PM, Aaron Boodman a...@chromium.org wrote: Is it possible to enforce these rules with code rather than capital letters? I don't think I've made this particular error, but it's something I could see myself doing. From their names, chrome/common seems like a

[chromium-dev] Re: question on profiles ChromeThread

2009-04-13 Thread Brett Wilson
On Mon, Apr 13, 2009 at 11:01 AM, Tim Steele t...@chromium.org wrote: I don't know of a reason we need more than 1 history thread, but I can't say for sure. HistoryService is responsible today for starting/stopping/destroying the history thread, so if we have multiple services then we need

[chromium-dev] Re: [linux] GTK and multi-threading

2009-04-10 Thread Brett Wilson
On Fri, Apr 10, 2009 at 12:42 PM, Evan Stade est...@chromium.org wrote: a) wrap the clipboard calls in gtk_threads_{enter,leave}(). This makes the calls thread safe, but it does so by locking a process-wide mutex, which will block our UI thread's main loop. I personally think this is OK.

[chromium-dev] Re: [Extensions] New API specs: bookmarks, history, downloads

2009-04-09 Thread Brett Wilson
On Wed, Apr 8, 2009 at 7:48 PM, Nick Baum nickb...@chromium.org wrote: Hi all, I fleshed out a few more APIs. I've put them in separate documents since the API pattern doc was getting a bit long. Below are some notes, feedback appreciated. In particular, I'd love feedback from Scott on

[chromium-dev] Re: How to fix check deps failure?

2009-03-30 Thread Brett Wilson
On Mon, Mar 30, 2009 at 10:12 AM, Jian Li jia...@chromium.org wrote: I got another failure for checkdeps: ERROR in C:/b/slave/chromium-rel-builder/build/src\chrome\test\worker\test_webworker.cc Illegal include: webkit/tools/test_shell/test_webworker_helper.h     Because of -webkit from

[chromium-dev] Re: Sandbox Interception Design

2009-03-27 Thread Brett Wilson
On Fri, Mar 27, 2009 at 1:50 PM, Book'em Dano daniel.c...@gmail.com wrote: At the top of sandbox\src\interception.h, it refers the reader to: http://wiki/Main/ChromeSandboxInterceptionDesign. Does anyone know where this document can be found? The sandbox info on

[chromium-dev] Re: Should all automatable browser APIs have per-object unique IDs?

2009-03-26 Thread Brett Wilson
On Thu, Mar 26, 2009 at 9:41 AM, Aaron Boodman a...@chromium.org wrote: I'm starting to dig into implementing the tab and window API as a first example of all the other browser-automation type APIs we eventually want to implement for extensions. One of my early assumptions was that the APIs

[chromium-dev] Re: Should all automatable browser APIs have per-object unique IDs?

2009-03-26 Thread Brett Wilson
On Thu, Mar 26, 2009 at 10:09 AM, Aaron Boodman a...@chromium.org wrote: On Thu, Mar 26, 2009 at 9:52 AM, Brett Wilson bre...@chromium.org wrote: What would IDs refer to in history? I've only barely looked into the history data model, but I was hoping it would be a persistent ID for each

[chromium-dev] Re: Should all automatable browser APIs have per-object unique IDs?

2009-03-26 Thread Brett Wilson
On Thu, Mar 26, 2009 at 11:37 AM, Aaron Boodman a...@chromium.org wrote: On Thu, Mar 26, 2009 at 11:22 AM, Brett Wilson bre...@chromium.org wrote: Those are exposed, but I wonder which cases an extension needs to refer to them. They are internal IDs generated by sqlite. If you clear your

[chromium-dev] Re: Including Droid font in Chrome?

2009-03-25 Thread Brett Wilson
On Wed, Mar 25, 2009 at 10:16 AM, Alex Russell slightly...@google.com wrote: MSFT did a new set of fonts to take specific advantage of ClearType: http://www.microsoft.com/typography/ClearTypeFonts.mspx The Core Fonts are still available and installed by default on all windows versions that

[chromium-dev] Re: ScrollBar Implementation

2009-03-25 Thread Brett Wilson
As Peter said, the ScrollBar object is not responsible for drawing scroll bars on pages, which is what the bug is about. Brett On Wed, Mar 25, 2009 at 1:15 PM, Mohamed Mansour m0.interact...@gmail.com wrote: There is no context menu for the ScrollBar. -m0 On Wed, Mar 25, 2009 at 3:55

[chromium-dev] Re: Omnibox q around Mac+Chromium best-practices.

2009-03-19 Thread Brett Wilson
I would do the thin Objective C mode, partially since all Chrome developers know C++. I could debug that code or make changes to it if I was doing something that affected it, but I would have a much harder time with Objective-C. Brett On Thu, Mar 19, 2009 at 4:00 PM, Scott Hess

[chromium-dev] Re: rev 11763 removes frame pointer from ResourceRequest

2009-03-16 Thread Brett Wilson
On Mon, Mar 16, 2009 at 4:25 PM, Marshall Greenblatt magreenbl...@gmail.com wrote: My needs in this case are related to the Chromium Embedded Framework (CEF). The code currently works as follows: 1. Retrieve our custom WebViewDelegate instance pointer via the WebFrame pointer passed to

[chromium-dev] Re: Doing some research with views and GTK.

2009-03-15 Thread Brett Wilson
On Sun, Mar 15, 2009 at 7:37 AM, Dean McNamee de...@chromium.org wrote: I had a discussion about Views with Scott.  I think I am on the side of the fence that porting views it not a good idea.  One of the things that came up is remote X, would it be possible to ever have good remote X

[chromium-dev] Re: Doing some research with views and GTK.

2009-03-15 Thread Brett Wilson
On Sun, Mar 15, 2009 at 10:09 AM, Adam Langley a...@chromium.org wrote: On Sun, Mar 15, 2009 at 8:55 AM, Dean McNamee de...@chromium.org wrote: UI elements like buttons are different, my understanding is that (at least in theory) it should be possible for all of those images of the different

[chromium-dev] Re: Thoughts on copy/paste

2009-03-11 Thread Brett Wilson
On Tue, Mar 10, 2009 at 7:44 AM, Avi Drissman a...@google.com wrote: I actually don't know. It seems like just simple state gathering but it might be more. The big concern is that it fits our model but no one else's. Everyone else is OK with the UI asking the renderer questions directly, and

[chromium-dev] Re: Thoughts on copy/paste

2009-03-11 Thread Brett Wilson
On Wed, Mar 11, 2009 at 10:01 AM, Avi Drissman a...@google.com wrote: Most platforms (meaning not Chrome) have the ability, when the user pulls down a menu, to adjust the states of the menu items within. For them, when the user selects the Edit menu, they're OK calling into WebCore and asking

[chromium-dev] Re: Thoughts on copy/paste

2009-03-11 Thread Brett Wilson
On Wed, Mar 11, 2009 at 10:11 AM, Amanda Walker ama...@chromium.org wrote: On Wed, Mar 11, 2009 at 2:07 PM, Brett Wilson bre...@chromium.org wrote: Do you mean when you click on the menu, blocking the browser until the renderer responds with whether the menus can be enabled? I would

[chromium-dev] Re: Unit tests and anonymous namespaces.

2009-03-03 Thread Brett Wilson
On Tue, Mar 3, 2009 at 3:55 PM, Scott Hess sh...@chromium.org wrote: On the Mac, code like this: namespace { class MyTest : public testing::Test { }; }  // namespace TEST_F(MyTest, ATest) { } generates errors like this: warning: ‘MyTest_ATest_Test’ has a field

[chromium-dev] Re: Unit tests and anonymous namespaces.

2009-03-03 Thread Brett Wilson
On Tue, Mar 3, 2009 at 4:49 PM, William Chan (陈智昌) willc...@chromium.org wrote: My old team never really used FRIEND_TEST. We found it ugly that our production code depended on test code. We typically used friended Peer classes defined in the unittest file, but not in the anonymous

[chromium-dev] Re: checked in file third_party/libxml/win32/include/xmlversion.h [Attn: Daniel Veillard]

2009-02-26 Thread Brett Wilson
On Thu, Feb 26, 2009 at 7:59 PM, eager_learner vijay.sankar.ra...@gmail.com wrote: Hello I am trying to integrate test a third party library available in source into the Visual Studio 2008 Pro environment. The problem seems to be that the  third_party/libxml/win32/include/xmlversion.h file

[chromium-dev] Re: Is there a scroll bar?

2009-02-17 Thread Brett Wilson
On Tue, Feb 17, 2009 at 3:43 PM, Marc-Andre Decoste m...@google.com wrote: Salut, I'm looking for a way to know, at the BrowserView (or even at the RenderWidgetHostViewWin) level, if there is one, and only one scroll bar visible. I thought I could ask Windows with ::GetScrollBarInfo, but

[chromium-dev] Re: How many GDI handles are reasonable?

2009-02-05 Thread Brett Wilson
On Thu, Feb 5, 2009 at 9:29 PM, Ian Fette i...@chromium.org wrote: Once again, my laptop is refusing to stand by and Windows is throwing up warning dialogs about Insufficient system resources exist to complete the API. Currently Chrome is sucking down a healthy 1043 gdi handles according to

[chromium-dev] Re: hash_maps, hash functions and gcc

2009-02-04 Thread Brett Wilson
2009/2/4 Paweł Hajdan Jr. phajdan...@chromium.org: I recently tried to port chrome/browser/safe_browsing/protocol_manager.cc to Linux, but there's one problem. ProtocolManager uses hash_map of URLFetcher*-s, and GCC needs a hash function for it. I see many ways we can deal with that, and I

[chromium-dev] Re: [linux] stracing chrome binaries

2009-02-04 Thread Brett Wilson
On Wed, Feb 4, 2009 at 2:58 PM, Adam Langley a...@chromium.org wrote: If you've ever tried running strace on our binaries, you've probably noticed the Ubuntu version messes up a lot because it cannot decode the 32-bit structures correctly. For example: [pid 21205] ... recvmsg resumed

[chromium-dev] Re: build problem (chromium.r9040.tar.gz)

2009-02-03 Thread Brett Wilson
On Tue, Feb 3, 2009 at 9:05 PM, t...@chromium.org wrote: What evan means is that after downloading the tar ball, you need to run gclient sync to get all the platform specific dependencies. We recently started generating the source tar ball on a regular basis and it doesn't include all the

[chromium-dev] Responsibility reviewing patches with multiple reviewers

2009-01-27 Thread Brett Wilson
There are a lot of patches lately with a lot of reviewers on them, especially related to porting since a lot of people might need to be in the loop for some changes. The problem is that there's no clear responsibility given in these reviews. If I'm the sole reviewer on a change, I know I have to

[chromium-dev] Re: Layout test and font stroking issue - help wanted

2009-01-16 Thread Brett Wilson
On Fri, Jan 16, 2009 at 9:32 AM, Sverrir Á. Berg sver...@google.com wrote: I'm running these tests: http://trac.webkit.org/export/39759/trunk/LayoutTests/fast/text/stroking-decorations.html http://trac.webkit.org/export/39759/trunk/LayoutTests/fast/text/stroking.html It seems that our font

[chromium-dev] 2-day merges and the cleanup schedule

2009-01-15 Thread Brett Wilson
I'm currently doing a 2-day merge rotation.As part of this, various layout tests are regressing or getting added that I'm inserting into the tests_fixable list. But, like every other layout test fixer, after my merges are done, I'll finally go back to my old work and not think about it any more.

[chromium-dev] Re: 2-day merges and the cleanup schedule

2009-01-15 Thread Brett Wilson
On Thu, Jan 15, 2009 at 1:20 PM, Pam Greene p...@chromium.org wrote: When fixing layout tests only means re-baselining, that's easy. But sometimes they break (or new ones fail) for deeper reasons, and the person doing the merge may not be the right one to make the fix (or may not be able to

[chromium-dev] Re: Making sense of startup

2009-01-15 Thread Brett Wilson
On Thu, Jan 15, 2009 at 1:19 PM, Amanda Walker ama...@chromium.org wrote: That's true. In the example I gave (Mac driver loading), each module has a property list that lists its dependencies (and version requirements, etc.). That's not quite as simple to do inside a single application, of

[chromium-dev] Re: fixing CommandLine

2009-01-14 Thread Brett Wilson
On Wed, Jan 14, 2009 at 6:22 PM, Evan Martin e...@chromium.org wrote: Our CommandLine class is very confusing -- it is not a class for working with command lines, but in fact a stealth singleton that wraps the command line used to start the process. Further, since it came from Windows, it

  1   2   >