Refactor ContentViewClient (1/6)

content/public/browser/web_contents_delegate.cc

Index: content/public/browser/web_contents_delegate.cc
diff --git a/content/public/browser/web_contents_delegate.cc b/content/public/browser/web_contents_delegate.cc
index 399255a3b004404535e33cddf324268b323e747b..dc35834a3cc9d5443068181af2ecd6a2dca6b2bd 100644
--- a/content/public/browser/web_contents_delegate.cc
+++ b/content/public/browser/web_contents_delegate.cc
@@ -208,6 +208,16 @@ void WebContentsDelegate::RequestMediaDecodePermission(
     const base::Callback<void(bool)>& callback) {
   callback.Run(false);
 }
+
+base::android::ScopedJavaLocalRef<jobject>
+WebContentsDelegate::GetContentVideoViewEmbedder() {
+  NOTREACHED() << "Implementation must be provided through subclass.";

[boliu, 2016/09/26 21:25:25]

generally content allow embedders to selectivey override methods, plus you
already have null checks I think, so just remove this line?

[Jinsuk Kim, 2016/09/27 10:08:59]

Should have been removed. Done.

+  return base::android::ScopedJavaLocalRef<jobject>();
+}
+
+bool WebContentsDelegate::ShouldBlockMediaRequest(const GURL& url) {
+  return false;
+}
 #endif
 
 bool WebContentsDelegate::RequestPpapiBrokerPermission(