WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
Bug 28404
[Qt] EditorClientQt::shouldShowDeleteInterface uses non-existent methods in HTMLElement
https://bugs.webkit.org/show_bug.cgi?id=28404
Summary
[Qt] EditorClientQt::shouldShowDeleteInterface uses non-existent methods in H...
George Wright
Reported
2009-08-17 12:06:48 PDT
Revision 47367 removes HTMLElement::className() which is used in the shouldShowDeleteInterface() method in EditorClientQt.cpp, causing a build failure.
Attachments
Return false in shouldShowDeleteInterface
(1.13 KB, patch)
2009-08-17 12:10 PDT
,
George Wright
eric
: review+
commit-queue
: commit-queue-
Details
Formatted Diff
Diff
View All
Add attachment
proposed patch, testcase, etc.
George Wright
Comment 1
2009-08-17 12:10:36 PDT
Created
attachment 34984
[details]
Return false in shouldShowDeleteInterface Every other port simply returns false for this function, and as the method it used to call in HTMLElement has been removed, this update makes the Qt port also return false and allows it to build again.
Eric Seidel (no email)
Comment 2
2009-08-17 17:13:02 PDT
Comment on
attachment 34984
[details]
Return false in shouldShowDeleteInterface OK. I would expect the WebKit level is expected to implement this.
Eric Seidel (no email)
Comment 3
2009-08-17 17:42:10 PDT
Comment on
attachment 34984
[details]
Return false in shouldShowDeleteInterface Rejecting patch 34984 from commit-queue. This patch will require manual commit. Patch
https://bugs.webkit.org/attachment.cgi?id=34984
from
bug 28404
failed to download and apply.
Eric Seidel (no email)
Comment 4
2009-08-17 17:46:15 PDT
Was this already landed? Hunk #1 succeeded at 1 with fuzz 3. patching file WebKit/qt/WebCoreSupport/EditorClientQt.cpp Reversed (or previously applied) patch detected! Assume -R? [n] Apply anyway? [n] Skipping patch. 1 out of 1 hunk ignored -- saving rejects to file WebKit/qt/WebCoreSupport/EditorClientQt.cpp.rej patch -p0 "WebKit/qt/WebCoreSupport/EditorClientQt.cpp" returned 1. Pass --force to ignore patch failures.
Adam Barth
Comment 5
2009-08-18 20:05:36 PDT
Please re-open if this hasn't already been landed.
George Wright
Comment 6
2010-05-17 08:35:19 PDT
This patch was never landed, and the fix hasn't been applied. I can't seem to mark this bug as reopened?
Eric Seidel (no email)
Comment 7
2010-05-17 11:04:58 PDT
reopening so the commit-bot will see it.
Eric Seidel (no email)
Comment 8
2010-05-17 11:05:24 PDT
Except... adding it back to the commit-queue w/o modification as you have done, will just cause it to fail again, I suspect.
WebKit Commit Bot
Comment 9
2010-05-19 04:29:38 PDT
Comment on
attachment 34984
[details]
Return false in shouldShowDeleteInterface Rejecting patch 34984 from commit-queue. Failed to run "[u'/Users/eseidel/Projects/CommitQueue/WebKitTools/Scripts/svn-apply', u'--reviewer', u'Eric Seidel', u'--force']" exit_code: 1 Parsed 2 diffs from patch file(s). patching file WebKit/qt/ChangeLog Hunk #1 succeeded at 1 with fuzz 3. patching file WebKit/qt/WebCoreSupport/EditorClientQt.cpp Hunk #1 FAILED at 107. 1 out of 1 hunk FAILED -- saving rejects to file WebKit/qt/WebCoreSupport/EditorClientQt.cpp.rej Full output:
http://webkit-commit-queue.appspot.com/results/2303319
George Wright
Comment 10
2010-05-19 06:16:56 PDT
Oh, I just noticed that a different fix for this bug was landed (a cursory glance at the source revealed that it looked the same, but it's actually been changed). I will re-close it.
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug