Changeset 231964 in webkit

Timestamp:

May 18, 2018 10:36:26 AM (6 years ago)

Author:

Fujii Hironori

Message:

[Win][MiniBrowser] Create MainWindow even in transparent mode
​https://bugs.webkit.org/show_bug.cgi?id=185597

Reviewed by Per Arne Vollan.

The current implementations doesn't create the MainWindow in
transparent mode. The screenshot is in Bug 58300.

There were three problems:

• Can't control the WebView. ie. going backward and forward.
• Can't quit the program.
• It makes implementation complicated.

• MiniBrowser/win/Common.cpp (resizeSubViews): Resize sub views

even in the transparent mode.

• MiniBrowser/win/WinMain.cpp:

(wWinMain): Create the main window even in the transparent mode.

Location:

trunk/Tools

Files:

• ChangeLog (modified) (1 diff)
• MiniBrowser/win/Common.cpp (modified) (2 diffs)
• MiniBrowser/win/WinMain.cpp (modified) (1 diff)

trunk/Tools/ChangeLog

@@ +1 @@
+2018-05-18  Fujii Hironori  <Hironori.Fujii@sony.com>
+
+        [Win][MiniBrowser] Create MainWindow even in transparent mode
+        https://bugs.webkit.org/show_bug.cgi?id=185597
+
+        Reviewed by Per Arne Vollan.
+
+        The current implementations doesn't create the MainWindow in
+        transparent mode. The screenshot is in Bug 58300.
+
+        There were three problems:
+        - Can't control the WebView. ie. going backward and forward.
+        - Can't quit the program.
+        - It makes implementation complicated.
+
+        * MiniBrowser/win/Common.cpp (resizeSubViews): Resize sub views
+        even in the transparent mode.
+        * MiniBrowser/win/WinMain.cpp:
+        (wWinMain): Create the main window even in the transparent mode.
+
 2018-05-18  Antoine Quint  <graouts@apple.com>
 

trunk/Tools/MiniBrowser/win/Common.cpp

@@ -110 +110 @@
 static void resizeSubViews()
 {
-    if (gMiniBrowser->usesLayeredWebView() || !gViewWindow)
-        return;
-
-    float scaleFactor = WebCore::deviceScaleFactorForWindow(gViewWindow);
+    float scaleFactor = WebCore::deviceScaleFactorForWindow(hMainWnd);
 
     RECT rcClient;
@@ -124 +121 @@
     MoveWindow(hForwardButtonWnd, width, 0, width, height, TRUE);
     MoveWindow(hURLBarWnd, width * 2, 0, rcClient.right, height, TRUE);
+
+    ::SendMessage(hURLBarWnd, static_cast<UINT>(WM_SETFONT), reinterpret_cast<WPARAM>(gMiniBrowser->urlBarFont()), TRUE);
+
+    if (gMiniBrowser->usesLayeredWebView() || !gViewWindow)
+        return;
     MoveWindow(gViewWindow, 0, height, rcClient.right, rcClient.bottom - height, TRUE);
-
-    ::SendMessage(hURLBarWnd, static_cast<UINT>(WM_SETFONT), reinterpret_cast<WPARAM>(gMiniBrowser->urlBarFont()), TRUE);
 }
 

trunk/Tools/MiniBrowser/win/WinMain.cpp

@@ -75 +75 @@
     float scaleFactor = WebCore::deviceScaleFactorForWindow(nullptr);
 
-    if (usesLayeredWebView) {
-        hURLBarWnd = CreateWindow(L"EDIT", L"Type URL Here",
-            WS_OVERLAPPEDWINDOW | WS_VISIBLE | WS_BORDER | ES_LEFT | ES_AUTOVSCROLL, 
-            scaleFactor * s_windowPosition.x, scaleFactor * (s_windowPosition.y + s_windowSize.cy), scaleFactor * s_windowSize.cx, scaleFactor * URLBAR_HEIGHT,
-            0, 0, hInst, 0);
-    } else {
-        hMainWnd = CreateWindow(szWindowClass, szTitle, WS_OVERLAPPEDWINDOW,
-            CW_USEDEFAULT, 0, CW_USEDEFAULT, 0, 0, 0, hInst, 0);
+    hMainWnd = CreateWindow(szWindowClass, szTitle, WS_OVERLAPPEDWINDOW,
+        CW_USEDEFAULT, 0, CW_USEDEFAULT, 0, 0, 0, hInst, 0);
 
-        if (!hMainWnd)
-            return FALSE;
+    if (!hMainWnd)
+        return FALSE;
 
-        hBackButtonWnd = CreateWindow(L"BUTTON", L"<", WS_CHILD | WS_VISIBLE  | BS_TEXT, 0, 0, 0, 0, hMainWnd, 0, hInst, 0);
-        hForwardButtonWnd = CreateWindow(L"BUTTON", L">", WS_CHILD | WS_VISIBLE | BS_TEXT, scaleFactor * CONTROLBUTTON_WIDTH, 0, 0, 0, hMainWnd, 0, hInst, 0);
-        hURLBarWnd = CreateWindow(L"EDIT", 0, WS_CHILD | WS_VISIBLE | WS_BORDER | ES_LEFT | ES_AUTOVSCROLL, scaleFactor * CONTROLBUTTON_WIDTH * 2, 0, 0, 0, hMainWnd, 0, hInst, 0);
+    hBackButtonWnd = CreateWindow(L"BUTTON", L"<", WS_CHILD | WS_VISIBLE  | BS_TEXT, 0, 0, 0, 0, hMainWnd, 0, hInst, 0);
+    hForwardButtonWnd = CreateWindow(L"BUTTON", L">", WS_CHILD | WS_VISIBLE | BS_TEXT, scaleFactor * CONTROLBUTTON_WIDTH, 0, 0, 0, hMainWnd, 0, hInst, 0);
+    hURLBarWnd = CreateWindow(L"EDIT", 0, WS_CHILD | WS_VISIBLE | WS_BORDER | ES_LEFT | ES_AUTOVSCROLL, scaleFactor * CONTROLBUTTON_WIDTH * 2, 0, 0, 0, hMainWnd, 0, hInst, 0);
 
-        ShowWindow(hMainWnd, nCmdShow);
-        UpdateWindow(hMainWnd);
-    }
+    ShowWindow(hMainWnd, nCmdShow);
+    UpdateWindow(hMainWnd);
 
     DefEditProc = reinterpret_cast<WNDPROC>(GetWindowLongPtr(hURLBarWnd, GWLP_WNDPROC));