Bug 76190 – [Qt] [PATCH] Use 'Ok' and 'Cancel' buttons in JavaScript confirm box.

• Component: WebKit Qt
• Status: RESOLVED FIXED

[Johannes Obermayr]

Created attachment 122267 [details]
Patch

Please use 'git am 0001-Use-Ok-and-Cancel-buttons-in-JavaScript-confirm-box.patch' to keep author information ...

Maybe you should create a qtwebkit-patches mailing list for a quick and comfortable sending patches via 'git send-email' ...

See: http://www.javascripter.net/faq/confirm.htm

Fixes: https://bugs.kde.org/show_bug.cgi?id=287629

[Alexander Færøy]

(In reply to comment #0)
> Please use 'git am 0001-Use-Ok-and-Cancel-buttons-in-JavaScript-confirm-box.patch' to keep author information ...

The WebKit project doesn't work like that since the official repository is a subversion repository even though most of the QtWebKit developers uses the Git trees since we can actively work on QtWebKit without ever having to "git push" or "svn commit" because we have tons of automated review and commit tools :-)

Please have a look at http://trac.webkit.org/wiki/QtWebKitContrib and http://www.webkit.org/coding/contributing.html and upload a version of your patch that contains a proper ChangeLog entry :-)

> Maybe you should create a qtwebkit-patches mailing list for a quick and comfortable sending patches via 'git send-email' ...

Wont work since all patches must be reviewed of an approved WebKit reviewer and the reviewer tool is built into our Bugzilla setup :-)

[Johannes Obermayr]

Sorry for wasting your time ...

[Alexander Færøy]

(In reply to comment #2)
> Sorry for wasting your time ...

You are not wasting our time. The patch looks clean enough for me, but I am not a reviewer. If you resubmit it with a proper ChangeLog entry, I will CC some reviewers who can review your changes :-)

[Johannes Obermayr]

With such stupid rules I regret of spending a few hours to detect the issue :-(

Fix it yourself ...

[Noam Rosenthal]

Comment on attachment 122267 [details]
Patch

Patch does not include changelog.

[Simon Hausmann]

(In reply to comment #4)
> With such stupid rules I regret of spending a few hours to detect the issue :-(

I'm sorry Johannes that you feel that way, but this is just like in KDE or any other open source project: There are rules about contributing source code.

In KDE you have to understand SVN, write a descriptive commit message and adhere to BUG:12344 etc. standards for the bugzilla integration. Then you need to get yourself access to the KDE SVN repository or find a kind soul to review & commit the patch for you.

In the Git project you need to use git send-email to send your patch to the git list, CC Junio and make sure your commit has the right Signed-off-by.

In Qt you have to upload your patch to Gerrit and it needs to have a unique Change-Id, generated by a git commit hook.

And lastly in WebKit you upload your patch to Bugzilla together with a ChangeLog entry. The documentation at http://www.webkit.org/coding/contributing.html explains the necessary steps and the tools available make it easy to "play by the rules". But as you can see, every project has such rules.

If you do not want to play by those rules but still get your change contributed, then you need to approach a fellow developer with kind words asking him to do the "paperwork" on your behalf. In WebKit's case however you still need to do the initial patch upload to accept the contribution licensing agreement, but it is possible to get help for getting your patch reviewed and landed manually.

I'm re-opening this bug because the issue you discovered still appears to be valid, regardless of your feelings towards the patch contribution process of WebKit.

[Johannes Obermayr]

Closing because nobody is interested in pushing it within 5 months.

See a good workflow:
1. http://lists.x.org/archives/xorg-devel/2012-May/031381.html
2. http://cgit.freedesktop.org/xorg/driver/xf86-video-intel/commit/?id=7fdd8bd6d2ba7bf113d4109e6bb6f750f909565f
3. http://lists.x.org/archives/xorg-devel/2012-May/031385.html

[Zoltan Arvai]

Created attachment 144821 [details]
new patch

Patch with changelog.

[Csaba Osztrogonác]

Reopen to let EWS bots test the patch

[WebKit Review Bot]

Comment on attachment 144821 [details]
new patch

Clearing flags on attachment: 144821

Committed r119315: <http://trac.webkit.org/changeset/119315>

[WebKit Review Bot]

All reviewed patches have been landed.  Closing bug.