Fix T69498: Crash on export UV Layout

Ensure there's a drawable context after the file browser window is
destroyed.
This commit is contained in:
Julian Eisel 2019-09-06 00:13:24 +02:00
parent fbf6898f58
commit 4c4a8bf588
Notes: blender-bot 2023-02-14 08:35:51 +01:00
Referenced by issue #69498, Crash on export UV Layout
2 changed files with 6 additions and 5 deletions

View File

@ -2376,6 +2376,8 @@ static int wm_handler_fileselect_do(bContext *C,
case EVT_FILESELECT_EXEC:
case EVT_FILESELECT_CANCEL:
case EVT_FILESELECT_EXTERNAL_CANCEL: {
wmWindow *ctx_win = CTX_wm_window(C);
/* remlink now, for load file case before removing*/
BLI_remlink(handlers, handler);
@ -2388,11 +2390,13 @@ static int wm_handler_fileselect_do(bContext *C,
BLI_assert(file_sa->spacetype == SPACE_FILE);
if (BLI_listbase_is_single(&file_sa->spacedata)) {
wmWindow *ctx_win = CTX_wm_window(C);
BLI_assert(ctx_win != win);
wm_window_close(C, wm, win);
CTX_wm_window_set(C, ctx_win); // wm_window_close() NULLs.
/* Some operators expect a drawable context (for EVT_FILESELECT_EXEC) */
wm_window_make_drawable(wm, ctx_win);
}
else if (file_sa->full) {
ED_screen_full_prevspace(C, file_sa);
@ -2406,7 +2410,7 @@ static int wm_handler_fileselect_do(bContext *C,
}
}
wm_handler_op_context(C, handler, CTX_wm_window(C)->eventstate);
wm_handler_op_context(C, handler, ctx_win->eventstate);
/* needed for UI_popup_menu_reports */

View File

@ -1240,9 +1240,6 @@ static ImBuf *blend_file_thumb(const bContext *C,
/* gets scaled to BLEN_THUMB_SIZE */
Depsgraph *depsgraph = CTX_data_ensure_evaluated_depsgraph(C);
/* Offscreen drawing requires a drawable window context. */
wm_window_make_drawable(wm, CTX_wm_window(C));
if (scene->camera) {
ibuf = ED_view3d_draw_offscreen_imbuf_simple(depsgraph,
scene,