Bug 37867 - [Qt] Patches to upstream from Qt for 2.0
Summary: [Qt] Patches to upstream from Qt for 2.0
Status: CLOSED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P1 Normal
Assignee: Nobody
URL:
Keywords: Qt
Depends on:
Blocks: 35784
  Show dependency treegraph
 
Reported: 2010-04-20 10:19 PDT by Jocelyn Turcotte
Modified: 2010-04-28 01:13 PDT (History)
7 users (show)

See Also:


Attachments
Build fix when !ENABLED(NETSCAPE_PLUGIN_API) (1.96 KB, patch)
2010-04-20 10:20 PDT, Jocelyn Turcotte
no flags Details | Formatted Diff | Diff
Build fix on Mac (3.16 KB, patch)
2010-04-20 10:21 PDT, Jocelyn Turcotte
no flags Details | Formatted Diff | Diff
Workaround syncqt dependency mechanism (1.33 KB, patch)
2010-04-20 10:23 PDT, Jocelyn Turcotte
no flags Details | Formatted Diff | Diff
Corrects JSC symbols visibility (1.20 KB, patch)
2010-04-20 10:24 PDT, Jocelyn Turcotte
no flags Details | Formatted Diff | Diff
Corrects autotests resources compilation in QTDIR_build (6.28 KB, patch)
2010-04-20 10:24 PDT, Jocelyn Turcotte
no flags Details | Formatted Diff | Diff
Build fix when !ENABLED(NETSCAPE_PLUGIN_API) v2 (2.01 KB, patch)
2010-04-20 11:19 PDT, Jocelyn Turcotte
no flags Details | Formatted Diff | Diff
Build fix on Mac v2 (3.24 KB, patch)
2010-04-20 11:20 PDT, Jocelyn Turcotte
vestbo: review-
vestbo: commit-queue-
Details | Formatted Diff | Diff
Corrects JSC symbols visibility v2 (1.33 KB, patch)
2010-04-20 11:22 PDT, Jocelyn Turcotte
no flags Details | Formatted Diff | Diff
Build fix on Mac v3 (3.20 KB, patch)
2010-04-21 01:20 PDT, Jocelyn Turcotte
no flags Details | Formatted Diff | Diff
Remove dep of JSC to QtGui (1.58 KB, patch)
2010-04-21 06:11 PDT, Jocelyn Turcotte
no flags Details | Formatted Diff | Diff
Remove autotest tr strings (2.04 KB, patch)
2010-04-21 06:12 PDT, Jocelyn Turcotte
no flags Details | Formatted Diff | Diff
MSVC 64 build fix (see comment) (2.73 KB, patch)
2010-04-21 06:15 PDT, Jocelyn Turcotte
kenneth: review-
kenneth: commit-queue-
Details | Formatted Diff | Diff
Disable Cpp0x (see comment) (2.22 KB, patch)
2010-04-21 06:19 PDT, Jocelyn Turcotte
no flags Details | Formatted Diff | Diff
MSVC 2010 build fix (2.64 KB, patch)
2010-04-21 06:19 PDT, Jocelyn Turcotte
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Jocelyn Turcotte 2010-04-20 10:19:31 PDT
Some patches that were committed directly in Qt that have to be upstreamed to ease next imports.
Comment 1 Jocelyn Turcotte 2010-04-20 10:20:27 PDT
Created attachment 53838 [details]
Build fix when !ENABLED(NETSCAPE_PLUGIN_API)
Comment 2 Jocelyn Turcotte 2010-04-20 10:21:03 PDT
Created attachment 53839 [details]
Build fix on Mac
Comment 3 Jocelyn Turcotte 2010-04-20 10:23:29 PDT
Created attachment 53840 [details]
Workaround syncqt dependency mechanism
Comment 4 Jocelyn Turcotte 2010-04-20 10:24:01 PDT
Created attachment 53841 [details]
Corrects JSC symbols visibility
Comment 5 Jocelyn Turcotte 2010-04-20 10:24:45 PDT
Created attachment 53842 [details]
Corrects autotests resources compilation in QTDIR_build
Comment 6 Kenneth Rohde Christiansen 2010-04-20 10:30:07 PDT
Comment on attachment 53838 [details]
Build fix when !ENABLED(NETSCAPE_PLUGIN_API)

