Bug 140611 - REGRESSION(r178486): It broke the !ENABLE(VIDEO) build
Summary: REGRESSION(r178486): It broke the !ENABLE(VIDEO) build
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P1 Critical
Assignee: Csaba Osztrogonác
URL:
Keywords:
Depends on:
Blocks: 140468
  Show dependency treegraph
 
Reported: 2015-01-19 01:52 PST by Csaba Osztrogonác
Modified: 2015-01-19 07:27 PST (History)
4 users (show)

See Also:


Attachments
Patch (2.71 KB, patch)
2015-01-19 01:55 PST, Csaba Osztrogonác
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Csaba Osztrogonác 2015-01-19 01:52:51 PST
build log:

../../Source/WebCore/testing/Internals.cpp: In member function 'bool WebCore::Internals::elementIsBlockingDisplaySleep(WebCore::Element*) const':
../../Source/WebCore/testing/Internals.cpp:2441:39: error: invalid use of incomplete type 'class WebCore::HTMLMediaElement'
In file included from ../../Source/WebCore/dom/Element.h:29:0,
                 from ../../Source/WebCore/loader/cache/CachedResourceRequest.h:29,
                 from ../../Source/WebCore/loader/cache/CachedResourceLoader.h:32,
                 from ../../Source/WebCore/testing/Internals.cpp:35:
../../Source/WebCore/dom/Document.h:112:7: error: forward declaration of 'class WebCore::HTMLMediaElement'
In file included from ../../Source/WebCore/dom/Node.h:39:0,
                 from ../../Source/WebCore/dom/ContainerNode.h:28,
                 from ../../Source/WebCore/dom/Position.h:29,
                 from ../../Source/WebCore/editing/VisiblePosition.h:30,
                 from ../../Source/WebCore/accessibility/AccessibilityObject.h:37,
                 from ../../Source/WebCore/accessibility/AXObjectCache.h:29,
                 from ../../Source/WebCore/testing/Internals.cpp:30:
../../Source/WTF/wtf/TypeCasts.h: In instantiation of 'typename std::conditional<std::is_const<Source>::value, const Target*, Target*>::type WTF::downcast(Source*) [with Target = WebCore::HTMLMediaElement; Source = WebCore::Element; typename std::conditional<std::is_const<Source>::value, const Target*, Target*>::type = WebCore::HTMLMediaElement*]':
../../Source/WebCore/testing/Internals.cpp:2440:72:   required from here
../../Source/WTF/wtf/TypeCasts.h:81:117: error: invalid static_cast from type 'WebCore::Element*' to type 'std::conditional<false, const WebCore::HTMLMediaElement*, WebCore::HTMLMediaElement*>::type {aka WebCore::HTMLMediaElement*}'
../../Source/WebCore/testing/Internals.cpp:2442:1: error: control reaches end of non-void function [-Werror=return-type]
In file included from ../../Source/WebCore/dom/Node.h:39:0,
                 from ../../Source/WebCore/dom/ContainerNode.h:28,
                 from ../../Source/WebCore/dom/Position.h:29,
                 from ../../Source/WebCore/editing/VisiblePosition.h:30,
                 from ../../Source/WebCore/accessibility/AccessibilityObject.h:37,
                 from ../../Source/WebCore/accessibility/AXObjectCache.h:29,
                 from ../../Source/WebCore/testing/Internals.cpp:30:
../../Source/WTF/wtf/TypeCasts.h: In function 'typename std::conditional<std::is_const<Source>::value, const Target*, Target*>::type WTF::downcast(Source*) [with Target = WebCore::HTMLMediaElement; Source = WebCore::Element; typename std::conditional<std::is_const<Source>::value, const Target*, Target*>::type = WebCore::HTMLMediaElement*]':
../../Source/WTF/wtf/TypeCasts.h:82:1: error: control reaches end of non-void function [-Werror=return-type]
cc1plus: all warnings being treated as errors
Comment 1 Csaba Osztrogonác 2015-01-19 01:54:04 PST
elementIsBlockingDisplaySleep is guarded in Source/WebCore/testing/Internals.idl:
[Conditional=VIDEO] boolean elementIsBlockingDisplaySleep(Element element);

We should guard its implementation too.
Comment 2 Csaba Osztrogonác 2015-01-19 01:55:17 PST
Created attachment 244884 [details]
Patch
Comment 3 WebKit Commit Bot 2015-01-19 07:27:55 PST
Comment on attachment 244884 [details]
Patch

Clearing flags on attachment: 244884

Committed r178646: <http://trac.webkit.org/changeset/178646>
Comment 4 WebKit Commit Bot 2015-01-19 07:27:59 PST
All reviewed patches have been landed.  Closing bug.