Attachment 339667 Details for Bug 185353 – Patch

[patch] Patch

bug-185353-20180505203358.patch (text/plain), 4.52 KB, created by Eric Carlson on 2018-05-05 20:33:58 PDT

(hide)

Description:

Filename:

MIME Type:

Creator: Eric Carlson

Created: 2018-05-05 20:33:58 PDT

Size: 4.52 KB

patch

obsolete

>Subversion Revision: 231384
>diff --git a/Source/WebCore/ChangeLog b/Source/WebCore/ChangeLog
>index 99990c71c2b226d7260aff50d7c974a0d28d8716..d14df6d09feaa79e3e27c7ef1a63b9445783ac6e 100644
>--- a/Source/WebCore/ChangeLog
>+++ b/Source/WebCore/ChangeLog
>@@ -1,3 +1,24 @@
>+2018-05-05  Eric Carlson  <eric.carlson@apple.com>
>+
>+        Text track cue logging should include cue text
>+        https://bugs.webkit.org/show_bug.cgi?id=185353
>+        <rdar://problem/40003565>
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        No new tests, tested manually.
>+
>+        * html/track/VTTCue.cpp:
>+        (WebCore::VTTCue::toJSONString const): Use toJSON.
>+        (WebCore::VTTCue::toJSON const): New.
>+        * html/track/VTTCue.h:
>+
>+        * platform/graphics/InbandTextTrackPrivateClient.h:
>+        (WebCore::GenericCueData::toJSONString const): Log m_content.
>+
>+        * platform/graphics/iso/ISOVTTCue.cpp:
>+        (WebCore::ISOWebVTTCue::toJSONString const): Log m_cueText.
>+
> 2018-05-04  Zalan Bujtas  <zalan@apple.com>
> 
>         [Simple line layout] Add support for line layout box generation with multiple text renderers.
>diff --git a/Source/WebCore/html/track/VTTCue.cpp b/Source/WebCore/html/track/VTTCue.cpp
>index 78f8adfc37e5d5c7c64b983fb7d0c54497646d7b..ed03bb874c4169e5785b36e7ea3a7d97321363f6 100644
>--- a/Source/WebCore/html/track/VTTCue.cpp
>+++ b/Source/WebCore/html/track/VTTCue.cpp
>@@ -1182,21 +1182,26 @@ const VTTCue* toVTTCue(const TextTrackCue* cue)
> String VTTCue::toJSONString() const
> {
>     auto object = JSON::Object::create();
>-
>-    TextTrackCue::toJSON(object.get());
>-
>-    object->setString(ASCIILiteral("vertical"), vertical());
>-    object->setBoolean(ASCIILiteral("snapToLines"), snapToLines());
>-    object->setDouble(ASCIILiteral("line"), m_linePosition);
>-    object->setDouble(ASCIILiteral("position"), position());
>-    object->setInteger(ASCIILiteral("size"), m_cueSize);
>-    object->setString(ASCIILiteral("align"), align());
>-    object->setString(ASCIILiteral("text"), text());
>-    object->setString(ASCIILiteral("regionId"), regionId());
>+    toJSON(object.get());
> 
>     return object->toJSONString();
> }
> 
>+void VTTCue::toJSON(JSON::Object& object) const
>+{
>+    TextTrackCue::toJSON(object);
>+
>+    object.setString(ASCIILiteral("text"), text());
>+    object.setString(ASCIILiteral("vertical"), vertical());
>+    object.setBoolean(ASCIILiteral("snapToLines"), snapToLines());
>+    object.setDouble(ASCIILiteral("line"), m_linePosition);
>+    object.setDouble(ASCIILiteral("position"), position());
>+    object.setInteger(ASCIILiteral("size"), m_cueSize);
>+    object.setString(ASCIILiteral("align"), align());
>+    object.setString(ASCIILiteral("text"), text());
>+    object.setString(ASCIILiteral("regionId"), regionId());
>+}
>+
> } // namespace WebCore
> 
> #endif
>diff --git a/Source/WebCore/html/track/VTTCue.h b/Source/WebCore/html/track/VTTCue.h
>index 3d968d62addbea0f84ddb0d534f8d2d8cc6efd94..88d4611b25eea7d4b013bc6d95e178eeb5437adc 100644
>--- a/Source/WebCore/html/track/VTTCue.h
>+++ b/Source/WebCore/html/track/VTTCue.h
>@@ -177,6 +177,8 @@ protected:
>     virtual Ref<VTTCueBox> createDisplayTree();
>     VTTCueBox& displayTreeInternal();
> 
>+    void toJSON(JSON::Object&) const final;
>+
> private:
>     void initialize(ScriptExecutionContext&);
>     void createWebVTTNodeTree();
>diff --git a/Source/WebCore/platform/graphics/InbandTextTrackPrivateClient.h b/Source/WebCore/platform/graphics/InbandTextTrackPrivateClient.h
>index ccbb3d655f93c814be21ed85ddc2e0b9715d1c7e..bc5c427f7580d0e45722259ccd4e8449726af35f 100644
>--- a/Source/WebCore/platform/graphics/InbandTextTrackPrivateClient.h
>+++ b/Source/WebCore/platform/graphics/InbandTextTrackPrivateClient.h
>@@ -120,6 +120,7 @@ inline String GenericCueData::toJSONString() const
> {
>     auto object = JSON::Object::create();
> 
>+    object->setString("text", m_content);
>     object->setDouble(ASCIILiteral("start"), m_startTime.toDouble());
>     object->setDouble(ASCIILiteral("end"), m_endTime.toDouble());
> 
>diff --git a/Source/WebCore/platform/graphics/iso/ISOVTTCue.cpp b/Source/WebCore/platform/graphics/iso/ISOVTTCue.cpp
>index a063bd22a36ff39d66f1fc67548604d9814923da..327fe9a7c7637c3a8b969d83852281b8f52536a0 100644
>--- a/Source/WebCore/platform/graphics/iso/ISOVTTCue.cpp
>+++ b/Source/WebCore/platform/graphics/iso/ISOVTTCue.cpp
>@@ -114,6 +114,7 @@ String ISOWebVTTCue::toJSONString() const
> {
>     auto object = JSON::Object::create();
> 
>+    object->setString(ASCIILiteral("text"), m_cueText);
>     object->setString(ASCIILiteral("sourceId"), m_sourceID);
>     object->setString(ASCIILiteral("id"), m_identifier);
>

Actions: View | Formatted Diff | Diff

Attachments on bug 185353: 339657 | 339667 | 339720 | 339753