Bug 43010 - [Qt] Qt test browser: enable a QGLWidget viewport with a command line argument
Summary: [Qt] Qt test browser: enable a QGLWidget viewport with a command line argument
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC All
: P2 Normal
Assignee: Nobody
URL:
Keywords: Performance, Qt
Depends on:
Blocks:
 
Reported: 2010-07-26 15:22 PDT by Noam Rosenthal
Modified: 2010-08-22 20:50 PDT (History)
2 users (show)

See Also:


Attachments
Patch (3.32 KB, patch)
2010-07-26 15:39 PDT, Noam Rosenthal
no flags Details | Formatted Diff | Diff
Patch (3.73 KB, patch)
2010-08-04 13:19 PDT, Noam Rosenthal
no flags Details | Formatted Diff | Diff
Patch (3.28 KB, patch)
2010-08-06 09:18 PDT, Noam Rosenthal
no flags Details | Formatted Diff | Diff
Patch (3.28 KB, patch)
2010-08-06 09:18 PDT, Noam Rosenthal
ariya.hidayat: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Noam Rosenthal 2010-07-26 15:22:58 PDT
This is needed to test GL-based AC rapidly.
Comment 1 Noam Rosenthal 2010-07-26 15:39:26 PDT
Created attachment 62623 [details]
Patch
Comment 2 WebKit Commit Bot 2010-08-02 10:53:34 PDT
Comment on attachment 62623 [details]
Patch

Rejecting patch 62623 from commit-queue.

noam.rosenthal@nokia.com does not have committer permissions according to http://trac.webkit.org/browser/trunk/WebKitTools/Scripts/webkitpy/common/config/committers.py.

- If you do not have committer rights please read http://webkit.org/coding/contributing.html for instructions on how to use bugzilla flags.

- If you have committer rights please correct the error in WebKitTools/Scripts/webkitpy/common/config/committers.py by adding yourself to the file (no review needed).  Due to bug 30084 the commit-queue will require a restart after your change.  Please contact eseidel@chromium.org to request a commit-queue restart.  After restart the commit-queue will correctly respect your committer rights.
Comment 3 WebKit Commit Bot 2010-08-03 19:49:04 PDT
Comment on attachment 62623 [details]
Patch

Rejecting patch 62623 from commit-queue.

Failed to run "[u'/Users/eseidel/Projects/CommitQueue/WebKitTools/Scripts/svn-apply', u'--reviewer', u'Kenneth Rohde Christiansen', u'--force']" exit_code: 1
Parsed 2 diffs from patch file(s).
patching file WebKitTools/ChangeLog
Hunk #1 succeeded at 1 with fuzz 3.
Hunk #2 succeeded at 3787 (offset 1415 lines).
patching file WebKitTools/QtTestBrowser/main.cpp
Hunk #1 FAILED at 287.
Hunk #2 FAILED at 1116.
Hunk #3 FAILED at 1132.
Hunk #4 succeeded at 163 with fuzz 1 (offset -996 lines).
3 out of 4 hunks FAILED -- saving rejects to file WebKitTools/QtTestBrowser/main.cpp.rej

Full output: http://queues.webkit.org/results/3568843
Comment 4 Noam Rosenthal 2010-08-04 13:19:05 PDT
Created attachment 63485 [details]
Patch
Comment 5 WebKit Commit Bot 2010-08-05 14:06:18 PDT
Comment on attachment 63485 [details]
Patch

Rejecting patch 63485 from commit-queue.

Failed to run "[u'/Users/eseidel/Projects/CommitQueue/WebKitTools/Scripts/svn-apply', u'--reviewer', u'Kenneth Rohde Christiansen', u'--force']" exit_code: 1
Last 500 characters of output:
 diffs from patch file(s).
patching file WebKitTools/ChangeLog
Hunk #1 succeeded at 1 with fuzz 3.
Hunk #2 succeeded at 4109 (offset 254 lines).
patching file WebKitTools/QtTestBrowser/launcherwindow.cpp
Hunk #1 succeeded at 366 with fuzz 2 (offset -41 lines).
patching file WebKitTools/QtTestBrowser/main.cpp
Hunk #1 FAILED at 116.
Hunk #2 FAILED at 136.
Hunk #3 succeeded at 160 with fuzz 1 (offset 2 lines).
2 out of 3 hunks FAILED -- saving rejects to file WebKitTools/QtTestBrowser/main.cpp.rej

Full output: http://queues.webkit.org/results/3630488
Comment 6 Noam Rosenthal 2010-08-06 09:18:17 PDT
Created attachment 63730 [details]
Patch

This QtTestBrowser thing is a moving target... hopefully no merge conflicts this time
Comment 7 Noam Rosenthal 2010-08-06 09:18:39 PDT
Created attachment 63731 [details]
Patch

This QtTestBrowser thing is a moving target... hopefully no merge conflicts this time
Comment 8 Eric Seidel (no email) 2010-08-08 03:18:48 PDT
Comment on attachment 63485 [details]
Patch

Cleared Kenneth Rohde Christiansen's review+ from obsolete attachment 63485 [details] so that this bug does not appear in http://webkit.org/pending-commit.
Comment 9 Ariya Hidayat 2010-08-17 17:32:16 PDT
Comment on attachment 63731 [details]
Patch

LGTM. re=me
Comment 10 Andreas Kling 2010-08-22 20:50:46 PDT
Committed r65792: <http://trac.webkit.org/changeset/65792>