[Webkit-unassigned] [Bug 136736] [EFL][WK2] Minibrowser : Add support for mouse middle button to open links in new window

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=136736





--- Comment #7 from Gyuyoung Kim <gyuyoung.kim at webkit.org>  2014-10-06 01:34:41 PST ---
(In reply to comment #4)

> > > Tools/MiniBrowser/efl/main.c:1900
> > > +        ewk_navigation_policy_decision_reject(decision); 
> > 
> > Doesn't we call ewk_navigation_policy_decision_accept() ? And It looks we need to call this reject function when ewk_navigation_policy_mouse_button_get(decision) is not EWK_EVENT_MOUSE_BUTTON_MIDDLE, isn't it ?
> 
> As per the bug https://bugs.webkit.org/show_bug.cgi?id=90953
> by default the navigation request is accepted. So I think there is no need to call ewk_navigation_policy_decision_accept().
> Also if we call reject function when ewk_navigation_policy_mouse_button_get(decision) is not EWK_EVENT_MOUSE_BUTTON_MIDDLE, then we will not be able to navigate to any page for all cases other than mouse middle button case. And adding reject at the end makes sure that we do not open the same page in parent and child window.

I don't understand why we should call reject() yet. Could you explain it again ?

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.