Improve renderer process crash handling in EWK.
authorPiotr Tworek <p.tworek@samsung.com>
Thu, 14 May 2015 09:32:28 +0000 (11:32 +0200)
committerYoungsoo Choi <kenshin.choi@samsung.com>
Tue, 10 Jul 2018 06:57:09 +0000 (06:57 +0000)
commiteed46bd4a665dc6c7e7a023ffadeef86b093674e
tree3f8f3bb5daa7ebf187eb16532176d8ef59696e7c
parentf59fc6043d618b5817baf4c9ec6173fbc5bb77c5
Improve renderer process crash handling in EWK.

Since the refactoring removed a lot of unnecessary inter dependencies
between EWK and content layers we no longer need to re-create
WebContents each time the renderer process has crashed. On UI process
side renderer process is only represented by the RenderWidgetHostView
class. This fix brings a big benefit of not loosing browsing history
whenever renderer process crashes. Such state is associated with
WebContents not RWHV.

This patch also includes additional bug fix which makes sure
LoadHTMLString function is not called directly from a callback informing
us about a crash. Calling any load method directly from the crash
callback triggers a DCHECK in mojo code when renderer process crashes
multiple times for the same WebContents instance.
Reviewed by: Antonio Gomes, SeungSeop Park, Youngha Jung

Change-Id: Id220b563bea90e6dedd0b55a26ef856fb1528069
Signed-off-by: Piotr Tworek <p.tworek@samsung.com>
tizen_src/ewk/efl_integration/eweb_view.cc
tizen_src/ewk/efl_integration/eweb_view.h