Re: [webkit-dev] Comments in the code (Was Please include function-level comments in change log entries)

2012-07-11 Thread John Mellor
On Fri, Jul 6, 2012 at 11:49 PM, Ryosuke Niwa rn...@webkit.org wrote: On Jul 6, 2012 3:16 PM, Per Bothner per.both...@oracle.com wrote: On 07/06/2012 02:02 PM, Ryosuke Niwa wrote: On Fri, Jul 6, 2012 at 12:54 PM, Per Bothner per.both...@oracle.com wrote: You're deluding yourself if you

Re: [webkit-dev] Comments in the code (Was Please include function-level comments in change log entries)

2012-07-11 Thread Ryosuke Niwa
On Wed, Jul 11, 2012 at 6:54 AM, John Mellor joh...@chromium.org wrote: On Fri, Jul 6, 2012 at 11:49 PM, Ryosuke Niwa rn...@webkit.org wrote: On Jul 6, 2012 3:16 PM, Per Bothner per.both...@oracle.com wrote: On 07/06/2012 02:02 PM, Ryosuke Niwa wrote: On Fri, Jul 6, 2012 at 12:54 PM, Per

Re: [webkit-dev] Comments in the code (Was Please include function-level comments in change log entries)

2012-07-11 Thread John Mellor
On Wed, Jul 11, 2012 at 4:21 PM, Ryosuke Niwa rn...@webkit.org wrote: On Wed, Jul 11, 2012 at 6:54 AM, John Mellor joh...@chromium.org wrote: Even obvious (to some) concepts like InlineBox have subtleties, for example not all inline-level elements have inline boxes. An unambiguous

Re: [webkit-dev] Comments in the code (Was Please include function-level comments in change log entries)

2012-07-11 Thread Ojan Vafai
On Wed, Jul 11, 2012 at 8:43 AM, John Mellor joh...@chromium.org wrote: On Wed, Jul 11, 2012 at 4:21 PM, Ryosuke Niwa rn...@webkit.org wrote: On Wed, Jul 11, 2012 at 6:54 AM, John Mellor joh...@chromium.org wrote: Even obvious (to some) concepts like InlineBox have subtleties, for example

Re: [webkit-dev] Comments in the code (Was Please include function-level comments in change log entries)

2012-07-11 Thread Ryosuke Niwa
On Jul 11, 2012 8:43 AM, John Mellor joh...@chromium.org wrote: On Wed, Jul 11, 2012 at 4:21 PM, Ryosuke Niwa rn...@webkit.org wrote: On Wed, Jul 11, 2012 at 6:54 AM, John Mellor joh...@chromium.org wrote: Even obvious (to some) concepts like InlineBox have subtleties, for example not all

Re: [webkit-dev] Comments in the code (Was Please include function-level comments in change log entries)

2012-07-11 Thread Yaar Schnitman
[(dev time of maintaining comments) + (risk of outdated comments causing bugs X dev time of fixing resulting bugs)] (dev time gained by more contributors each being more knowledgable) No? On Wed, Jul 11, 2012 at 8:53 AM, Ryosuke Niwa rn...@webkit.org wrote: On Jul 11, 2012 8:43 AM, John

Re: [webkit-dev] Comments in the code (Was Please include function-level comments in change log entries)

2012-07-11 Thread Ryosuke Niwa
On Wed, Jul 11, 2012 at 9:30 AM, Yaar Schnitman y...@chromium.org wrote: [(dev time of maintaining comments) + (risk of outdated comments causing bugs X dev time of fixing resulting bugs)] (dev time gained by more contributors each being more knowledgable) No? How did you reach such a

Re: [webkit-dev] Comments in the code (Was Please include function-level comments in change log entries)

