Add window channel#201
Add window channel#201swift-kim merged 7 commits intoflutter-tizen:flutter-2.5.1-tizenfrom HakkyuKim:flutter-2.5.1-add-window-channel
Conversation
|
Resizing the window for evas_gl is not displaying the same as ecore_wl2 and I can't figure out why. The engines used in the following images are compiled using the same logic with different WearableTizenRendererEcoreWl2TizenRendererEvasGLTVTizenRendererEcoreWl2TizenRendererEvasGL@bbrto21 @xiaowei-guan Would you happen to have any information on this? |
|
Just curious... why did you disable at x64? (Is this for evas issue? Same result on x64?) |
My understanding was that you could only test apis that are purely EFL on x64, the current implementation calls tizen only apis; |
swift-kim
left a comment
There was a problem hiding this comment.
Otherwise looks good to me!
Done! Thanks for the review:) |
There was a problem hiding this comment.
My understanding was that you could only test apis that are purely EFL on x64, the current implementation calls tizen only apis; ecore_wl2_window_position_set and ecore_wl2_egl_window_resize_with_rotation. Or am I mistaken?
First , it's not really important. I just asked....
I think I changed the window geometry in x64 before.. and we could only test the window channel operation (Not real behavior!) at unit test.. I just wondered why it was blocked at this time. Now I understand through your response.
Thanks! It's pretty cool! 👍
I'm working on another task, so I can't afford to look into this. Sorry for the late reply. |
I was double checking if I was correct about the |
|
Please resolve conflicts. |
Also: - remove unused include - sort channels alphabetically - sort channel initializations alphabetically
Also: - call `result->NotImplemented()` instead of `result->Error()` on unsupported window resizing.
Also: - Add `explicit` to `WindowChannel` constructor. - Disable `OnGeometryChange` for evas_gl. - Remove unused include.
Done, merging would've been better, the conflict was: -event.width = static_cast<size_t>(width);
-event.height = static_cast<size_t>(height);
+event.width = width;
+event.height = height; |
* Add window channel * Obtain real screen size * Rename some namings Also: - remove unused include - sort channels alphabetically - sort channel initializations alphabetically * Move SetGeometry to TizenRender to remove macros Also: - call `result->NotImplemented()` instead of `result->Error()` on unsupported window resizing. * Finish resolving previous conflict * Suppress unused private field warning Also: - Add `explicit` to `WindowChannel` constructor. - Disable `OnGeometryChange` for evas_gl. - Remove unused include. * Explicitly initialize with zeros
* Add window channel * Obtain real screen size * Rename some namings Also: - remove unused include - sort channels alphabetically - sort channel initializations alphabetically * Move SetGeometry to TizenRender to remove macros Also: - call `result->NotImplemented()` instead of `result->Error()` on unsupported window resizing. * Finish resolving previous conflict * Suppress unused private field warning Also: - Add `explicit` to `WindowChannel` constructor. - Disable `OnGeometryChange` for evas_gl. - Remove unused include. * Explicitly initialize with zeros
* Add window channel * Obtain real screen size * Rename some namings Also: - remove unused include - sort channels alphabetically - sort channel initializations alphabetically * Move SetGeometry to TizenRender to remove macros Also: - call `result->NotImplemented()` instead of `result->Error()` on unsupported window resizing. * Finish resolving previous conflict * Suppress unused private field warning Also: - Add `explicit` to `WindowChannel` constructor. - Disable `OnGeometryChange` for evas_gl. - Remove unused include. * Explicitly initialize with zeros
* Add window channel * Obtain real screen size * Rename some namings Also: - remove unused include - sort channels alphabetically - sort channel initializations alphabetically * Move SetGeometry to TizenRender to remove macros Also: - call `result->NotImplemented()` instead of `result->Error()` on unsupported window resizing. * Finish resolving previous conflict * Suppress unused private field warning Also: - Add `explicit` to `WindowChannel` constructor. - Disable `OnGeometryChange` for evas_gl. - Remove unused include. * Explicitly initialize with zeros
* Add window channel * Obtain real screen size * Rename some namings Also: - remove unused include - sort channels alphabetically - sort channel initializations alphabetically * Move SetGeometry to TizenRender to remove macros Also: - call `result->NotImplemented()` instead of `result->Error()` on unsupported window resizing. * Finish resolving previous conflict * Suppress unused private field warning Also: - Add `explicit` to `WindowChannel` constructor. - Disable `OnGeometryChange` for evas_gl. - Remove unused include. * Explicitly initialize with zeros
* Add window channel * Obtain real screen size * Rename some namings Also: - remove unused include - sort channels alphabetically - sort channel initializations alphabetically * Move SetGeometry to TizenRender to remove macros Also: - call `result->NotImplemented()` instead of `result->Error()` on unsupported window resizing. * Finish resolving previous conflict * Suppress unused private field warning Also: - Add `explicit` to `WindowChannel` constructor. - Disable `OnGeometryChange` for evas_gl. - Remove unused include. * Explicitly initialize with zeros
* Add window channel * Obtain real screen size * Rename some namings Also: - remove unused include - sort channels alphabetically - sort channel initializations alphabetically * Move SetGeometry to TizenRender to remove macros Also: - call `result->NotImplemented()` instead of `result->Error()` on unsupported window resizing. * Finish resolving previous conflict * Suppress unused private field warning Also: - Add `explicit` to `WindowChannel` constructor. - Disable `OnGeometryChange` for evas_gl. - Remove unused include. * Explicitly initialize with zeros
* Add window channel * Obtain real screen size * Rename some namings Also: - remove unused include - sort channels alphabetically - sort channel initializations alphabetically * Move SetGeometry to TizenRender to remove macros Also: - call `result->NotImplemented()` instead of `result->Error()` on unsupported window resizing. * Finish resolving previous conflict * Suppress unused private field warning Also: - Add `explicit` to `WindowChannel` constructor. - Disable `OnGeometryChange` for evas_gl. - Remove unused include. * Explicitly initialize with zeros




The use case for setting window geometry at runtime is explained here (internal repo).
Test app: https://github.com/HakkyuKim/flutter_tizen_window_resize_test_app
TODO
Can't figure out why, decided not to support in this PR since it's not important compared to other tasks.
Must also call
ecore_wl2_window_position_set.Test with rotation.I'll write a "remaining issues" in the issue tab.
Check dynamic type ofUse conditional compilation.TizenRendererinWindowChannel::HandleMethodCall().