Bug 32481 – [WML] Compilation error in wml enabled builds.

• Status: RESOLVED
• Resolution: FIXED
• Component: WebCore Misc.

[Nayan]

WML enabled builds are now failing with following error,

DerivedSources/WMLElementFactory.cpp: In function ‘WTF::PassRefPtr<WebCore::WMLElement> WebCore::inputConstructor(const WebCore::QualifiedName&, WebCore::Document*, bool)’:
DerivedSources/WMLElementFactory.cpp:124: error: cannot allocate an object of abstract type ‘WebCore::WMLInputElement’
../../WebCore/wml/WMLInputElement.h:32: note:   because the following virtual functions are pure within ‘WebCore::WMLInputElement’:
../../WebCore/dom/InputElement.h:51: note: 	virtual void WebCore::InputElement::setValue(const WebCore::String&, bool)
../../WebCore/dom/InputElement.h:52: note: 	virtual void WebCore::InputElement::setValueForUser(const WebCore::String&)
make[1]: *** [DerivedSources/libWebCore_la-WMLElementFactory.lo] Error 1
make[1]: Leaving directory `/home/nayankk/Browsers/webkit/WebKitBuild/Release'
make: *** [all] Error 2

Failed to build WebKit using 'make'!


Version of WebKit used is,

nayankk@dreams:~/Browsers/webkit$ svn info
Path: .
URL: http://svn.webkit.org/repository/webkit/trunk
Repository Root: http://svn.webkit.org/repository/webkit
Repository UUID: 268f45cc-cd09-0410-ab3c-d52691b4dbfc
Revision: 52059
Node Kind: directory
Schedule: normal
Last Changed Author: [e-mail]
Last Changed Rev: 52059
Last Changed Date: 2009-12-13 08:58:42 +0530 (Sun, 13 Dec 2009)

[Nayan]

This error is due to the changes committed with http://trac.webkit.org/changeset/51602. With this changeset, signature of 'WebCore::InputElement::setValue' is changed and 'WebCore::InputElement::setValueForUser' pure virtual function is added to 'InputElement' class.

'InputElement' class is the parent class for 'HTMLInputElement' and 'WMLInputElement' class. With 51602 changeset, changes in HTMLInputElement were done, but WMLInputElement wasn't changed to define these virtual functions and hence resulting in compilation errors.

[Nayan]

Created attachment 44751 [details]
Proposed patch

Proposed patch to resolve compilation error is attached for review. This simply changes the signature of 'setValue' function and defines 'setValueForUser' function in WMLInputElement class.

[WebKit Review Bot]

style-queue ran check-webkit-style on attachment 44751 [details] without any errors.

[Eric Seidel (no email)]

Comment on attachment 44751 [details]
Proposed patch

Should we just change setValueForUser to have a default implementation instead of being pure virtual?  Is it really only useful to HTML?  Otherwise this looks fine.

[Nayan]

(In reply to comment #4)
> (From update of attachment 44751 [details])
> Should we just change setValueForUser to have a default implementation instead
> of being pure virtual?  Is it really only useful to HTML?  Otherwise this looks
> fine.

Thanks Eric for the review comments. Looks like, changes in changeset 51602 were done to detect a login form AutoFill from JavaScript. Since WML with Javascript is a rare scenario, I doubt these changes are required in WML. I guess Darin/Timothy will have some inputs here.

[WebKit Commit Bot]

Comment on attachment 44751 [details]
Proposed patch

Clearing flags on attachment: 44751

Committed r52107: <http://trac.webkit.org/changeset/52107>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.