Remove leftover code from EWebView

Patch removes internal classes/methods no longer needed,
namely:

- EWebView::SelectClosestWord
- AsyncRequestHitTestPrivate (declaration only)
- class WebViewAsyncRequestHitTestDataInternalCallback

Original beta/m47 patches:
- http://165.213.202.130/gerrit/#/c/102076/
- http://165.213.202.130/gerrit/#/c/102075/

Bug: http://107.108.218.239/bugzilla/show_bug.cgi?id=15459

Reviewed by: djmix.kim, g.czajkowski

Change-Id: Icf84b4f921ecd943f01e747b0de9e3e05c7ca5b0
Signed-off-by: Antonio Gomes <a1.gomes@samsung.com>

diff --git a/tizen_src/ewk/efl_integration/eweb_context.cc b/tizen_src/ewk/efl_integration/eweb_context.cc
index b781a3427a736fe40bc8c9250d8dfd643b9a0793..d3f0a53ea1b1dfe38e4a9e16b1f9acb9246fb127 100644 (file)
--- a/tizen_src/ewk/efl_integration/eweb_context.cc
+++ b/tizen_src/ewk/efl_integration/eweb_context.cc
@@ -49,8 +49,6 @@ using std::string;
 using std::pair;
 using std::map;
 
-EWebContext* EWebContext::default_context_ = NULL;
-
 namespace {
 
 /**

diff --git a/tizen_src/ewk/efl_integration/eweb_context.h b/tizen_src/ewk/efl_integration/eweb_context.h
index 988fb2f48c47373d53fbe81477356412d4016727..67ea7984da9b9f38418107fbf4a51271f28c33b0 100644 (file)
--- a/tizen_src/ewk/efl_integration/eweb_context.h
+++ b/tizen_src/ewk/efl_integration/eweb_context.h
@@ -179,7 +179,6 @@ class EWebContext {
   ~EWebContext();
   friend class Ewk_Context;
 
-  static EWebContext* default_context_;
   scoped_ptr<WebCacheManagerEfl> web_cache_manager_;
   scoped_ptr<content::BrowserContextEfl> browser_context_;
   scoped_ptr<Ewk_Cookie_Manager> ewk_cookie_manager_;

diff --git a/tizen_src/ewk/efl_integration/eweb_view.cc b/tizen_src/ewk/efl_integration/eweb_view.cc
index 3e1d330ded11c3677187ec8dad6a6adaa4eb15ba..337bb3720a7a2503842fd1643a312d063cc33e70 100644 (file)
--- a/tizen_src/ewk/efl_integration/eweb_view.cc
+++ b/tizen_src/ewk/efl_integration/eweb_view.cc
@@ -145,54 +145,6 @@ class WebViewAsyncRequestHitTestDataUserCallback: public WebViewAsyncRequestHitT
   void* user_data_;
 };
 
-class WebViewAsyncRequestHitTestDataInternalCallback: public WebViewAsyncRequestHitTestDataCallback
-{
- public:
-  typedef void (EWebView::*Callback)(int, int, Ewk_Hit_Test_Mode, _Ewk_Hit_Test*);
-
- public:
-  WebViewAsyncRequestHitTestDataInternalCallback(int x,
-                                                 int y,
-                                                 Ewk_Hit_Test_Mode mode,
-                                                 Callback cb)
-      : WebViewAsyncRequestHitTestDataCallback(x, y, mode)
-      , callback_(cb) {
-  }
-
-  void Run(_Ewk_Hit_Test* hit_test, EWebView* web_view) override {
-    DCHECK(callback_);
-    (web_view->*callback_)(GetX(), GetY(), GetMode(), hit_test);
-  }
-
- private:
-  Callback callback_;
-};
-
-class AsyncHitTestRequest
-{
- public:
-  AsyncHitTestRequest(int x, int y, Ewk_Hit_Test_Mode mode,
-      Ewk_View_Hit_Test_Request_Callback callback, void* user_data)
-    : x_(x)
-    , y_(y)
-    , mode_(mode)
-    , callback_(callback)
-    , user_data_(user_data) {
-  }
-
-  void Run(_Ewk_Hit_Test *hit_test, Evas_Object* web_view) {
-    DCHECK(callback_);
-    callback_(web_view, x_, y_, mode_, hit_test, user_data_);
-  }
-
- private:
-  int x_;
-  int y_;
-  Ewk_Hit_Test_Mode mode_;
-  Ewk_View_Hit_Test_Request_Callback callback_;
-  void* user_data_;
-};
-
 class WebViewGeolocationPermissionCallback {
  public:
   WebViewGeolocationPermissionCallback(Ewk_View_Geolocation_Permission_Callback cb, void* data)
@@ -1297,15 +1249,6 @@ SelectionControllerEfl* EWebView::GetSelectionController() const {
   return view ? view->GetSelectionController() : 0;
 }
 
-void EWebView::SelectClosestWord(const gfx::Point& touch_point) {
-  int view_x, view_y;
-  EvasToBlinkCords(touch_point.x(), touch_point.y(), &view_x, &view_y);
-
-  RenderViewHost* render_view_host = web_contents_->GetRenderViewHost();
-  render_view_host->Send(new ViewMsg_SelectClosestWord(
-      render_view_host->GetRoutingID(), view_x, view_y));
-}
-
 void EWebView::SelectLinkText(const gfx::Point& touch_point) {
   float device_scale_factor = gfx::Screen::GetNativeScreen()->
       GetPrimaryDisplay().device_scale_factor();

diff --git a/tizen_src/ewk/efl_integration/eweb_view.h b/tizen_src/ewk/efl_integration/eweb_view.h
index 5e2f922f33ddd0c98312cc8721afb5db745c58da..d424a6a52511b798430f8ff017de712e6ad938f1 100644 (file)
--- a/tizen_src/ewk/efl_integration/eweb_view.h
+++ b/tizen_src/ewk/efl_integration/eweb_view.h
@@ -305,7 +305,6 @@ class EWebView {
   void MoveCaret(const gfx::Point& point);
   void QuerySelectionStyle();
   void OnQuerySelectionStyleReply(const SelectionStylePrams& params);
-  void SelectClosestWord(const gfx::Point& touch_point);
   void SelectLinkText(const gfx::Point& touch_point);
   bool GetSelectionRange(Eina_Rectangle* left_rect, Eina_Rectangle* right_rect);
   Eina_Bool ClearSelection();
@@ -547,10 +546,6 @@ class EWebView {
       create_new_window_web_contents_cb_;
 
 private:
-  Eina_Bool AsyncRequestHitTestPrivate(
-      int x, int y, Ewk_Hit_Test_Mode mode,
-      AsyncHitTestRequest* asyncHitTestRequest);
-
   gfx::Vector2d previous_scroll_position_;
 
   gfx::Point context_menu_position_;