Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(134)

Unified Diff: Source/core/rendering/RenderMediaControls.cpp

Issue 456323002: [WIP] Re-implement MediaControls in Blink-in-JS (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 6 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: Source/core/rendering/RenderMediaControls.cpp
diff --git a/Source/core/rendering/RenderMediaControls.cpp b/Source/core/rendering/RenderMediaControls.cpp
index 6288aa3969d8087ddf1e4f2898fa61c128ea5b9a..9cf650368045566e52e95c0ee95f8c47708a3160 100644
--- a/Source/core/rendering/RenderMediaControls.cpp
+++ b/Source/core/rendering/RenderMediaControls.cpp
@@ -68,7 +68,7 @@ static bool paintMediaButton(GraphicsContext* context, const IntRect& rect, Imag
static bool paintMediaMuteButton(RenderObject* object, const PaintInfo& paintInfo, const IntRect& rect)
{
- HTMLMediaElement* mediaElement = toParentMediaElement(object);
+ /*HTMLMediaElement* mediaElement = toParentMediaElement(object);
if (!mediaElement)
return false;
@@ -90,12 +90,13 @@ static bool paintMediaMuteButton(RenderObject* object, const PaintInfo& paintInf
if (mediaElement->volume() <= 0.66)
return paintMediaButton(paintInfo.context, rect, soundLevel2);
- return paintMediaButton(paintInfo.context, rect, soundLevel3);
+ return paintMediaButton(paintInfo.context, rect, soundLevel3);*/
+ return true;
}
static bool paintMediaPlayButton(RenderObject* object, const PaintInfo& paintInfo, const IntRect& rect)
{
- HTMLMediaElement* mediaElement = toParentMediaElement(object);
+ /*HTMLMediaElement* mediaElement = toParentMediaElement(object);
if (!mediaElement)
return false;
@@ -106,7 +107,8 @@ static bool paintMediaPlayButton(RenderObject* object, const PaintInfo& paintInf
if (!hasSource(mediaElement))
return paintMediaButton(paintInfo.context, rect, mediaPlayDisabled);
- return paintMediaButton(paintInfo.context, rect, mediaControlElementType(object->node()) == MediaPlayButton ? mediaPlay : mediaPause);
+ return paintMediaButton(paintInfo.context, rect, mediaControlElementType(object->node()) == MediaPlayButton ? mediaPlay : mediaPause);*/
+ return true;
}
static bool paintMediaOverlayPlayButton(RenderObject* object, const PaintInfo& paintInfo, const IntRect& rect)
@@ -323,7 +325,7 @@ static bool paintMediaFullscreenButton(RenderObject* object, const PaintInfo& pa
static bool paintMediaToggleClosedCaptionsButton(RenderObject* object, const PaintInfo& paintInfo, const IntRect& rect)
{
- HTMLMediaElement* mediaElement = toParentMediaElement(object);
+ /*HTMLMediaElement* mediaElement = toParentMediaElement(object);
if (!mediaElement)
return false;
@@ -333,7 +335,8 @@ static bool paintMediaToggleClosedCaptionsButton(RenderObject* object, const Pai
if (mediaElement->closedCaptionsVisible())
return paintMediaButton(paintInfo.context, rect, mediaClosedCaptionButton);
- return paintMediaButton(paintInfo.context, rect, mediaClosedCaptionButtonDisabled);
+ return paintMediaButton(paintInfo.context, rect, mediaClosedCaptionButtonDisabled);*/
+ return true;
}
« Source/core/html/shadow/MediaControls.idl ('K') | « Source/core/html/shadow/MediaControls.js ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698