Reviewed by Sam.
authorkmccullo <kmccullo@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 23 Oct 2007 23:11:39 +0000 (23:11 +0000)
committerkmccullo <kmccullo@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 23 Oct 2007 23:11:39 +0000 (23:11 +0000)
        - After talking with Steve I now see that the WebKit server must be
        running for DCOM to create the distributed objects, which makes sense
        and currently WebKit only allows one instance to be running at a time
        which avoids accidentally connecting to the wrong server
        - In light of this I have removed the code for the attach box and
        NotificationServer and known server names, since they are all extranious
        now.

        * Drosera/win/Drosera.cpp: Removed notification classes and known server
        names.
        (_tWinMain): Uses init instead of initUI.
        (droseraWndProc): No longer creats an attach dialog box.
        (Drosera::Drosera): Does the OleInitialize so COM is ready to go and
        it's not manditory to call init before doing COM stuff.
        (Drosera::init): calls initUI and will call attach when the
        functionality exists.
        (Drosera::initUI): Has changed very little.
        (Drosera::attach): Changed the signature to reflect that we no longer
        need the dictionary of known server names.
        * Drosera/win/Drosera.h: Removed notification classes and known server
        names.  Also renamed and moved some functions.
        * Drosera/win/Drosera.vcproj/Drosera.rc: Removed the Attach box.
        * Drosera/win/Drosera.vcproj/Drosera.vcproj: Moved resource.h from the
        headers to the resources folder.
        * Drosera/win/resource.h: Removed the Attach box.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@26927 268f45cc-cd09-0410-ab3c-d52691b4dbfc

WebKitTools/ChangeLog
WebKitTools/Drosera/win/Drosera.cpp
WebKitTools/Drosera/win/Drosera.h
WebKitTools/Drosera/win/Drosera.vcproj/Drosera.rc
WebKitTools/Drosera/win/Drosera.vcproj/Drosera.vcproj
WebKitTools/Drosera/win/resource.h

index d910c64ca42012ff47413b6a246c30a96a75b233..1f531eb035b40527f45a52625cb64a1c95c405a5 100644 (file)
@@ -1,3 +1,33 @@
+2007-10-23  Kevin McCullough  <kmccullough@apple.com>
+
+        Reviewed by Sam.
+
+        - After talking with Steve I now see that the WebKit server must be
+        running for DCOM to create the distributed objects, which makes sense
+        and currently WebKit only allows one instance to be running at a time
+        which avoids accidentally connecting to the wrong server
+        - In light of this I have removed the code for the attach box and
+        NotificationServer and known server names, since they are all extranious
+        now.
+
+        * Drosera/win/Drosera.cpp: Removed notification classes and known server
+        names.
+        (_tWinMain): Uses init instead of initUI.
+        (droseraWndProc): No longer creats an attach dialog box.
+        (Drosera::Drosera): Does the OleInitialize so COM is ready to go and
+        it's not manditory to call init before doing COM stuff.
+        (Drosera::init): calls initUI and will call attach when the
+        functionality exists. 
+        (Drosera::initUI): Has changed very little.
+        (Drosera::attach): Changed the signature to reflect that we no longer
+        need the dictionary of known server names.
+        * Drosera/win/Drosera.h: Removed notification classes and known server
+        names.  Also renamed and moved some functions.
+        * Drosera/win/Drosera.vcproj/Drosera.rc: Removed the Attach box.
+        * Drosera/win/Drosera.vcproj/Drosera.vcproj: Moved resource.h from the
+        headers to the resources folder. 
+        * Drosera/win/resource.h: Removed the Attach box.
+
 2007-10-22  Adam Roben  <aroben@apple.com>
 
         Windows build fix
index b34721f3742948b6beef2a18310597112e0747c5..f0f618ecafbd3fddff6dc5cce57d96a9acd6a199 100644 (file)
@@ -40,7 +40,6 @@
 #include <WebKit/IWebMutableURLRequest.h>
 #include <WebKit/IWebView.h>
 #include <WebKit/WebKit.h>
-#include <windowsx.h>
 
 const unsigned MAX_LOADSTRING = 100;
 
@@ -55,7 +54,6 @@ extern "C" BOOL InitializeCoreGraphics();
 
 ATOM registerDroseraClass(HINSTANCE hInstance);
 LRESULT CALLBACK droseraWndProc(HWND hWnd, UINT message, WPARAM wParam, LPARAM lParam);
-INT_PTR CALLBACK attachWndProc(HWND hDlg, UINT message, WPARAM wParam, LPARAM lParam);
 INT_PTR CALLBACK aboutWndProc(HWND hDlg, UINT message, WPARAM wParam, LPARAM lParam);
 
 HINSTANCE Drosera::getInst() { return hInst; }
@@ -76,7 +74,7 @@ int APIENTRY _tWinMain(HINSTANCE hInstance,
 
     Drosera drosera;
 
-    HRESULT ret = drosera.initUI(hInstance, nCmdShow);
+    HRESULT ret = drosera.init(hInstance, nCmdShow);
     if (FAILED(ret))
         return ret;
 
@@ -134,9 +132,6 @@ LRESULT CALLBACK droseraWndProc(HWND hWnd, UINT message, WPARAM wParam, LPARAM l
                 case ID_HELP_ABOUT:
                     DialogBox(Drosera::getInst(), MAKEINTRESOURCE(IDD_ABOUTBOX), hWnd, ::aboutWndProc);
                     break;
-                case ID_FILE_ATTACH:
-                    DialogBox(Drosera::getInst(), MAKEINTRESOURCE(IDD_ATTACH), hWnd, ::attachWndProc);
-                    break;
                 case ID_FILE_EXIT:
                     DestroyWindow(hWnd);
                     break;
@@ -180,33 +175,25 @@ INT_PTR CALLBACK aboutWndProc(HWND hDlg, UINT message, WPARAM wParam, LPARAM lPa
     return (INT_PTR)FALSE;
 }
 
-// Message handler for Attach box.
-INT_PTR CALLBACK attachWndProc(HWND hDlg, UINT message, WPARAM wParam, LPARAM lParam)
-{
-    UNREFERENCED_PARAMETER(lParam);
-    switch (message) {
-        case WM_INITDIALOG:
-            return (INT_PTR)TRUE;
-
-        case WM_COMMAND:
-            if (LOWORD(wParam) == IDOK || LOWORD(wParam) == IDCANCEL) {
-                EndDialog(hDlg, LOWORD(wParam));
-                return (INT_PTR)TRUE;
-            }
-            break;
-    }
-    return (INT_PTR)FALSE;
-}
-
 ////////////////// End Setup Windows Specific Interface //////////////////
 
 Drosera::Drosera()
     : m_hWnd(0)
     , m_debuggerClient(new DebuggerClient())
-    , m_knownServerNames(new ServerDictionary())
 {
+    OleInitialize(0);
 }
 
+HRESULT Drosera::init(HINSTANCE hInstance, int nCmdShow)
+{
+    HRESULT ret = initUI(hInstance, nCmdShow);
+    if (FAILED(ret))
+        return ret;
+
+    return ret;
+}
+
+
 HRESULT Drosera::initUI(HINSTANCE hInstance, int nCmdShow)
 {
     // Initialize global strings
@@ -228,10 +215,6 @@ HRESULT Drosera::initUI(HINSTANCE hInstance, int nCmdShow)
     if (FAILED(ret))
         return ret;
 
-    ret = OleInitialize(0);
-    if (FAILED(ret))
-        return ret;
-
     ret = CoCreateInstance(CLSID_WebView, 0, CLSCTX_ALL, IID_IWebView, (void**)&m_webView);
     if (FAILED(ret))
         return ret;
@@ -267,11 +250,9 @@ HRESULT Drosera::initUI(HINSTANCE hInstance, int nCmdShow)
     ShowWindow(m_hWnd, nCmdShow);
     UpdateWindow(m_hWnd);
 
-    return 0;
+    return ret;
 }
 
-
-
 LRESULT Drosera::onSize(WPARAM, LPARAM)
 {
     if (!m_webViewPrivate)
@@ -294,29 +275,12 @@ bool Drosera::webViewLoaded() const
     return m_debuggerClient->webViewLoaded();
 }
 
-// FIXME: The below functionality cannot be implemented until the Notification and WebScriptDebugServer is implmented on Windows
-
-void Drosera::applicationDidFinishLaunching()
-{
-    // Adding functions to be associated with notifications
-}
-
 // Server Detection Callbacks
 
-void Drosera::serverLoaded()
-{
-}
-
-void Drosera::serverUnloaded()
-{
-}
-
-HRESULT Drosera::attach(int sender)
+HRESULT Drosera::attach(const std::wstring& serverName)
 {
     // Get selected server
-    unsigned int row = sender;
-    std::wstring key = m_knownServerNames->get(row);
-    m_debuggerClient->initWithServerName(key);
+    m_debuggerClient->initWithServerName(serverName);
 
     HRESULT ret = m_webView->setFrameLoadDelegate(m_debuggerClient.get());
     if (FAILED(ret))
@@ -351,4 +315,3 @@ HRESULT Drosera::attach(int sender)
 
     return ret;
 }
-
index 09815685f362bce1fd1dbbab48820ba75be193de..b61eb0ae9cf198c436b3e0ff5765966af2a4c577 100644 (file)
 
 class DebuggerClient;
 
-typedef HashMap<unsigned, std::wstring> ServerDictionary;
-
 class Drosera {
 public:
+    Drosera();
+
     static HINSTANCE getInst();
     static void setInst(HINSTANCE);
 
-    Drosera();
-    HRESULT initUI(HINSTANCE hInstance, int nCmdShow);
+    HRESULT init(HINSTANCE hInstance, int nCmdShow);
     LRESULT onSize(WPARAM, LPARAM);
 
-    bool webViewLoaded() const;
-    void applicationDidFinishLaunching();
-
-    void serverLoaded();
-    void serverUnloaded();
-    HRESULT attach(int sender);
+    HRESULT attach(const std::wstring& serverName);
 
-    ServerDictionary* knownServers() const { return m_knownServerNames.get(); }
+    bool webViewLoaded() const;
 
 private:
+    HRESULT initUI(HINSTANCE hInstance, int nCmdShow);
+    HRESULT initServer();
+
     HWND m_hWnd;
 
     COMPtr<IWebView> m_webView;
     COMPtr<IWebViewPrivate> m_webViewPrivate;
 
     OwnPtr<DebuggerClient> m_debuggerClient;
-    OwnPtr<ServerDictionary> m_knownServerNames;
 };
 
 #endif //Drosera_H
index 0885f47c9a4a59d11921ab6a86bb8c07be8bafef..2e16fa397424db1505a7cf7a3ea57d85a9091eaf 100755 (executable)
@@ -42,8 +42,6 @@ IDC_DROSERA MENU
 BEGIN\r
     POPUP "&File"\r
     BEGIN\r
-        MENUITEM "Attach",                      ID_FILE_ATTACH\r
-        MENUITEM SEPARATOR\r
         MENUITEM "Close Current File",          ID_FILE_CLOSECURRENTFILE\r
         MENUITEM "Exit",                        ID_FILE_EXIT\r
     END\r
@@ -100,17 +98,6 @@ BEGIN
     CTEXT           "Drosera Version 1.0",IDC_STATIC,0,37,146,8,SS_NOPREFIX\r
 END\r
 \r
-IDD_ATTACH DIALOGEX 0, 0, 194, 132\r
-STYLE DS_SETFONT | DS_MODALFRAME | DS_FIXEDSYS | DS_CENTER | WS_POPUP | WS_CAPTION | WS_SYSMENU\r
-EXSTYLE WS_EX_TOOLWINDOW\r
-CAPTION "Dialog"\r
-FONT 8, "MS Shell Dlg", 400, 0, 0x1\r
-BEGIN\r
-    DEFPUSHBUTTON   "Attach",IDAttach,136,111,50,14\r
-    PUSHBUTTON      "Cancel",IDCANCEL,85,111,50,14\r
-    LISTBOX         IDC_LIST1,7,7,179,98,LBS_SORT | LBS_NOINTEGRALHEIGHT | WS_VSCROLL | WS_HSCROLL | WS_TABSTOP\r
-END\r
-\r
 \r
 #ifdef APSTUDIO_INVOKED\r
 /////////////////////////////////////////////////////////////////////////////\r
@@ -140,25 +127,6 @@ END
 #endif    // APSTUDIO_INVOKED\r
 \r
 \r
-/////////////////////////////////////////////////////////////////////////////\r
-//\r
-// DESIGNINFO\r
-//\r
-\r
-#ifdef APSTUDIO_INVOKED\r
-GUIDELINES DESIGNINFO \r
-BEGIN\r
-    IDD_ATTACH, DIALOG\r
-    BEGIN\r
-        LEFTMARGIN, 7\r
-        RIGHTMARGIN, 186\r
-        TOPMARGIN, 7\r
-        BOTTOMMARGIN, 125\r
-    END\r
-END\r
-#endif    // APSTUDIO_INVOKED\r
-\r
-\r
 /////////////////////////////////////////////////////////////////////////////\r
 //\r
 // String Table\r
index af204d40c1f3baddf599fa9990ba192b8206020d..6e38fa3d5607d52a46d01845fc1df6b9e030aafd 100755 (executable)
                                RelativePath="..\HelperFunctions.h"\r
                                >\r
                        </File>\r
-                       <File\r
-                               RelativePath="..\resource.h"\r
-                               >\r
-                       </File>\r
                        <File\r
                                RelativePath="..\ServerConnection.h"\r
                                >\r
                                        />\r
                                </FileConfiguration>\r
                        </File>\r
+                       <File\r
+                               RelativePath="..\resource.h"\r
+                               >\r
+                       </File>\r
                        <File\r
                                RelativePath="..\..\Images\small.ico"\r
                                >\r
index adea1e07db0f337027a0021a0368ab18da0e4fd4..4ba014e3502f338a41cd338ae0851d2244dbd118 100644 (file)
@@ -8,8 +8,6 @@
 #define IDI_DROSERA                     107
 #define IDI_SMALL                       108
 #define IDC_DROSERA                     109
-#define IDD_ATTACH                      129
-#define IDAttach                        1000
 #define IDC_LIST1                       1001
 #define ID_DEBUG_SHOWCONSOLE            32771
 #define ID_DEBUG_CONTINUE               32772
@@ -29,7 +27,6 @@
 #define ID_FIND_USESELECTIONFORFIND     32787
 #define ID_FIND_JUMPTOSELECTION         32788
 #define ID_EDIT_SPECIALCHARACTERS       32789
-#define ID_FILE_ATTACH                  32790
 #define ID_FILE_EXIT                    32791
 #define ID_WINDOW                       32793
 #define ID_HELP_ABOUT                   32794