Re: [webkit-dev] Approving / Rejects PRs on GitHub when not a reviewer?

2023-11-28 Thread Tim Horton via webkit-dev
+1! The bugzilla-style “unofficial r=me” comment was much clearer for exactly these reasons. > On Nov 28, 2023, at 10:27 AM, Chris Dumez via webkit-dev > wrote: > > Hi, > > Back in the Bugzilla days, only reviewers were allowed to r+ or r- patches. > Non-reviewers were - of course - encourag

Re: [webkit-dev] Smart Pointers Usage Guidelines

2023-08-21 Thread Tim Horton via webkit-dev
> On Aug 21, 2023, at 4:39 PM, Ryosuke Niwa wrote: > > >> On Aug 21, 2023, at 4:34 PM, Darin Adler wrote: >> >> >>> On Aug 21, 2023, at 4:31 PM, Tim Horton via webkit-dev >>> wrote: >>> >>>> One subtle thing is that even

Re: [webkit-dev] Smart Pointers Usage Guidelines

2023-08-21 Thread Tim Horton via webkit-dev
> On Aug 21, 2023, at 4:34 PM, Darin Adler wrote: > > >> On Aug 21, 2023, at 4:31 PM, Tim Horton via webkit-dev >> wrote: >> >>> One subtle thing is that even when a member variable is already Ref / >>> RefPtr / CheckedRef / CheckedPtr, we

Re: [webkit-dev] Smart Pointers Usage Guidelines

2023-08-21 Thread Tim Horton via webkit-dev
> On Aug 21, 2023, at 4:25 PM, Ryosuke Niwa via webkit-dev > wrote: > > Hi all, > > It has been a while since I last announced the plan to adopt smart pointers > using clang static analyzer: > https://lists.webkit.org/pipermail/webkit-dev/2020-September/031386.html > > Here are some updates.

Re: [webkit-dev] OK to flatten WTF's header directory?

2022-02-01 Thread Tim Horton via webkit-dev
> On Feb 1, 2022, at 12:30 PM, Elliott Williams via webkit-dev > wrote: > > Hi webkit-dev, > > I’m working on fixing some ambiguities in our Xcode projects to permit > adoption of Xcode’s new build system and better parallelize our builds. I > noticed that WTF’s headers (/usr/lib/include/wt

[webkit-dev] Aditya Keerthi is now a WebKit reviewer!

2021-10-28 Thread Tim Horton via webkit-dev
Hello! I am delighted to announce that Aditya Keerthi is now a WebKit reviewer. Now it's time for all of you to send him tricky patches and see how he does :) Congratulations, Aditya! --Tim ___ webkit-dev mailing list webkit-dev@lists.webkit.org https

Re: [webkit-dev] New EWS Non-Unified builder

2021-04-29 Thread Tim Horton via webkit-dev
> On Apr 29, 2021, at 9:02 PM, Darin Adler wrote: > >> On Apr 29, 2021, at 9:01 PM, Tim Horton wrote: >> >> This is a huge problem for people on the Apple platforms using the Xcode >> projects. Nearly every time someone adds a file they have to add random

Re: [webkit-dev] New EWS Non-Unified builder

2021-04-29 Thread Tim Horton via webkit-dev
> On Apr 29, 2021, at 7:15 PM, Darin Adler via webkit-dev > wrote: > >> On Apr 29, 2021, at 12:04 PM, Ross Kirsling via webkit-dev >> wrote: >> >> Yeah, I think it's important to clarify that nobody is "using >> non-Unified-Source building for their development", at least to my >> knowled

Re: [webkit-dev] Having trouble building

2019-10-05 Thread Tim Horton
> On Oct 4, 2019, at 8:55 PM, Ken Russell wrote: > > Hi, > > Recently I upgraded to Xcode 11.0 and re-synced to top-of-tree WebKit. With a > clean WebKitBuild directory I'm seeing the following build error: What OS are you on? You might want to peek at https://bugs.webkit.org/show_bug.cgi?

Re: [webkit-dev] Moving to Python 3

2019-07-12 Thread Tim Horton
> On Jul 12, 2019, at 12:18 PM, Jonathan Bedard wrote: > > Hello WebKit developers, > > Now that the Catalina developer seeds are available, it is official that the > new Mac developer tools come with Python 3. As a result, we need to continue > the ongoing discussion about migrating our Pyt

Re: [webkit-dev] Which WTF headers still should be in C++11?

2019-06-19 Thread Tim Horton
This was a hopefully-shortlived internal complication. I think this is already fixed and we could probably revert this change. Is that right, Alex? > On Jun 19, 2019, at 7:38 PM, Fujii Hironori wrote: > > Hi, > > wtf/CheckedArithmetic.h has been converted from C++14 to C++11. > > Bug 195187 –

Re: [webkit-dev] Requesting feedback about EWS comments on Bugzilla bugs

2019-06-15 Thread Tim Horton
> On Jun 15, 2019, at 21:13, Aakash Jain wrote: > >  > Hi Everyone, > > I am gathering feedback about EWS - specially about the comments which EWS > makes on the Bugzilla bugs. Currently, the comments are not very > user-friendly/polished/readable and are sometimes very noisy (e.g.: 72 > c

Re: [webkit-dev] Unified source builds and adding or removing files...

2019-03-19 Thread Tim Horton
We already had this discussion; see the thread titled "Unified sources have broken our #include hygiene" > On Mar 19, 2019, at 3:48 PM, Said Abou-Hallawa wrote: > > I have been working on patches that require adding and removing cpp files > from WebCore/Sources.txt. Almost every time I add or

Re: [webkit-dev] Code Style: Opinion on returning void

2019-02-07 Thread Tim Horton
> On Feb 7, 2019, at 12:47 PM, Daniel Bates wrote: > > Hi all, > > Something bothers me about code like: > > void f(); > void g() > { > if (...) > return f(); > return f(); > } ⸘do people do this‽ > I prefer: > > void g() > { > if (...) { > f(); > return

Re: [webkit-dev] WebKit Commit Queue blocked by build errors in ToT

2018-12-07 Thread Tim Horton
t; >> On Dec 7, 2018, at 8:45 AM, Chris Fleizach wrote: >> >> Looks like there's quite a few of these. Will try to get them all >> >>> On Dec 7, 2018, at 8:37 AM, Tim Horton wrote: >>> >>> Someone in UnifiedSource28-mm.mm is “using n

Re: [webkit-dev] WebKit Commit Queue blocked by build errors in ToT

2018-12-07 Thread Tim Horton
Someone in UnifiedSource28-mm.mm is “using namespace WebCore” outside of the root namespace. > On Dec 7, 2018, at 08:31, Chris Fleizach wrote: > > I'm hitting this tougher one now. Namespace conflict between Rect in Carbon > and WebCore::Rect > > Any ideas? > Thanks for your help > > n file

Re: [webkit-dev] Unified sources have broken our #include hygiene

2018-09-01 Thread Tim Horton
> On Sep 1, 2018, at 17:14, Simon Fraser wrote: > > Unified sources allow you to get away without #including all the files you > need in a .cpp file, because some earlier file in the group has probably > already included them for you. > > This means that when you add a new file to the build

Re: [webkit-dev] Experimental features enabled by default

2018-06-27 Thread Tim Horton
Alright, how’s it looking now? :) > On Jun 26, 2018, at 19:06, Tim Horton wrote: > > > >> On Jun 26, 2018, at 18:55, Michael Catanzaro wrote: >> >> >> Hi, >> >> We're a lot closer after r233227! >> >> There is still IsSec

Re: [webkit-dev] Experimental features enabled by default

2018-06-26 Thread Tim Horton
> On Jun 26, 2018, at 18:55, Michael Catanzaro wrote: > > > Hi, > > We're a lot closer after r233227! > > There is still IsSecureContextAttributeEnabled. Shall we switch the default > value of that one to use DEFAULT_EXPERIMENTAL_FEATURES_ENABLED instead of > true? Not sure, waiting for o

Re: [webkit-dev] Experimental features enabled by default

2018-06-20 Thread Tim Horton
Resending from the right address. > On Jun 20, 2018, at 11:12 AM, Michael Catanzaro wrote: > > Hi, Apple folks have been chatting about this elsewhere, so it’s a humorous coincidence to see this here at the same time. > I noticed in glancing over WebPreferences.yaml that we have a large numbe

Re: [webkit-dev] Update WebKit style to better accommodate accepted Obj-C block syntax

2017-10-03 Thread Tim Horton
Whatever we decide, we should probably also get it in https://webkit.org/code-style-guidelines/ > On Oct 3, 2017, at 4:51 PM, Megan Gardner wrote: > > > Currently, the webkit style checker does not allow ^{ at the beginning of > Obj-C blocks. It demands that there is a space between the ^ and

Re: [webkit-dev] Are CSS constant properties and viewport fit features still experimental?

2017-09-22 Thread Tim Horton
> On Sep 22, 2017, at 12:08, Michael Catanzaro wrote: > > Hi, > > I noticed today that in our experimental features list (in > WebPreferencesDefinitions.h), CSS constant properties and viewport fit are > both enabled unconditionally. > > Should I move them to the non-experimental section of

Re: [webkit-dev] RFC: Removing feature flag defaults in build-webkit

2017-09-21 Thread Tim Horton
> On Sep 21, 2017, at 13:25, Michael Catanzaro wrote: > > On Thu, Sep 21, 2017 at 2:26 PM, Tim Horton wrote: >> Sure. And someday I think we should go down that path, but generating the >> xcconfigs will be trickier (but also a positive because people already have

Re: [webkit-dev] RFC: Removing feature flag defaults in build-webkit

2017-09-21 Thread Tim Horton
> On Sep 21, 2017, at 12:04, Konstantin Tokarev wrote: > > > > 21.09.2017, 22:01, "Tim Horton" : >>> On Sep 21, 2017, at 11:57, Konstantin Tokarev wrote: >>> >>> 21.09.2017, 21:54, "Tim Horton" : >>>> Hi, all

Re: [webkit-dev] RFC: Removing feature flag defaults in build-webkit

2017-09-21 Thread Tim Horton
> On Sep 21, 2017, at 11:57, Konstantin Tokarev wrote: > > > > 21.09.2017, 21:54, "Tim Horton" : >> Hi, all! >> >> Would anybody mind if build-webkit stopped being a source of default values >> of feature flags? >> >> We have de

[webkit-dev] RFC: Removing feature flag defaults in build-webkit

2017-09-21 Thread Tim Horton
Hi, all! Would anybody mind if build-webkit stopped being a source of default values of feature flags? We have defaults for feature flags in FeatureDefines.xcconfig for Xcode, WebKitFeatures.cmake (and Options*.cmake for platform overrides) for CMake, and FeatureDefines.h for everyone. This is

Re: [webkit-dev] Proposal: Remove CSS color-correction property

2015-07-01 Thread Tim Horton
> On Jul 1, 2015, at 23:36, Dirk Schulze wrote: > > Hi Darin, > >> On Jul 1, 2015, at 5:16 PM, Darin Adler wrote: >> >> Hi folks. >> >> WebKit has a CSS property named color-correction. It’s still prefixed, so >> some would call it -webkit-color-correction and I don’t think it’s yet been >

Re: [webkit-dev] WEBCORE_EXPORT

2015-02-23 Thread Tim Horton
They also seem like prime candidates for stylebot rules. > On Feb 23, 2015, at 3:31 PM, Darin Adler wrote: > > Hooray! > > Lets document these somewhere so people don’t have to read the webkit-dev > archives to learn these rules. Maybe in the header that defines > WEBCORE_EXPORT? Maybe in a w

Re: [webkit-dev] Abandoned Tests

2015-01-22 Thread Tim Horton
> On Jan 22, 2015, at 14:02, Benjamin Poulain wrote: > > On 1/21/15 8:48 PM, Brent Fulgham wrote: >> I’ve been reviewing the various Windows layout tests, and have run across a >> few tests that seem to be leftovers from the Chromium project. I think all >> ports currently skip these tests: >>

Re: [webkit-dev] Is commit-queue broken ?

2014-11-26 Thread Tim Horton
I think the necessary emails have been sent to resolve it, but everyone is on vacation :( I can't commit manually either. > On Nov 26, 2014, at 19:33, Gyuyoung Kim wrote: > > Hello, > > It looks WebKit queue doesn't work now. Does anyone check it ? > > Gyuyoung. > > > > -- > Sent from Gy

Re: [webkit-dev] Port-specific OpenType support

2014-09-14 Thread Tim Horton
On 2014.09.14, at 01:11, Dirk Schulze wrote: > > On Sep 11, 2014, at 11:03 PM, Martin Robinson wrote: > >> On Thu, Sep 11, 2014 at 11:44 AM, Myles C. Maxfield >> wrote: >>> My question to WebKit-Dev is: Are there any ports who: >>> 1) Want continued support for SVG fonts, and >>> 2) Whose p

Re: [webkit-dev] TestWebKitAPI doesn't initialize WKWebView from command line

2014-08-26 Thread Tim Horton
On 2014.08.26, at 04:44, Daniel Lazarenko wrote: > Hello, > > I’m trying to run the WKWebView API tests using this command: > ./TestWebKitAPI --gtest_filter=_WKDownload.DownloadRequest > This binary is located in webkit/WebKitBuild/Debug, I’ve built it using Xcode > beta 6 on Yosemite. You

Re: [webkit-dev] Please avoid inheriting concrete types over WebCore/WebKit boundary

2014-08-22 Thread Tim Horton
> On Aug 22, 2014, at 11:07, Antti Koivisto wrote: > > Hi, > > To make callbacks from WebCore to WebKit(2) we have generally used approach > where WebCore exports an abstract client interface which is then implemented > on the WebKit side. > > More recently there has been some proliferation

Re: [webkit-dev] Questions about ports

2014-08-06 Thread Tim Horton
> On Aug 5, 2014, at 23:48, Jacques-Olivier wrote: > > Hi everyone, > > I have a couple of questions regarding the different ports of WebKit: > > 1) I suppose the port used by Safari Mac OS X is the Apple’s mac port (built > by default) > Is there a planing of when a version of WebKit is bein

Re: [webkit-dev] Comment on the bug & email author/reviewer before reverting a patch

2014-07-09 Thread Tim Horton
> On Jul 9, 2014, at 12:10 PM, Maciej Stachowiak wrote: > > > Could we teach webkitbot to do an appropriate notification with a waiting > period? Either as part of rollout or add a new command to do it. It already does. The “waiting period” is defined by when the person who asked for the rol

Re: [webkit-dev] PSA: please sort WebCore.exp.in when you change it

2014-07-07 Thread Tim Horton
> On Jul 7, 2014, at 1:51 PM, Bem Jones-Bey wrote: > > > On Jul 7, 2014, at 11:55 , Tim Horton wrote: > >> >>> On Jul 7, 2014, at 11:41 AM, Simon Fraser wrote: >>> >>> We have a handy script that sorts the export file; pleas

Re: [webkit-dev] PSA: please sort WebCore.exp.in when you change it

2014-07-07 Thread Tim Horton
> On Jul 7, 2014, at 11:41 AM, Simon Fraser wrote: > > We have a handy script that sorts the export file; please use it whenever you > modify WebCore.exp.in: > > ./Tools/Scripts/sort-export-file Source/WebCore/WebCore.exp.in It’s also smart enough to require no arguments at all, and just sort

Re: [webkit-dev] PDF: context menu and view mode

2014-04-06 Thread Tim Horton
On Apr 6, 2014, at 2:00 AM, Rüdiger Cordes wrote: > Hello list, > > this is my first mail and I want so say "Hi" to you. > > The reason why I entered this list: > I would like to know why the context menu for opened > PDFs in Safari has changed after r153570 in r153759. The context menu for

Re: [webkit-dev] EWS doesn't lie!

2014-02-10 Thread Tim Horton
On Feb 10, 2014, at 12:51 AM, youenn fablet wrote: > Is it by design that only mac bots run regression tests? Certainly not. > Technical issue? I think so; look at http://build.webkit.org/dashboard/; the Mac port is the only one that’s even close to green enough to make running the tests on

Re: [webkit-dev] MathML

2013-12-20 Thread Tim Horton
On Dec 20, 2013, at 4:57 PM, Peter Frane wrote: > Why did my email bounce? I'm a subscribed. Thanks. It didn’t. > From: pfran...@hotmail.com > To: webkit-dev@lists.webkit.org > Date: Sat, 21 Dec 2013 08:15:07 +0800 > Subject: [webkit-dev] MathML > > To whom it may concern, > > I'm the develo

Re: [webkit-dev] Test flakiness.

2013-12-03 Thread Tim Horton
On Dec 3, 2013, at 1:51 PM, Samuel White wrote: > Hey everyone, > > I've noticed the following build message: > > Unexpected flakiness: text-only failures (1) > platform/mac/accessibility/search-predicate-element-count.html [ Failure > Pass ] > > Oddly, when this test is run ALONE (locally

Re: [webkit-dev] Layout Tests Failure

2013-10-09 Thread Tim Horton
On 2013.10.09, at 19:18, Gurpreet Kaur wrote: > address space needed by 'datetime.dll' (0x211) is already occupied http://cygwin.com/faq-nochunks.html#faq.using.fixing-fork-failures and http://cygwin.com/faq-nochunks.html#faq.using.bloda might be helpful. I’ve seen this before.__

Re: [webkit-dev] WebGL :: Win32 => build successful and rendering assumed so => but white page

2013-10-03 Thread Tim Horton
On 2013.10.03, at 09:06, Hugo Machefer wrote: > Dear all. Might not the best place -here- to describe my issue [Win32], but > I don't know where exactly this shall be submitted, as a novice by WebKit. To > recap, I managed to compile sources by Microsoft Visual > Studio 2010 using CoreGraph

Re: [webkit-dev] Webkit 2 drawing

2013-06-04 Thread Tim Horton
On Jun 4, 2013, at 10:18 PM, Vishvesh Community wrote: > Hi All, > I have been going through the webkit 2 code. But I am not able to > findout how the bitmaps are shared between the webprocess and the UI process. > Can anybody point me to the appropriate class or share some details on it

Re: [webkit-dev] Buildsystem cleanup

2013-04-08 Thread Tim Horton
On Apr 8, 2013, at 3:44 PM, Patrick Gansterer wrote: >> I wasn’t aware that the build worked without a VS2010 pThreads build. I was >> running into issues because of the pThreads dll having embedded manifest >> problems earlier. Did you do anything special to circumvent that issue? > > I comm

Re: [webkit-dev] Re-enabling pixel tests

2013-04-04 Thread Tim Horton
We should probably sort out some of our known and solvable OS-version-dependent color profile bugs *before* attempting any mass rebaselines, to avoid having to do them twice. On Apr 4, 2013, at 7:37 PM, Benjamin Poulain wrote: > On Thu, Apr 4, 2013 at 7:26 PM, Ryosuke Niwa wrote: > Now that a

Re: [webkit-dev] Please don't land patches without rebaselining tests for at least one platform

2013-03-22 Thread Tim Horton
On Mar 22, 2013, at 1:34 PM, Ryosuke Niwa wrote: > On Thu, Mar 21, 2013 at 6:43 PM, Ryosuke Niwa wrote: > On Thu, Mar 21, 2013 at 6:39 PM, Silvia Pfeiffer > wrote: > On Fri, Mar 22, 2013 at 12:30 PM, Ryosuke Niwa wrote: > On Thu, Mar 21, 2013 at 6:05 PM, Maciej Stachowiak wrote: > On Mar 21

Re: [webkit-dev] Proposal for coding guidelines: Do not use fall-through switch cases inside #ifdef's

2012-08-18 Thread Tim Horton
to true". The line is correct as it stands. > -Jake > > On Sat, Aug 18, 2012 at 8:20 PM, Tim Horton wrote: > > > On Aug 17, 2012, at 11:57 AM, Bruno Abinader wrote: > > > On Fri, Aug 17, 2012 at 2:24 PM, Joe Mason wrote: > >> What do you mean by &q

Re: [webkit-dev] Proposal for coding guidelines: Do not use fall-through switch cases inside #ifdef's

2012-08-18 Thread Tim Horton
On Aug 17, 2012, at 11:57 AM, Bruno Abinader wrote: > On Fri, Aug 17, 2012 at 2:24 PM, Joe Mason wrote: >> What do you mean by "precompiler"? The preprocessor? Or is this a >> precompiled headers thing? > > Indeed, s/precompiler/preprocessor :) > >> >> Are you sure there wasn't a typo in