Bug 78611 - [Qt] Eliminate first set of QtWidgets dependencies from WebCore
Summary: [Qt] Eliminate first set of QtWidgets dependencies from WebCore
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Simon Hausmann
URL:
Keywords:
Depends on:
Blocks: 78109
  Show dependency treegraph
 
Reported: 2012-02-14 09:42 PST by Simon Hausmann
Modified: 2012-02-16 02:07 PST (History)
6 users (show)

See Also:


Attachments
Patch (30.92 KB, patch)
2012-02-14 10:02 PST, Simon Hausmann
no flags Details | Formatted Diff | Diff
Patch (32.10 KB, patch)
2012-02-15 03:45 PST, Simon Hausmann
kenneth: review+
webkit-ews: commit-queue-
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Simon Hausmann 2012-02-14 09:42:07 PST
[Qt] Eliminate first set of QtWidgets dependencies from WebCore
Comment 1 Simon Hausmann 2012-02-14 10:02:22 PST
Created attachment 126993 [details]
Patch
Comment 2 Early Warning System Bot 2012-02-14 10:14:59 PST
Comment on attachment 126993 [details]
Patch

Attachment 126993 [details] did not pass qt-ews (qt):
Output: http://queues.webkit.org/results/11522308
Comment 3 WebKit Review Bot 2012-02-15 01:13:57 PST
Attachment 126993 [details] did not pass style-queue:

Failed to run "['Tools/Scripts/update-webkit']" exit_code: 9

Last 3072 characters of output:
p/tests/inspector/elements-test.js
	M	LayoutTests/inspector/styles/undo-change-property-expected.txt
	M	LayoutTests/inspector/styles/undo-add-property-expected.txt
	M	LayoutTests/inspector/styles/undo-add-new-rule-expected.txt
	M	LayoutTests/inspector/styles/undo-set-selector-text-expected.txt
	M	LayoutTests/inspector/styles/undo-change-property.html
	M	LayoutTests/inspector/styles/undo-add-property.html
	M	LayoutTests/inspector/styles/undo-add-new-rule.html
	M	LayoutTests/inspector/styles/undo-set-selector-text.html
	M	LayoutTests/inspector/styles/undo-property-toggle-expected.txt
	M	LayoutTests/inspector/styles/undo-property-toggle.html
	M	LayoutTests/inspector/elements/undo-dom-edits-2-expected.txt
	M	LayoutTests/inspector/elements/undo-dom-edits-expected.txt
	M	LayoutTests/ChangeLog
	M	Source/WebCore/ChangeLog
	M	Source/WebCore/inspector/front-end/DOMAgent.js
	M	Source/WebCore/inspector/front-end/CSSStyleModel.js
	M	Source/WebCore/inspector/front-end/ElementsPanel.js
	M	Source/WebCore/inspector/InspectorHistory.cpp
	M	Source/WebCore/inspector/InspectorDOMAgent.h
	M	Source/WebCore/inspector/Inspector.json
	M	Source/WebCore/inspector/InspectorCSSAgent.cpp
	M	Source/WebCore/inspector/DOMEditor.cpp
	M	Source/WebCore/inspector/InspectorDOMAgent.cpp
	M	Source/WebCore/inspector/InspectorHistory.h
r107792 = a9730c4e2cf51fc2723bd4e3129fbec8e7be2e6d (refs/remotes/origin/master)
First, rewinding head to replay your work on top of it...
Applying: [Mac][Win][WK2] Switch to RFC 6455 protocol for WebSockets
Using index info to reconstruct a base tree...
<stdin>:1578: trailing whitespace.
        
<stdin>:1647: trailing whitespace.
    
<stdin>:1657: trailing whitespace.
    
<stdin>:1672: trailing whitespace.
        return 0;        
<stdin>:1674: trailing whitespace.
    
warning: squelched 7 whitespace errors
warning: 12 lines add whitespace errors.
Falling back to patching base and 3-way merge...
warning: too many files (created: 168753 deleted: 3), skipping inexact rename detection
Auto-merging LayoutTests/ChangeLog
CONFLICT (content): Merge conflict in LayoutTests/ChangeLog
Auto-merging LayoutTests/platform/wk2/Skipped
Auto-merging Source/WebCore/ChangeLog
Auto-merging Source/WebCore/css/CSSCalculationValue.cpp
Auto-merging Source/WebCore/css/CSSCalculationValue.h
Auto-merging Source/WebCore/css/CSSParser.cpp
Auto-merging Source/WebKit/mac/ChangeLog
CONFLICT (content): Merge conflict in Source/WebKit/mac/ChangeLog
Auto-merging Source/WebKit2/ChangeLog
CONFLICT (content): Merge conflict in Source/WebKit2/ChangeLog
Auto-merging Tools/ChangeLog
CONFLICT (content): Merge conflict in Tools/ChangeLog
Failed to merge in the changes.
Patch failed at 0001 [Mac][Win][WK2] Switch to RFC 6455 protocol for WebSockets

When you have resolved this problem run "git rebase --continue".
If you would prefer to skip this patch, instead run "git rebase --skip".
To restore the original branch and stop rebasing run "git rebase --abort".

rebase refs/remotes/origin/master: command returned error: 1

Died at Tools/Scripts/update-webkit line 164.


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 4 Simon Hausmann 2012-02-15 03:45:02 PST
Created attachment 127153 [details]
Patch
Comment 5 Kenneth Rohde Christiansen 2012-02-15 03:55:39 PST
Comment on attachment 127153 [details]
Patch

r=me, good work. Nice descriptions :-)
Comment 6 Early Warning System Bot 2012-02-15 05:54:23 PST
Comment on attachment 127153 [details]
Patch

Attachment 127153 [details] did not pass qt-ews (qt):
Output: http://queues.webkit.org/results/11515745
Comment 7 Simon Hausmann 2012-02-15 07:24:38 PST
Committed r107811: <http://trac.webkit.org/changeset/107811>
Comment 8 Csaba Osztrogonác 2012-02-15 07:39:25 PST
(In reply to comment #7)
> Committed r107811: <http://trac.webkit.org/changeset/107811>

It broke 4 tests on Qt 4.8 (on Qt5 plugin tests are skipped because of https://bugs.webkit.org/show_bug.cgi?id=70023)

Could you check if these new results are correct or not?
Comment 9 Csaba Osztrogonác 2012-02-15 07:39:58 PST
Results can be found here:
http://build.webkit.sed.hu/results/x86-64%20Linux%20Qt%20Release/r107811%20%2833778%29/results.html
Comment 11 Csaba Osztrogonác 2012-02-15 23:15:32 PST
(In reply to comment #9)
> Results can be found here:
> http://build.webkit.sed.hu/results/x86-64%20Linux%20Qt%20Release/r107811%20%2833778%29/results.html

I updated the expected files - http://trac.webkit.org/changeset/107892
I assumed the new ones are correct. Please validate them.

And otherwise Mac build is still broken. Are you going to fix it?
Comment 12 Simon Hausmann 2012-02-16 02:07:26 PST
(In reply to comment #11)
> (In reply to comment #9)
> > Results can be found here:
> > http://build.webkit.sed.hu/results/x86-64%20Linux%20Qt%20Release/r107811%20%2833778%29/results.html
> 
> I updated the expected files - http://trac.webkit.org/changeset/107892
> I assumed the new ones are correct. Please validate them.

Argh, yes, I forgot to update them. Thanks! The new ones you submitted are correct.

> And otherwise Mac build is still broken. Are you going to fix it?

Prospective build fix landed in r107913. I'll watch the bot.