yueying0083 / javachromiumembedded

Automatically exported from code.google.com/p/javachromiumembedded
0 stars 0 forks source link

Split CefClientDelegate in several handlers to get a more cef-like implementation. #46

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
This issue is an enhancement of JCEF (patch 2 of 10)
----------------------------------------------------
Purpose: Split CefClientDelegate in several handlers to
get a more cef-like implementation.

This patch relies on the attached patch file of issue #. 
If you want to try these code changes, add the patch of issue # first and 
afterwards add the patch file attached to this issue. Otherwise you'll get some 
patch-errors.

With this patch, the class CefClientDelegate is divided into the handler 
interfaces
- CefDisplayHandler
- CefRenderHandler
- CefMessageRouterHandler
- CefFocusHandler
to get the JCEF implementation closer to its JNI counterpart CEF.

For each of these interfaces an adapter class is provided which methods are 
empty. 
- CefDisplayHandlerAdapter
- CefRenderHandlerAdapter
- CefMessageRouterHandlerAdapter
- CefFocusHandlerAdapter
These classes exists as convenience for creating handler objects.

CefClient is extended by the methods
- addDisplayHandler(CefDisplayHandler handler)
- removeDisplayHandler()
- addFocusHandler(CefFocusHandler handler)
- removeFocusHandler()
- addMessageRouterHandler(CefMessageRouterHandler handler)
- removeMessageRouterHandler()
- addRenderHandler(CefRenderHandler handler)
- removeRenderHandler()
to add and remove the handler implementations to the client.

Please see attached patch file for further informations.

Original issue reported on code.google.com by k...@censhare.de on 8 Mar 2014 at 7:53

Attachments:

GoogleCodeExporter commented 9 years ago
This patch relies on the attached patch file of issue #45. 
If you want to try these code changes, add the patch of issue #45 first and 
afterwards add the patch file attached to this issue. Otherwise you'll get some 
patch-errors.

Original comment by k...@censhare.de on 8 Mar 2014 at 7:55

GoogleCodeExporter commented 9 years ago

Original comment by magreenb...@gmail.com on 12 Mar 2014 at 9:50

GoogleCodeExporter commented 9 years ago

Original comment by magreenb...@gmail.com on 12 Mar 2014 at 9:51

GoogleCodeExporter commented 9 years ago

Original comment by magreenb...@gmail.com on 12 Mar 2014 at 9:51

GoogleCodeExporter commented 9 years ago
Added in revision 33.

Original comment by magreenb...@gmail.com on 28 Mar 2014 at 10:51