Hugh McMaster : conhost: Copy Character Attributes (colors) to the new screen buffer.

[Alexandre Julliard]

Module: wine
Branch: master
Commit: b17eb271120b95e5307549a083080542097f7028
URL:    https://source.winehq.org/git/wine.git/?a=commit;h=b17eb271120b95e5307549a08...

Author: Hugh McMaster 
Date:   Thu Jul 22 23:36:19 2021 +1000

conhost: Copy Character Attributes (colors) to the new screen buffer.

CreateConsoleScreenBuffer() copies the Character Attributes (colors)
to the new screen buffer when called.

Signed-off-by: Hugh McMaster 
Signed-off-by: Jacek Caban 
Signed-off-by: Alexandre Julliard

---

dlls/kernel32/tests/console.c | 6 +++---
 programs/conhost/conhost.c    | 3 ++-
 2 files changed, 5 insertions(+), 4 deletions(-)

diff --git a/dlls/kernel32/tests/console.c b/dlls/kernel32/tests/console.c
index 9fa0ce06854..5f9342ffd44 100644
--- a/dlls/kernel32/tests/console.c
+++ b/dlls/kernel32/tests/console.c
@@ -1047,7 +1047,7 @@ static void test_new_screen_buffer_color_attributes(HANDLE hConOut)
     ok(ret, "GetConsoleScreenBufferInfoEx failed: error %u\n", GetLastError());
     CloseHandle(hConOut2);

-    todo_wine ok(csbi2.wAttributes == orig_attr, "Character Attributes should have been copied: "
+    ok(csbi2.wAttributes == orig_attr, "Character Attributes should have been copied: "
        "got %#x, expected %#x\n", csbi2.wAttributes, orig_attr);
     todo_wine ok(csbi2.wPopupAttributes != orig_popup, "Popup Attributes should not match original value\n");
     todo_wine ok(csbi2.wPopupAttributes == orig_attr, "Popup Attributes should match Character Attributes\n");
@@ -1068,7 +1068,7 @@ static void test_new_screen_buffer_color_attributes(HANDLE hConOut)
     ok(ret, "GetConsoleScreenBufferInfoEx failed: error %u\n", GetLastError());
     CloseHandle(hConOut2);

-    todo_wine ok(csbi2.wAttributes == attr, "Character Attributes should have been copied: "
+    ok(csbi2.wAttributes == attr, "Character Attributes should have been copied: "
        "got %#x, expected %#x\n", csbi2.wAttributes, attr);
     todo_wine ok(csbi2.wPopupAttributes != orig_popup, "Popup Attributes should not match original value\n");
     todo_wine ok(csbi2.wPopupAttributes == attr, "Popup Attributes should match Character Attributes\n");
@@ -1092,7 +1092,7 @@ static void test_new_screen_buffer_color_attributes(HANDLE hConOut)
     ok(ret, "GetConsoleScreenBufferInfoEx failed: error %u\n", GetLastError());
     CloseHandle(hConOut2);

-    todo_wine ok(csbi2.wAttributes == orig_attr, "Character Attributes should have been copied: "
+    ok(csbi2.wAttributes == orig_attr, "Character Attributes should have been copied: "
        "got %#x, expected %#x\n", csbi2.wAttributes, orig_attr);
     todo_wine ok(csbi2.wPopupAttributes != orig_popup, "Popup Attributes should not match original value\n");
     todo_wine ok(csbi2.wPopupAttributes == orig_attr, "Popup Attributes should match Character Attributes\n");
diff --git a/programs/conhost/conhost.c b/programs/conhost/conhost.c
index 7db1261d3be..805df880897 100644
--- a/programs/conhost/conhost.c
+++ b/programs/conhost/conhost.c
@@ -85,7 +85,6 @@ static struct screen_buffer *create_screen_buffer( struct console *console, int
     screen_buffer->cursor_visible = 1;
     screen_buffer->width          = width;
     screen_buffer->height         = height;
-    screen_buffer->attr           = 0x07;
     screen_buffer->popup_attr     = 0xf5;
     screen_buffer->font.weight    = FW_NORMAL;
     screen_buffer->font.pitch_family = FIXED_PITCH | FF_DONTCARE;
@@ -96,6 +95,7 @@ static struct screen_buffer *create_screen_buffer( struct console *console, int
         screen_buffer->max_height = console->active->max_height;
         screen_buffer->win.right  = console->active->win.right  - console->active->win.left;
         screen_buffer->win.bottom = console->active->win.bottom - console->active->win.top;
+        screen_buffer->attr       = console->active->attr;
     }
     else
     {
@@ -103,6 +103,7 @@ static struct screen_buffer *create_screen_buffer( struct console *console, int
         screen_buffer->max_height = height;
         screen_buffer->win.right  = width - 1;
         screen_buffer->win.bottom = height - 1;
+        screen_buffer->attr       = FOREGROUND_BLUE|FOREGROUND_GREEN|FOREGROUND_RED;
     }

if (wine_rb_put( &screen_buffer_map, LongToPtr(id), &screen_buffer->entry ))