Merge ~abreu-alexandre/oxide:mh into oxide:master

Proposed by Alexandre Abreu
Status: Merged
Approved by: Chris Coulson
Approved revision: 0df6a7ce132e7ae6e2ec5aa1e97667c9fe01127b
Merge reported by: Chris Coulson
Merged at revision: not available
Proposed branch: ~abreu-alexandre/oxide:mh
Merge into: oxide:master
Diff against target: 58 lines (+1/-10)
2 files modified
shared/renderer/media/oxide_web_media_player.cc (+0/-5)
shared/renderer/media/oxide_web_media_player.h (+1/-5)
Reviewer Review Type Date Requested Status
Chris Coulson Approve
Review via email: mp+314426@code.launchpad.net

Description of the change

Fix mh build w/ deprecated media player paint()

To post a comment you must log in.
Revision history for this message
Chris Coulson (chrisccoulson) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/shared/renderer/media/oxide_web_media_player.cc b/shared/renderer/media/oxide_web_media_player.cc
2index 9fd3e41..a641a5b 100644
3--- a/shared/renderer/media/oxide_web_media_player.cc
4+++ b/shared/renderer/media/oxide_web_media_player.cc
5@@ -32,7 +32,6 @@
6 #include "gpu/command_buffer/client/gles2_interface.h"
7 #include "gpu/command_buffer/common/mailbox_holder.h"
8 #include "media/base/bind_to_current_loop.h"
9-#include "media/base/media_keys.h"
10 #include "media/base/media_switches.h"
11 #include "media/base/timestamp_constants.h"
12 #include "media/base/video_frame.h"
13@@ -406,10 +405,6 @@ size_t WebMediaPlayer::videoDecodedByteCount() const {
14 return 0;
15 }
16
17-void WebMediaPlayer::paint(blink::WebCanvas*, const blink::WebRect&, unsigned char alpha, SkXfermode::Mode) {
18- NOTIMPLEMENTED();
19-}
20-
21 void WebMediaPlayer::OnHidden() {
22 // RendererMediaPlayerManager will not call SuspendAndReleaseResources() if we
23 // were already in the paused state; thus notify the MediaWebContentsObserver
24diff --git a/shared/renderer/media/oxide_web_media_player.h b/shared/renderer/media/oxide_web_media_player.h
25index 2380906..22427d8 100644
26--- a/shared/renderer/media/oxide_web_media_player.h
27+++ b/shared/renderer/media/oxide_web_media_player.h
28@@ -17,12 +17,10 @@
29 #include "content/public/renderer/render_frame_observer.h"
30 #include "gpu/command_buffer/common/mailbox.h"
31 #include "media/base/demuxer_stream.h"
32-#include "media/base/media_keys.h"
33 #include "media/blink/webmediaplayer_delegate.h"
34 #include "third_party/WebKit/public/platform/WebMediaPlayer.h"
35 #include "third_party/WebKit/public/platform/WebSize.h"
36 #include "third_party/WebKit/public/platform/WebURL.h"
37-#include "third_party/skia/include/core/SkXfermode.h"
38 #include "ui/gfx/geometry/rect_f.h"
39 #include "shared/common/oxide_message_enums.h"
40
41@@ -93,7 +91,7 @@ class WebMediaPlayer : public blink::WebMediaPlayer,
42 // Methods for painting.
43 void paint(blink::WebCanvas* canvas,
44 const blink::WebRect& rect,
45- SkPaint&);
46+ SkPaint&) override;
47
48 // True if the loaded media has a playable video/audio track.
49 bool hasVideo() const;
50@@ -127,8 +125,6 @@ class WebMediaPlayer : public blink::WebMediaPlayer,
51 size_t audioDecodedByteCount() const;
52 size_t videoDecodedByteCount() const;
53
54- void paint(blink::WebCanvas*, const blink::WebRect&, unsigned char alpha, SkXfermode::Mode);
55- //
56 // Media player callback handlers.
57 void OnMediaMetadataChanged(const base::TimeDelta& duration, int width,
58 int height, bool success);

Subscribers

People subscribed via source and target branches