Fix an invalid engine handle error

[bbrto21]

• To send something to the engine, the engine must be running.
• FlutterTizenView::SendInitialGeometry is used to send the geometry
  after running the engine

Signed-off-by: Boram Bae boram21.bae@samsung.com

Fixes flutter-tizen/flutter-tizen#361.

[JSUYA]

• FlutterTizenView::SendInitialGeometry is used to send the geometry
  after running the engine

Not directly related to this patch, but...
SendInitialGeometry() is called regardless of engine run fail.

  if (!view->engine()->IsRunning()) {
    view->engine()->RunEngine();
  }

  view->SendInitialGeometry();

How about adding an if condition?

[bbrto21]

@JSUYA As you already mentioned, I also don't think that's the scope of this PR. So if you really think it's necessary, you can open the PR yourself.

[swift-kim]

@bbrto21 @JSUYA That looks like a bug we currently have. FlutterDesktopViewCreateFromNewWindow should return nullptr if RunEngine fails. That is,

if (!view->engine()->IsRunning()) {
  if (!view->engine()->RunEngine()) {
    return nullptr;
  }
}

[bbrto21]

@bbrto21 @JSUYA That looks like a bug we currently have. FlutterDesktopViewCreateFromNewWindow should return nullptr if RunEngine fails. That is,

@JSUYA Would you please fix this bug?