Bug 153133

Summary: [GTK] Fix build of RenderThemeGtk without VIDEO by including HTMLInputElement
Product: WebKit Reporter: Olivier Blin <olivier.blin>
Component: WebKitGTKAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: bugs-noreply, cgarcia, commit-queue, esprehn+autocc, glenn, kondapallykalyan, mcatanzaro, zan
Priority: P2    
Version: Other   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Patch
none
Patch none

Description Olivier Blin 2016-01-15 09:43:05 PST
Gtk build fails when VIDEO is disabled:

WebKit/Source/WebCore/rendering/RenderThemeGtk.cpp: In member function ‘virtual bool WebCore::RenderThemeGtk::shouldHaveCapsLockIndicator(WebCore::HTMLInputElement&) const’:
WebKit/Source/WebCore/rendering/RenderThemeGtk.cpp:1045:19: error: invalid use of incomplete type ‘class WebCore::HTMLInputElement’
     return element.isPasswordField();
                   ^
Comment 1 Olivier Blin 2016-01-15 09:51:38 PST
Created attachment 269062 [details]
Patch
Comment 2 Michael Catanzaro 2016-01-15 11:51:40 PST
Comment on attachment 269062 [details]
Patch

Thanks. Please fix the changelog entry though; there should be the title of the bug only once, then the URL of the bug, the reviewed by line, then your description of the issue.
Comment 3 Olivier Blin 2016-01-15 14:27:23 PST
Created attachment 269098 [details]
Patch
Comment 4 Olivier Blin 2016-01-15 14:29:08 PST
Created attachment 269099 [details]
Patch
Comment 5 WebKit Commit Bot 2016-01-15 15:22:59 PST
Comment on attachment 269099 [details]
Patch

Clearing flags on attachment: 269099

Committed r195153: <http://trac.webkit.org/changeset/195153>
Comment 6 WebKit Commit Bot 2016-01-15 15:23:01 PST
All reviewed patches have been landed.  Closing bug.