Title: [145403] trunk/Source/WebCore
- Revision
- 145403
- Author
- [email protected]
- Date
- 2013-03-11 14:06:44 -0700 (Mon, 11 Mar 2013)
Log Message
Fix build for r145397
Unreviewed.
* html/shadow/MediaControls.cpp:
(WebCore::MediaControls::createTextTrackDisplay):
* html/shadow/MediaControlsChromium.cpp:
(WebCore::MediaControlsChromium::createTextTrackDisplay):
* html/shadow/MediaControlsGtk.cpp:
(WebCore::MediaControlsGtk::createTextTrackDisplay):
Modified Paths
Diff
Modified: trunk/Source/WebCore/ChangeLog (145402 => 145403)
--- trunk/Source/WebCore/ChangeLog 2013-03-11 21:03:58 UTC (rev 145402)
+++ trunk/Source/WebCore/ChangeLog 2013-03-11 21:06:44 UTC (rev 145403)
@@ -1,3 +1,16 @@
+2013-03-11 Dima Gorbik <[email protected]>
+
+ Fix build for r145397
+
+ Unreviewed.
+
+ * html/shadow/MediaControls.cpp:
+ (WebCore::MediaControls::createTextTrackDisplay):
+ * html/shadow/MediaControlsChromium.cpp:
+ (WebCore::MediaControlsChromium::createTextTrackDisplay):
+ * html/shadow/MediaControlsGtk.cpp:
+ (WebCore::MediaControlsGtk::createTextTrackDisplay):
+
2013-03-11 Abhishek Arya <[email protected]>
Add ASSERT_WITH_SECURITY_IMPLICATION to catch bad casts.
Modified: trunk/Source/WebCore/html/shadow/MediaControls.cpp (145402 => 145403)
--- trunk/Source/WebCore/html/shadow/MediaControls.cpp 2013-03-11 21:03:58 UTC (rev 145402)
+++ trunk/Source/WebCore/html/shadow/MediaControls.cpp 2013-03-11 21:06:44 UTC (rev 145403)
@@ -382,7 +382,7 @@
m_textDisplayContainer->setMediaController(m_mediaController);
// Insert it before the first controller element so it always displays behind the controls.
- insertBefore(textDisplayContainer.release(), m_panel, IGNORE_EXCEPTION, true);
+ insertBefore(textDisplayContainer.release(), m_panel, IGNORE_EXCEPTION, AttachLazily);
}
void MediaControls::showTextTrackDisplay()
Modified: trunk/Source/WebCore/html/shadow/MediaControlsChromium.cpp (145402 => 145403)
--- trunk/Source/WebCore/html/shadow/MediaControlsChromium.cpp 2013-03-11 21:03:58 UTC (rev 145402)
+++ trunk/Source/WebCore/html/shadow/MediaControlsChromium.cpp 2013-03-11 21:06:44 UTC (rev 145403)
@@ -215,7 +215,7 @@
// Insert it before the first controller element so it always displays behind the controls.
// In the Chromium case, that's the enclosure element.
- insertBefore(textDisplayContainer.release(), m_enclosure, ASSERT_NO_EXCEPTION, true);
+ insertBefore(textDisplayContainer.release(), m_enclosure, ASSERT_NO_EXCEPTION, AttachLazily);
}
#endif
Modified: trunk/Source/WebCore/html/shadow/MediaControlsGtk.cpp (145402 => 145403)
--- trunk/Source/WebCore/html/shadow/MediaControlsGtk.cpp 2013-03-11 21:03:58 UTC (rev 145402)
+++ trunk/Source/WebCore/html/shadow/MediaControlsGtk.cpp 2013-03-11 21:06:44 UTC (rev 145403)
@@ -223,7 +223,7 @@
m_textDisplayContainer->setMediaController(m_mediaController);
// Insert it before the first controller element so it always displays behind the controls.
- insertBefore(textDisplayContainer.release(), m_enclosure, ASSERT_NO_EXCEPTION, true);
+ insertBefore(textDisplayContainer.release(), m_enclosure, ASSERT_NO_EXCEPTION, AttachLazily);
}
#endif
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes
