Revert "Fix crash on windows viewer when starting in vulkan."

This reverts commit fcd6816.

Reason for revert: going to do this a different way

Original change's description:
> Fix crash on windows viewer when starting in vulkan.
>
> setRequestedDisplayParams is called when setting up viewer before we've
> inited the Window. On windows this tries to attach the Window for a given
> backend. However, we haven't set fBackend yet. Later on we will directly
> call attach.
>
> Change-Id: I4bd6586478f2b040e5913314c4e47e92fc893a60
> Reviewed-on: https://skia-review.googlesource.com/c/skia/+/344756
> Reviewed-by: Brian Osman <[email protected]>
> Commit-Queue: Greg Daniel <[email protected]>

[email protected],[email protected]

Change-Id: Ie4d33b557868e4906df9b7a3988072fdc79a19c7
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/344897
Reviewed-by: Greg Daniel <[email protected]>
Commit-Queue: Greg Daniel <[email protected]>

Author: egdaniel

tools/sk_app/Window.h

@@ -52,7 +52,6 @@ class Window {
     virtual bool scaleContentToFit() const { return false; }
 
     enum BackendType {
-        kUnknown_BackendType,
 #ifdef SK_GL
         kNativeGL_BackendType,
 #endif

tools/sk_app/android/Window_android.cpp

@@ -66,8 +66,6 @@ void Window_android::initDisplay(ANativeWindow* window) {
                     window_context_factory::MakeVulkanForAndroid(window, fRequestedDisplayParams);
             break;
 #endif
-        case kUnknown_BackendType:
-            SkUNREACHABLE;
     }
     this->onBackendCreated();
 }

tools/sk_app/ios/Window_ios.mm

@@ -100,8 +100,6 @@ - (WindowViewController*)initWithWindow:(sk_app::Window_ios*)initWindow;
         case kRaster_BackendType:
             fWindowContext = MakeRasterForIOS(info, fRequestedDisplayParams);
             break;
-        case kUnknown_BackendType:
-            SkUNREACHABLE;
     }
     this->onBackendCreated();
 

tools/sk_app/mac/Window_mac.mm

@@ -148,8 +148,6 @@ - (MainView*)initWithWindow:(sk_app::Window_mac*)initWindow;
         case kRaster_BackendType:
             fWindowContext = MakeRasterForMac(info, fRequestedDisplayParams);
             break;
-        case kUnknown_BackendType:
-            SkUNREACHABLE;
     }
     this->onBackendCreated();
 

tools/sk_app/unix/Window_unix.cpp

@@ -418,8 +418,6 @@ bool Window_unix::attach(BackendType attachType) {
             fWindowContext =
                     window_context_factory::MakeRasterForXlib(winInfo, fRequestedDisplayParams);
             break;
-        case kUnknown_BackendType:
-            SkUNREACHABLE;
     }
     this->onBackendCreated();
 

tools/sk_app/win/Window_win.cpp

@@ -383,8 +383,6 @@ bool Window_win::attach(BackendType attachType) {
                 window_context_factory::MakeD3D12ForWin(fHWnd, fRequestedDisplayParams);
             break;
 #endif
-        case kUnknown_BackendType:
-            SkUNREACHABLE;
     }
     this->onBackendCreated();
 
@@ -405,9 +403,7 @@ void Window_win::setRequestedDisplayParams(const DisplayParams& params, bool all
         fWindowContext = nullptr;
         this->closeWindow();
         this->init(fHInstance);
-        if (fBackend != kUnknown_BackendType) {
-            this->attach(fBackend);
-        }
+        this->attach(fBackend);
     }
 
     INHERITED::setRequestedDisplayParams(params, allowReattach);

tools/sk_app/win/Window_win.h

@@ -35,7 +35,7 @@ class Window_win : public Window {
 
     HINSTANCE fHInstance;
     HWND      fHWnd;
-    BackendType fBackend = kUnknown_BackendType;
+    BackendType fBackend;
 
     using INHERITED = Window;
 };