Changeset 237441 in webkit

Timestamp:

Oct 25, 2018, 5:39:36 PM (8 years ago)

Author:

Devin Rousso

Message:

Fix build after r237431 for platforms that don't support FULLSCREEN_API
Need the bug URL (OOPS!).

Reviewed by Joseph Pecoraro.

No new tests. No change in behavior.

• inspector/agents/InspectorDOMAgent.cpp:

(WebCore::InspectorDOMAgent::addEventListenersToNode):

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• inspector/agents/InspectorDOMAgent.cpp (modified) (2 diffs)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2018-10-25  Devin Rousso  <drousso@apple.com>
+
+        Fix build after r237431 for platforms that don't support FULLSCREEN_API
+
+        Reviewed by Joseph Pecoraro.
+
+        No new tests. No change in behavior.
+
+        * inspector/agents/InspectorDOMAgent.cpp:
+        (WebCore::InspectorDOMAgent::addEventListenersToNode):
+
 2018-10-25  Devin Rousso  <drousso@apple.com>
 

trunk/Source/WebCore/inspector/agents/InspectorDOMAgent.cpp

@@ -247 +247 @@
         RefPtr<JSON::Object> data = JSON::Object::create();
 
+#if ENABLE(FULLSCREEN_API)
         if (event.type() == eventNames().webkitfullscreenchangeEvent)
             data->setBoolean("enabled"_s, !!node->document().webkitFullscreenElement());
+#endif // ENABLE(FULLSCREEN_API)
 
         auto timestamp = m_domAgent.m_environment.executionStopwatch()->elapsedTime().seconds();
@@ -2150 +2152 @@
     };
 
-    if (is<Document>(node))
+#if ENABLE(FULLSCREEN_API)
+    if (is<Document>(node) || is<HTMLMediaElement>(node))
         createEventListener(eventNames().webkitfullscreenchangeEvent);
-    else if (is<HTMLMediaElement>(node)) {
-        createEventListener(eventNames().webkitfullscreenchangeEvent);
+#endif // ENABLE(FULLSCREEN_API)
+
+    if (is<HTMLMediaElement>(node)) {
         createEventListener(eventNames().abortEvent);
         createEventListener(eventNames().canplayEvent);