Re: [webkit-dev] Asking for review by pinging bugs---another approach

2012-01-05 Thread Andreas Kling
On Thu, Jan 5, 2012 at 7:49 AM, Adam Barth aba...@webkit.org wrote: Not to pick on anyone in particular, but when reading bugmail I occasionally see messages like pinging for review. I review a lot of patches, but I don't find these messages particularly helpful because I don't know whether

Re: [webkit-dev] Tools/Scripts/build-webkit --gtk --debug --makeargs=-j1 taking up to 80% of memory

2012-01-05 Thread Soo-Hyun Choi
Sachin, On Thu, Jan 5, 2012 at 15:18, sachin nikam skni...@gmail.com wrote: i am running ubuntu 11.10 64 bit. I was able to eventually build with --makeargs=-j1. I will try increase the swap space and see if it helps. WebKit Debug build over Ubuntu 64bit machines should've been fine without

Re: [webkit-dev] Asking for review by pinging bugs---another approach

2012-01-05 Thread Hajime Morrita
Does it make sense for webkit-patch post/upload to support --ping to acknowledge a set of reviewers using a specific form of comment? As a reviewer, some kind of specific pattern would be helpful to filter such ping mails which contain, for example, ping:morrita@. Maybe this can be taken as a

Re: [webkit-dev] Asking for review by pinging bugs---another approach

2012-01-05 Thread Adam Barth
Sure. Feel free to add that feature. Obviously we don't want to force people to use it, but it might become popular. Adam On Jan 5, 2012 3:12 AM, Hajime Morrita morr...@chromium.org wrote: Does it make sense for webkit-patch post/upload to support --ping to acknowledge a set of reviewers

Re: [webkit-dev] layout tests: how are some compared against image, and others only text?

2012-01-05 Thread Stephen White
On Wed, Jan 4, 2012 at 6:39 PM, Elliot Poger epo...@chromium.org wrote: What is it that causes some tests to require baseline images (and not text files) for comparison, while others require text and not image baselines? (I know that I can specifically SKIP comparison against IMAGE and/or

[webkit-dev] Selection model is broken

2012-01-05 Thread Ojan Vafai
Selections on the web are document ordered (i.e. all the DOM between two points). So, any CSS that puts elements out of document order gives a crappy selection to the user. div style=position:relativefoodiv style=position:absolute;left:-100pxbar/divbaz/div. If you select foobaz, clearly no user

Re: [webkit-dev] Selection model is broken

2012-01-05 Thread Benjamin Poulain
On Thu, Jan 5, 2012 at 3:31 PM, Ojan Vafai o...@chromium.org wrote: Is this too crazy? Are there other solutions? I also do not like the selection model but I am unsure about your suggestion. If we use the visual model for selection, finding the order of the content would get tricky (because

Re: [webkit-dev] Selection model is broken

2012-01-05 Thread Ryosuke Niwa
On Thu, Jan 5, 2012 at 3:31 PM, Ojan Vafai o...@chromium.org wrote: Selections on the web are document ordered (i.e. all the DOM between two points). So, any CSS that puts elements out of document order gives a crappy selection to the user. div style=position:relativefoodiv

Re: [webkit-dev] Selection model is broken

2012-01-05 Thread Ojan Vafai
On Thu, Jan 5, 2012 at 3:47 PM, Ryosuke Niwa rn...@webkit.org wrote: On Thu, Jan 5, 2012 at 3:31 PM, Ojan Vafai o...@chromium.org wrote: Selections on the web are document ordered (i.e. all the DOM between two points). So, any CSS that puts elements out of document order gives a crappy

[webkit-dev] Understanding WebKit layering and layering violations

2012-01-05 Thread Fady Samuel
Hi all, I've been working on WebKit off and on for a while now but I must admit that, up to this point, I still don't have a firm grasp of all the layering in WebKit. What depends on what, and what cannot depend on what? What is the motivation of each of these individual layers? I understand the

Re: [webkit-dev] Selection model is broken

2012-01-05 Thread Ryosuke Niwa
On Thu, Jan 5, 2012 at 4:03 PM, Ojan Vafai o...@chromium.org wrote: On Thu, Jan 5, 2012 at 3:47 PM, Ryosuke Niwa rn...@webkit.org wrote: On Thu, Jan 5, 2012 at 3:31 PM, Ojan Vafai o...@chromium.org wrote: Selections on the web are document ordered (i.e. all the DOM between two points). So,

Re: [webkit-dev] Selection model is broken

2012-01-05 Thread Ojan Vafai
On Thu, Jan 5, 2012 at 3:44 PM, Benjamin Poulain benja...@webkit.orgwrote: On Thu, Jan 5, 2012 at 3:31 PM, Ojan Vafai o...@chromium.org wrote: Is this too crazy? Are there other solutions? I also do not like the selection model but I am unsure about your suggestion. If we use the visual

Re: [webkit-dev] Understanding WebKit layering and layering violations

2012-01-05 Thread Adam Barth
On Thu, Jan 5, 2012 at 4:18 PM, Fady Samuel fsam...@chromium.org wrote: I've been working on WebKit off and on for a while now but I must admit that, up to this point, I still don't have a firm grasp of all the layering in WebKit. What depends on what, and what cannot depend on what? What is

Re: [webkit-dev] Selection model is broken

2012-01-05 Thread Ojan Vafai
Another related bug https://bugs.webkit.org/show_bug.cgi?id=62664. On Thu, Jan 5, 2012 at 4:30 PM, Ojan Vafai o...@chromium.org wrote: On Thu, Jan 5, 2012 at 3:44 PM, Benjamin Poulain benja...@webkit.orgwrote: On Thu, Jan 5, 2012 at 3:31 PM, Ojan Vafai o...@chromium.org wrote: Is this too

Re: [webkit-dev] Selection model is broken

2012-01-05 Thread Ojan Vafai
Sorry, that was too terse, https://bugs.webkit.org/show_bug.cgi?id=62664 is related in that we also should consider making tabindex order depend on visual ordering. On Thu, Jan 5, 2012 at 7:05 PM, Ojan Vafai o...@chromium.org wrote: Another related bug

[webkit-dev] question on Structure.

2012-01-05 Thread PandaCanFly
Hi, all. I am reading the implementation of JavascriptCore. I think Class Structure, Class JSCell, Class JSObject are the point to understand other codes. The member functions of Structure whose name is addPropertyTransition, addPropertyTransitionToExistingStructure, really confuse me.