Reviewed by Sam.
authorkmccullo <kmccullo@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 17 Oct 2007 00:57:12 +0000 (00:57 +0000)
committerkmccullo <kmccullo@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 17 Oct 2007 00:57:12 +0000 (00:57 +0000)
        - Minor mac improvements based on observations I made while implementing the
        Windows code.

        * Drosera/mac/DebuggerApplication.mm: Attach does not need to create the server
        then hand it to the Client who will retain it, it should just be the client who
        creates and owns the server.
        (-[DebuggerApplication attach:]):
        * Drosera/mac/DebuggerClient.h: Can now be created with just a server name.
        * Drosera/mac/DebuggerClient.mm: Creates the server.
        (-[DebuggerClient initWithServerName:]):
        * Drosera/mac/ServerConnection.mm: Does not need to include 2 header files, but
        can instead forward declare what it needs.

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

WebKitTools/ChangeLog
WebKitTools/Drosera/mac/DebuggerApplication.mm
WebKitTools/Drosera/mac/DebuggerClient.h
WebKitTools/Drosera/mac/DebuggerClient.mm
WebKitTools/Drosera/mac/ServerConnection.mm

index b0aab075c4cb8d44b4333eb1a7e4c8dd9624187a..fc04fe9471253a77b1775654297535fa1925b50b 100644 (file)
@@ -1,3 +1,20 @@
+2007-10-16  Kevin McCullough  <kmccullough@apple.com>
+
+        Reviewed by Sam.
+
+        - Minor mac improvements based on observations I made while implementing the
+        Windows code.
+
+        * Drosera/mac/DebuggerApplication.mm: Attach does not need to create the server
+        then hand it to the Client who will retain it, it should just be the client who
+        creates and owns the server.
+        (-[DebuggerApplication attach:]):
+        * Drosera/mac/DebuggerClient.h: Can now be created with just a server name.
+        * Drosera/mac/DebuggerClient.mm: Creates the server.
+        (-[DebuggerClient initWithServerName:]):
+        * Drosera/mac/ServerConnection.mm: Does not need to include 2 header files, but
+        can instead forward declare what it needs.
+
 2007-10-16  Darin Adler  <darin@apple.com>
 
         Reviewed by Geoff Garen.
index 2ce22b45885fcdb682a79c23ef6032c3442f0992..35cfa2448e90f66c1747a7bb686f91d52933b18c 100644 (file)
     unsigned int row = [[attachTable selectedRowIndexes] firstIndex];
     NSString *key = [[knownServerNames allKeys] objectAtIndex:row];
 
-    ServerConnection *server = [[ServerConnection alloc] initWithServerName:key];
-    DebuggerClient *document = [[DebuggerClient alloc] initWithServerConnection:server];
-    [server release];
+    DebuggerClient *document = [[DebuggerClient alloc] initWithServerName:key];
     [document showWindow:sender];
 }
 
index 93cc2c633a36ce131e35d92e0900eb3d3aaaea09..9b1095d0e6ce8182a8ae789417521563e2fb53db 100644 (file)
  * THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
-@class WebView;
-@class ServerConnection;
-
 class DebuggerDocument;
 
+@class ServerConnection;
+@class WebView;
+
 @interface DebuggerClient : NSWindowController
 {
     IBOutlet WebView *webView;
@@ -42,7 +42,7 @@ class DebuggerDocument;
 
 + (void)log:(NSString *)msg;
 
-- (id)initWithServerConnection:(ServerConnection *)serverConn;
+- (id)initWithServerName:(NSString *)serverConn;
 - (IBAction)pause:(id)sender;
 - (IBAction)resume:(id)sender;
 - (IBAction)stepInto:(id)sender;
index f99821542bfadadf4be52210ef6b5fe8c35e26bb..5606029c1e7744fa51750741bf95485f7c687107 100644 (file)
@@ -60,10 +60,10 @@ static NSString *DebuggerStepOutToolbarItem = @"DebuggerStepOutToolbarItem";
 
 #pragma mark -
 
-- (id)initWithServerConnection:(ServerConnection *)serverConn;
+- (id)initWithServerName:(NSString *)serverName;
 {
     if ((self = [super init])) {
-        server = [serverConn retain];
+        server = [[ServerConnection alloc] initWithServerName:serverName];
         debuggerDocument = new DebuggerDocument(server);
     }
 
index 02d2a6a2e6a535e668fa54a0208f529fd0696e09..6814aedd9a5f2bad990dde54819f21f0d39103c2 100644 (file)
@@ -30,8 +30,6 @@
 #import "config.h"
 #import "ServerConnection.h"
 
-#import "DebuggerApplication.h"
-#import "DebuggerClient.h"
 #import "DebuggerDocument.h"
 
 #import <JavaScriptCore/JSContextRef.h>
@@ -39,6 +37,8 @@
 #import <JavaScriptCore/JSStringRefCF.h>
 #import <JavaScriptCore/RetainPtr.h>
 
+@class DebuggerApplication;
+
 @implementation ServerConnection
 
 #pragma mark -