The Changelog should have a reference to the bug report and it should not have [Qt] prefixed as this is not Qt specific. Please fix that and re-upload a new patch.
Comment 7 Kenneth Rohde Christiansen 2010-04-20 10:31:16 PDT
Comment on attachment 53839 [details]
Build fix on Mac


> +        - Make sure JavaScriptCore's debug_and_release config follows WebCore's

...follow that of WebCore.

I would like either Laszlo or Simon to take a look at this one.
Comment 8 Kenneth Rohde Christiansen 2010-04-20 10:33:11 PDT
Comment on attachment 53840 [details]
Workaround syncqt dependency mechanism


>  
> +        [Qt] Hide the OpenGL and QtMultimedia dependencies from syncqt.

What is syncqt? maybe the script used for synching... Is that script in WebKit?


> +            tobe|!tobe: QT += multimedia

I don't understand this? tobe or not tobe?
Comment 9 Kenneth Rohde Christiansen 2010-04-20 10:34:50 PDT
Comment on attachment 53841 [details]
Corrects JSC symbols visibility


> +# Add these two lines both in QTDIR_build and out because we don't include qbase.pri

... which does that?

and out? what do you mean?

> +contains(QT_CONFIG, reduce_exports):CONFIG += hide_symbols
> +unix:contains(QT_CONFIG, reduce_relocations):CONFIG += bsymbolic_functions
> +
>  CONFIG(QTDIR_build) {
>      # Make sure we compile both debug and release on mac when inside Qt.
>      # This line was extracted from qbase.pri instead of including the whole file
Comment 10 Jocelyn Turcotte 2010-04-20 11:19:47 PDT
Created attachment 53852 [details]
Build fix when !ENABLED(NETSCAPE_PLUGIN_API) v2
Comment 11 Jocelyn Turcotte 2010-04-20 11:20:19 PDT
Created attachment 53853 [details]
Build fix on Mac v2
Comment 12 Jocelyn Turcotte 2010-04-20 11:22:50 PDT
Created attachment 53854 [details]
Corrects JSC symbols visibility v2
Comment 13 WebKit Review Bot 2010-04-20 11:23:50 PDT
Attachment 53852 [details] did not pass style-queue:

Failed to run "['WebKitTools/Scripts/check-webkit-style', '--no-squash']" exit_code: 2
Last 3072 characters of output:
  any such category.  The filter input "-whitespace,+whitespace/braces" fails
  the category "whitespace/tab" and passes "whitespace/braces".

  Examples: --filter=-whitespace,+whitespace/braces
            --filter=-whitespace,-runtime/printf,+runtime/printf_format
            --filter=-,+build/include_what_you_use

Paths:
  Certain style-checking behavior depends on the paths relative to
  the WebKit source root of the files being checked.  For example,
  certain types of errors may be handled differently for files in
  WebKit/gtk/webkit/ (e.g. by suppressing "readability/naming" errors
  for files in this directory).

  Consequently, if the path relative to the source root cannot be
  determined for a file being checked, then style checking may not
  work correctly for that file.  This can occur, for example, if no
  WebKit checkout can be found, or if the source root can be detected,
  but one of the files being checked lies outside the source tree.

  If a WebKit checkout can be detected and all files being checked
  are in the source tree, then all paths will automatically be
  converted to paths relative to the source root prior to checking.
  This is also useful for display purposes.

  Currently, this command can detect the source root only if the
  command is run from within a WebKit checkout (i.e. if the current
  working directory is below the root of a checkout).  In particular,
  it is not recommended to run this script from a directory outside
  a checkout.

  Running this script from a top-level WebKit source directory and
  checking only files in the source tree will ensure that all style
  checking behaves correctly -- whether or not a checkout can be
  detected.  This is because all file paths will already be relative
  to the source root and so will not need to be converted.

Options:
  -h, --help            show this help message and exit
  -f RULES, --filter-rules=RULES
                        set a filter to control what categories of style
                        errors to report.  Specify a filter using a comma-
                        delimited list of boolean filter rules, for example "
                        --filter -whitespace,+whitespace/braces".  To display
                        all categories and which are enabled by default, pass
                        no value (e.g. '-f ""' or '--filter=').
  -g COMMIT, --git-commit=COMMIT, --git-diff=COMMIT, --git-since=COMMIT
                        check all changes after the given git commit.
  -m INT, --min-confidence=INT
                        set the minimum confidence of style errors to report.
                        Can be an integer 1-5, with 1 displaying all errors.
                        Defaults to 1.
  -o FORMAT, --output-format=FORMAT
                        set the output format, which can be "emacs" or "vs7"
                        (for Visual Studio).  Defaults to "emacs".
  -v, --verbose         enable verbose logging.

This script can miss errors and does not substitute for code review.

ERROR: no such option: --no-squash


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 14 Tor Arne Vestbø 2010-04-20 12:12:17 PDT
Comment on attachment 53853 [details]
Build fix on Mac v2


> +# Output in JavaScriptCore/<config>
> +CONFIG(debug, debug|release): JAVASCRIPTCORE_DESTDIR = debug
> +CONFIG(release, debug|release): JAVASCRIPTCORE_DESTDIR = release
> +# Use different targets to prevent parallel builds file clashes on Mac
> +CONFIG(debug, debug|release): JAVASCRIPTCORE_TARGET = jscored
> +CONFIG(release, debug|release): JAVASCRIPTCORE_TARGET = jscore

Nit, please combine these into scoped blocks, let's try to avoid repeated CONFIG(...) all over the place
Comment 15 Jocelyn Turcotte 2010-04-21 01:20:57 PDT
Created attachment 53929 [details]
Build fix on Mac v3

Here is your patch
Comment 16 Jocelyn Turcotte 2010-04-21 06:11:26 PDT
Created attachment 53946 [details]
Remove dep of JSC to QtGui
Comment 17 Jocelyn Turcotte 2010-04-21 06:12:20 PDT
Created attachment 53947 [details]
Remove autotest tr strings
Comment 18 Jocelyn Turcotte 2010-04-21 06:15:27 PDT
Created attachment 53948 [details]
MSVC 64 build fix (see comment)

Part of this patch will break the patch in bug #35607
I don't know what we do in that case, up to you.
Comment 19 Jocelyn Turcotte 2010-04-21 06:19:01 PDT
Created attachment 53949 [details]
Disable Cpp0x (see comment)

This patch is currently useful only if the developer modified its mkspec to build Qt using Cpp0x.

I don't know if this should go in trunk, maybe it should go in the staging area instead until there is actually a more official way to enable Cpp0x in qt (e.g. an option in configure).
What do you think?
Comment 20 Jocelyn Turcotte 2010-04-21 06:19:46 PDT
Created attachment 53950 [details]
MSVC 2010 build fix
Comment 21 Kenneth Rohde Christiansen 2010-04-21 07:08:55 PDT
Comment on attachment 53948 [details]
MSVC 64 build fix (see comment)


> +        [Qt] Fix the MSVC 64bit build.

The change is not in Qt-only code, so you need to remove the [Qt]. I would prefer you adding at least the not Qt specific patches in separate bug reports.
Comment 22 Eric Seidel (no email) 2010-04-21 18:11:23 PDT
Comment on attachment 53838 [details]
Build fix when !ENABLED(NETSCAPE_PLUGIN_API)

Cleared Kenneth Rohde Christiansen's review+ from obsolete attachment 53838 [details] so that this bug does not appear in http://webkit.org/pending-commit.
Comment 23 Jocelyn Turcotte 2010-04-22 01:21:16 PDT
Comment on attachment 53948 [details]
MSVC 64 build fix (see comment)

Patch has been moved to bug #37980
Comment 24 WebKit Commit Bot 2010-04-22 06:34:46 PDT
Comment on attachment 53840 [details]
Workaround syncqt dependency mechanism

Clearing flags on attachment: 53840

Committed r58092: <http://trac.webkit.org/changeset/58092>
Comment 25 WebKit Commit Bot 2010-04-22 07:56:19 PDT
Comment on attachment 53842 [details]
Corrects autotests resources compilation in QTDIR_build

Clearing flags on attachment: 53842

Committed r58096: <http://trac.webkit.org/changeset/58096>
Comment 26 WebKit Commit Bot 2010-04-22 09:19:30 PDT
Comment on attachment 53852 [details]
Build fix when !ENABLED(NETSCAPE_PLUGIN_API) v2

Rejecting patch 53852 from commit-queue.

Failed to run "[u'/Users/eseidel/Projects/CommitQueue/WebKitTools/Scripts/svn-apply', u'--reviewer', u'Simon Hausmann', u'--force']" exit_code: 1
patching file WebCore/ChangeLog
Hunk #1 succeeded at 1 with fuzz 3.
patching file WebCore/plugins/PluginView.cpp
Hunk #1 succeeded at 1297 with fuzz 2 (offset -1 lines).
Hunk #2 succeeded at 1308 (offset -1 lines).
Hunk #3 succeeded at 1318 with fuzz 1 (offset -1 lines).
patching file WebCore/plugins/PluginViewNone.cpp
Hunk #1 FAILED at 126.
1 out of 1 hunk FAILED -- saving rejects to file WebCore/plugins/PluginViewNone.cpp.rej

Full output: http://webkit-commit-queue.appspot.com/results/1863029
Comment 27 WebKit Commit Bot 2010-04-22 09:38:23 PDT
Comment on attachment 53854 [details]
Corrects JSC symbols visibility v2

Clearing flags on attachment: 53854

Committed r58100: <http://trac.webkit.org/changeset/58100>
Comment 28 WebKit Commit Bot 2010-04-22 10:13:40 PDT
Comment on attachment 53946 [details]
Remove dep of JSC to QtGui

Clearing flags on attachment: 53946

Committed r58104: <http://trac.webkit.org/changeset/58104>
Comment 29 WebKit Commit Bot 2010-04-22 10:33:05 PDT
Comment on attachment 53947 [details]
Remove autotest tr strings

Clearing flags on attachment: 53947

Committed r58105: <http://trac.webkit.org/changeset/58105>
Comment 30 Laszlo Gombos 2010-04-22 18:48:15 PDT
Comment on attachment 53852 [details]
Build fix when !ENABLED(NETSCAPE_PLUGIN_API) v2

Marking patch obsolete as this issue a DUP of bug 36621 and patch failed to apply as fix for bug 36621 is already landed.

Next time I think it would be better to open a separate bug for separate issues.
Comment 31 Laszlo Gombos 2010-04-22 20:40:10 PDT
Comment on attachment 53950 [details]
MSVC 2010 build fix

JavaScriptCore/runtime/Structure.cpp:172
 +          else
Can we use c++ cast here instead of the C style cast ? 

JavaScriptCore/runtime/Structure.cpp:179
 +      }
Same here.
Comment 32 WebKit Commit Bot 2010-04-26 03:48:41 PDT
Comment on attachment 53929 [details]
Build fix on Mac v3

Rejecting patch 53929 from commit-queue.

Failed to run "[u'/Users/eseidel/Projects/CommitQueue/WebKitTools/Scripts/svn-apply', u'--reviewer', u'Tor Arne Vestb\xf8', u'--force']" exit_code: 1
patching file JavaScriptCore/ChangeLog
Hunk #1 succeeded at 1 with fuzz 3.
patching file JavaScriptCore/JavaScriptCore.pri
Hunk #2 succeeded at 72 (offset 2 lines).
patching file JavaScriptCore/JavaScriptCore.pro
Hunk #1 FAILED at 8.
1 out of 1 hunk FAILED -- saving rejects to file JavaScriptCore/JavaScriptCore.pro.rej

Full output: http://webkit-commit-queue.appspot.com/results/1796120
Comment 33 Jocelyn Turcotte 2010-04-27 07:23:00 PDT
Comment on attachment 53929 [details]
Build fix on Mac v3

Clearing flags on attachment: 53929

Committed r58306: <http://trac.webkit.org/changeset/58306>
Comment 34 Jocelyn Turcotte 2010-04-27 07:23:34 PDT
Comment on attachment 53949 [details]
Disable Cpp0x (see comment)

