Contributing to the Qt Port of WebKit

​Gitorious.org is hosting a ​public git repository of the QtWebKit port.

Coding Style

Effective immediately all commits to QtWebKit should use the ​WebKit coding style for all files in the project. That includes qwebview(.h|cpp), qwebpage(.h|.cpp) and all others. Repeat: the Qt style should not be used. Thank you.

Reporting Bugs

Bug reports, feature requests, and patches for the Qt-port of WebKit should all be submitted to the WebKit Bugzilla.

Please follow the standard ​reporting guidelines and ​steps for writing good bug reports, with the following specifics for the Qt port:

• All bugs related to the Qt port of WebKit should have the keyword 'Qt'

• Use the component to specify which area of the code you're targeting
  • There are components for rendering, plugins, tests, text, forms, etc.
  • The 'WebKit Qt' component should only be used for bugs/features in the QtWebKit API

• Add the keyword 'Performance' if the bug is performance-related

• Use the severity 'Enhancement' for feature-requests/enhancements
• Use the severity 'Blocker' for compilation errors/build breaks
• Use the severity 'Critical' for crashes
• Use the severities 'Minor, Normal, Major' for bugs

You can use ​this handy template as a starting point.

Useful filters for existing Qt bugs

​Bugs (unconfirmed or new) ​Bugs (assigned)

​API ​Compilation ​Crashes ​Enhancements ​Rendering and Layout ​Performance ​Plugins ​Tools and Tests

​Qt 4.6 Release Backlog

Checking out the sources

Developers who wish to contribute to the port can check out the sources like so:

git clone git://gitorious.org/qtwebkit/qtwebkit.git

Hacking on QtWebKit

• QtWebKit overview

Running the tests

To run the tests the normal WebKit layout test infrastructure is used. In addition you need a fixed set of fonts for testing. You can download them using Git from Gitorious:

git clone git://gitorious.org/qtwebkit/testfonts.git

Then you have to set the WEBKIT_TESTFONTS variable to point to the newly checked-out testfonts directory.

As the last step you can just run

run-webkit-tests

from the WebKitTools/Scripts directory.

Individual tests can be run just directly using for example

run-webkit-tests LayoutTests/fast/forms/plaintext-mode-1.html

For debugging at test it may also be useful to run it directly in DumpRenderTree (with gdb):

WebKitBuild/Debug/bin/DumpRenderTree -v LayoutTests/fast/forms/plaintext-mode-1.html

The expected output can usually be found in the file with the same name as the test itself but with the .expected extension.

Building API Docs

Before you can build the API docs you need make sure to have Qt build from sources and run make docs in your Qt build directory once. This will ensure that the Qt API documentation is built and can be referenced from the QtWebKit docs and it will also ensure that qdoc (the tool that generates the API docs) is built.

To build the QtWebKit API docs simply run make docs in your build directory, i.e. WebKitBuild/Release. The output will be generated in HTML in the doc/html subdirectory.

Information on Git for QtWebKit developers

• QtWebKit Git Instructions
• Using Git with WebKit
• ​A Git cheat sheet
• ​An excellent article on Git for computer scientists

The Plan

• TODO List
• Stabilization Journal

Communication

We use the regular webkit-dev mailing list to coordinate. You can also find us in the #webkit channel on irc.freenode.net. We have two mailing lists specific to the QtWebKit port

• ​Bug Mailing List
• ​Commits Mailing List

Obtaining Check-In Privileges

Contributors with a proven track record of good patch submissions and that have demonstrated an ability to work well with the community can obtain check-in privileges to the QtWebKit git repository.

The various branches of the git repository mirror the branches in the ​upstream public webkit svn repository. Patches that land on a branch in the git repository are not automatically pushed to the svn repository. Patches intended for the svn repository must first be reviewed and then committed by developers with ​sufficient privileges.

Patches that are not specific to the QtWebKit port of the webkit project should not be submitted to the git repository rather they should be contributed directly to the upstream public webkit svn repository.

To obtain check in privileges to the git repository the contributor should inquire from one of the Administrator(s) of the ​qtwebkit-developers team.