Make EWK use WebContentsViewDelegate for SetPageTitle handling.

This patch makes EWK to set page title via WebContentsViewDelegate::SetPageTitle fucntion.

Change-Id: I88d4011bb946d78515ff0e4d67a27a9e42f44d25
Signed-off-by: hh4813.kim <hh4813.kim@samsung.com>

diff --git a/tizen_src/chromium_impl/content/browser/web_contents/web_contents_view_efl.cc b/tizen_src/chromium_impl/content/browser/web_contents/web_contents_view_efl.cc
index 6c75935..e3c01bd 100644 (file)
--- a/tizen_src/chromium_impl/content/browser/web_contents/web_contents_view_efl.cc
+++ b/tizen_src/chromium_impl/content/browser/web_contents/web_contents_view_efl.cc
@@ -75,7 +75,8 @@ RenderWidgetHostViewBase* WebContentsViewEfl::CreateViewForPopupWidget(
 }
 
 void WebContentsViewEfl::SetPageTitle(const base::string16& title) {
-  DCHECK(native_view_);
+  if (delegate_)
+    delegate_->SetPageTitle(title);
 }
 
 void WebContentsViewEfl::UpdateDragDest(RenderViewHost* host) {

diff --git a/tizen_src/ewk/efl_integration/web_contents_view_delegate_ewk.cc b/tizen_src/ewk/efl_integration/web_contents_view_delegate_ewk.cc
index 8664202..883b1d2 100644 (file)
--- a/tizen_src/ewk/efl_integration/web_contents_view_delegate_ewk.cc
+++ b/tizen_src/ewk/efl_integration/web_contents_view_delegate_ewk.cc
@@ -31,3 +31,7 @@ void WebContentsViewDelegateEwk::ShowPopupMenu(
 void WebContentsViewDelegateEwk::HidePopupMenu() {
   web_view_->HidePopupMenu();
 }
+
+void WebContentsViewDelegateEwk::SetPageTitle(const base::string16& title) {
+  web_view_->SmartCallback<EWebViewCallbacks::TitleChange>().call(base::UTF16ToUTF8(title).c_str());
+}

diff --git a/tizen_src/ewk/efl_integration/web_contents_view_delegate_ewk.h b/tizen_src/ewk/efl_integration/web_contents_view_delegate_ewk.h
index b59978c..a885fbd 100644 (file)
--- a/tizen_src/ewk/efl_integration/web_contents_view_delegate_ewk.h
+++ b/tizen_src/ewk/efl_integration/web_contents_view_delegate_ewk.h
@@ -33,6 +33,8 @@ class WebContentsViewDelegateEwk : public content::WebContentsViewDelegate {
       bool allow_multiple_selection) override;
   void HidePopupMenu() override;
 
+  void SetPageTitle(const base::string16& title) override;
+
  private:
   EWebView* web_view_;
 };