Clearing flags on attachment: 53949

Committed r58305: <http://trac.webkit.org/changeset/58305>
Comment 35 Jocelyn Turcotte 2010-04-27 07:24:19 PDT
Comment on attachment 53950 [details]
MSVC 2010 build fix

Clearing flags on attachment: 53950

Committed r58308: <http://trac.webkit.org/changeset/58308>
Comment 36 Jocelyn Turcotte 2010-04-27 07:26:13 PDT
Resolving bug after all patches have been landed

(In reply to comment #30)
> Next time I think it would be better to open a separate bug for separate
> issues.

Yep agreed, this bug became a bigger mess than I thought.
Comment 37 WebKit Review Bot 2010-04-27 07:30:57 PDT
http://trac.webkit.org/changeset/58305 might have broken Qt Linux ARMv5 Release
The following changes are on the blame list:
http://trac.webkit.org/changeset/58305
http://trac.webkit.org/changeset/58306
http://trac.webkit.org/changeset/58307
http://trac.webkit.org/changeset/58308
Comment 38 WebKit Review Bot 2010-04-27 07:31:06 PDT
http://trac.webkit.org/changeset/58306 might have broken Qt Linux ARMv5 Release
The following changes are on the blame list:
http://trac.webkit.org/changeset/58305
http://trac.webkit.org/changeset/58306
http://trac.webkit.org/changeset/58307
http://trac.webkit.org/changeset/58308
Comment 39 WebKit Review Bot 2010-04-27 07:31:17 PDT
http://trac.webkit.org/changeset/58308 might have broken Qt Linux ARMv5 Release
The following changes are on the blame list:
http://trac.webkit.org/changeset/58305
http://trac.webkit.org/changeset/58306
http://trac.webkit.org/changeset/58307
http://trac.webkit.org/changeset/58308
Comment 40 Simon Hausmann 2010-04-27 23:36:22 PDT
<cherry-pick-for-backport: r58307>
Comment 41 Simon Hausmann 2010-04-28 01:12:51 PDT
Revision r58092 cherry-picked into qtwebkit-2.0 with commit d5e8869917b882c4d2a6e449682bd12d3ca95e65
Revision r58096 cherry-picked into qtwebkit-2.0 with commit e765f06a9508bea1d49a883d811ddaca7c92b50a
Revision r58100 cherry-picked into qtwebkit-2.0 with commit f07a3c62f795c418e6656ef664b14ed50282aea0
Revision r58104 cherry-picked into qtwebkit-2.0 with commit 11d902dbdede03de1637ead81fac9bd3a69ebc3b
Revision r58105 cherry-picked into qtwebkit-2.0 with commit e4ff26f12a8ce50d12662487f2a1d8c1820eabb2
Revision r58305 cherry-picked into qtwebkit-2.0 with commit 70fd060abd5646593a64d8f17518f67fc3a058d6
Revision r58306 cherry-picked into qtwebkit-2.0 with commit ec1dbe948b373b65ea3727518da577a089158a46
Revision r58307 cherry-picked into qtwebkit-2.0 with commit 9f1b689350cc6aa46978a3c1eedfdda7199c2072
Revision r58308 cherry-picked into qtwebkit-2.0 with commit b3af4e186eda0106db92c90cc2adaa7f8d26f169