2012-07-11 Thread Adam Klein
On Wed, Jul 11, 2012 at 9:51 AM, Ryosuke Niwa rn...@webkit.org wrote: On Wed, Jul 11, 2012 at 9:30 AM, Yaar Schnitman y...@chromium.org wrote: [(dev time of maintaining comments) + (risk of outdated comments causing bugs X dev time of fixing resulting bugs)] (dev time gained by more

Re: [webkit-dev] Comments in the code (Was Please include function-level comments in change log entries)

2012-07-11 Thread Per Bothner
On 07/11/2012 09:30 AM, Yaar Schnitman wrote: [(dev time of maintaining comments) + (risk of outdated comments causing bugs X dev time of fixing resulting bugs)] (dev time gained by more contributors each being more knowledgable) No? Perhaps: [(dev time of maintaining comments) + (risk of

Re: [webkit-dev] Comments in the code (Was Please include function-level comments in change log entries)

2012-07-11 Thread Ryosuke Niwa
On Wed, Jul 11, 2012 at 10:44 AM, Adam Klein ad...@chromium.org wrote: On Wed, Jul 11, 2012 at 9:51 AM, Ryosuke Niwa rn...@webkit.org wrote: On Wed, Jul 11, 2012 at 9:30 AM, Yaar Schnitman y...@chromium.org wrote: [(dev time of maintaining comments) + (risk of outdated comments causing

Re: [webkit-dev] New feature flag: ENABLE_CSS_POSITION_STICKY

2012-07-11 Thread Simon Fraser
I think we've resolved our differences around the fixed position scrolling issues, so that's OK. Simon On Jun 27, 2012, at 6:38 PM, Adam Barth aba...@webkit.org wrote: There seems to be some amount of controversy at the moment regarding how we should implement scrolling and fixed position

Re: [webkit-dev] New feature flag: ENABLE_CSS_POSITION_STICKY

2012-07-11 Thread Adam Barth
Yes. Thanks for holding off for a bit. :) Adam On Wed, Jul 11, 2012 at 11:03 AM, Simon Fraser simon.fra...@apple.comwrote: I think we've resolved our differences around the fixed position scrolling issues, so that's OK. Simon On Jun 27, 2012, at 6:38 PM, Adam Barth aba...@webkit.org

Re: [webkit-dev] highlighting a frame

2012-07-11 Thread W. James MacLean
If you're just looking to draw a border, will the focus rings mechanism suffice? On Tue, Jul 10, 2012 at 1:37 AM, Patrick East patri...@bsquare.com wrote: Hello, We are working on a custom build of WebKit that will highlight a frame when the mouse moves over it (the intent is to have it

[webkit-dev] C++ unit tests for WebKit?

2012-07-11 Thread Hans Muller
Bear Travis, Alexandru Chiculita, and I have been working on the WebKit CSS Exclusions implementation. I've been working on a set of C++ classes that are used to compute the layout of inline content around or within exclusion shapes. When I got started, before attempting to integrate the

Re: [webkit-dev] C++ unit tests for WebKit?

2012-07-11 Thread Benjamin Poulain
On Wed, Jul 11, 2012 at 2:23 PM, Hans Muller hmul...@adobe.com wrote: Have the merits of C++ unit tests been debated before? I noticed that a bug representing as much has been around since 2008 (https://bugs.webkit.org/show_bug.cgi?id=21010). The last comment (2009) concludes with: I'd be

Re: [webkit-dev] C++ unit tests for WebKit?

2012-07-11 Thread Konrad Piascik
Don't we have gunit tests in Tools/TestWebKitAPI already? From: webkit-dev-boun...@lists.webkit.org [webkit-dev-boun...@lists.webkit.org] on behalf of Hans Muller [hmul...@adobe.com] Sent: Wednesday, July 11, 2012 5:23 PM To: webkit-dev Subject:

Re: [webkit-dev] Removing BUILDING_ON / TARGETING macros in favor of system availability macros

2012-07-11 Thread Mark Rowe
On 2012-07-10, at 16:24, Mark Rowe mr...@apple.com wrote: I'm open to feedback on this proposal, but I'd like to move forward with this change in the next day or two if no one objects. Given the lack of outcry I've posted patches on https://bugs.webkit.org/show_bug.cgi?id=91015 that make

Re: [webkit-dev] Comments in the code (Was Please include function-level comments in change log entries)

2012-07-11 Thread Alec Flett
I'm gonna chime in here, as I am 3 months into working on WebKit and the lack of comments has driven me absolutely crazy. This is a bit of a rant from a self-declared newbie, but I'm trying to be constructive. I've contributed to a handful of open and closed source projects. I am trying to help,

Re: [webkit-dev] C++ unit tests for WebKit?

2012-07-11 Thread Hans Muller
It looks like Tools/TestWebKitAPI/Tests/WebCore just contains one small KURL test. I could certainly add more for the Exclusions shape classes, but I got that feeling that maybe you get about being the first to take a table at an empty restaurant. Perhaps it's just that most of the C++ classes in

Re: [webkit-dev] C++ unit tests for WebKit?

2012-07-11 Thread Adam Barth
Generally speaking, WebKit's testing philosophy is to test at API boundaries, typically either a given port's WebKit API or the web platform API. The benefit of that approach is that it makes it easier for us to refactor the internals of WebCore without being constrained by fragile tests---only

Re: [webkit-dev] C++ unit tests for WebKit?

2012-07-11 Thread Hans Muller
Thanks for the thoughtful reply. It was exactly the community's conventions and customs that I was trying to get a handle on. Not to put too fine a point on it, but I assume that adding unit tests to TestWebKitAPI or writing tests that depend on APIs defined in Internals.idl wouldn't be

Re: [webkit-dev] C++ unit tests for WebKit?

2012-07-11 Thread Adam Barth
On Wed, Jul 11, 2012 at 4:31 PM, Hans Muller hmul...@adobe.com wrote: Thanks for the thoughtful reply. It was exactly the community's conventions and customs that I was trying to get a handle on. Not to put too fine a point on it, but I assume that adding unit tests to TestWebKitAPI or

Re: [webkit-dev] C++ unit tests for WebKit?

2012-07-11 Thread Hajime Morrita
(Resending from the right address) FYI, Chromium port has some unit-like tests: http://trac.webkit.org/browser/trunk/Source/WebKit/chromium/tests Apparently it is not community's responsibility to maintain these. But it might be useful for developing some low-level standalone/low-level stuff

Re: [webkit-dev] Can protocol handler be moved to the modules ?

2012-07-11 Thread Gyuyoung Kim
I file a bug to extract a client interface for register protocol handler from ChromeClient. - Bug 90940 - Add ProtocolHandlerClient.h to the Modules/protocolhandler (https://bugs.webkit.org/show_bug.cgi?id=90940) In order to support this, the client needs to be supplementable first. I'm

Re: [webkit-dev] Removing BUILDING_ON / TARGETING macros in favor of system availability macros

2012-07-11 Thread Mark Mentovai
Tony brought me in to comment on what impact this might have on the Chromium Mac build. It shouldn’t have any impact. Any use of the compiler-defined macros is fine. In Chrome code, we usually use MAC_OS_X_VERSION_MAX_ALLOWED and MAC_OS_X_VERSION_MIN_REQUIRED from AvailabilityMacros.h, along with

Re: [webkit-dev] Removing BUILDING_ON / TARGETING macros in favor of system availability macros

2012-07-11 Thread Mark Rowe
On 2012-07-11, at 17:46, Mark Mentovai m...@chromium.org wrote: Tony brought me in to comment on what impact this might have on the Chromium Mac build. It shouldn’t have any impact. Any use of the compiler-defined macros is fine. I agree. The only impact I expect this to have is if I've

Re: [webkit-dev] Removing BUILDING_ON / TARGETING macros in favor of system availability macros

2012-07-11 Thread Mark Mentovai
Mark Rowe wrote: TN2064 appears to have been last modified in 2003, so it predates the existence of Availability.h. Well, that’s about as long as I’ve been writing SDK- and deployment-target-aware code… Availability.h was introduced with the iOS SDK in order to support availability macros