@@ -175,15 +175,13 @@ func handleConn(conn net.Conn, b *browser.Browser, sessionName string, confirmSt
175175 }
176176 return true , nil
177177
178- // Phase 2: Navigation
179178 case "back" :
180179 err = b .Back ()
181180 case "forward" :
182181 err = b .Forward ()
183182 case "reload" :
184183 err = b .Reload ()
185184
186- // Phase 2: Information retrieval
187185 case "get.title" :
188186 var text string
189187 text , err = b .GetTitle ()
@@ -267,7 +265,6 @@ func handleConn(conn net.Conn, b *browser.Browser, sessionName string, confirmSt
267265 resp .Text = text
268266 }
269267
270- // Phase 2: State queries
271268 case "is.visible" :
272269 if req .ID <= 0 {
273270 err = fmt .Errorf ("id must be positive" )
@@ -299,7 +296,6 @@ func handleConn(conn net.Conn, b *browser.Browser, sessionName string, confirmSt
299296 resp .BoolResult = & v
300297 }
301298
302- // Phase 2: Wait commands
303299 case "wait" :
304300 err = b .Wait (req .WaitDuration )
305301 case "wait.selector" :
@@ -329,7 +325,6 @@ func handleConn(conn net.Conn, b *browser.Browser, sessionName string, confirmSt
329325 }
330326 err = b .WaitFunc (req .Expression )
331327
332- // Phase 3: Screenshot
333328 case "screenshot" :
334329 if req .FilePath == "" {
335330 err = fmt .Errorf ("file path is required" )
@@ -349,7 +344,6 @@ func handleConn(conn net.Conn, b *browser.Browser, sessionName string, confirmSt
349344 })
350345 }
351346
352- // Phase 3: PDF
353347 case "pdf" :
354348 if req .FilePath == "" {
355349 err = fmt .Errorf ("file path is required" )
@@ -360,7 +354,6 @@ func handleConn(conn net.Conn, b *browser.Browser, sessionName string, confirmSt
360354 PrintBG : req .PDFArgs .PrintBG ,
361355 })
362356
363- // Phase 3: Eval
364357 case "eval" :
365358 if req .Expression == "" {
366359 err = fmt .Errorf ("expression is required" )
@@ -372,7 +365,6 @@ func handleConn(conn net.Conn, b *browser.Browser, sessionName string, confirmSt
372365 resp .Text = result
373366 }
374367
375- // Phase 3: Find
376368 case "find.role" :
377369 var result * browser.FindResults
378370 var findOpts []browser.FindOption
@@ -432,7 +424,6 @@ func handleConn(conn net.Conn, b *browser.Browser, sessionName string, confirmSt
432424 resp .Text = result .Text
433425 }
434426
435- // Phase 4: Drag
436427 case "drag" :
437428 if req .ID <= 0 {
438429 err = fmt .Errorf ("source id must be positive" )
@@ -444,7 +435,6 @@ func handleConn(conn net.Conn, b *browser.Browser, sessionName string, confirmSt
444435 }
445436 err = b .Drag (req .ID , req .DstID )
446437
447- // Phase 4: Upload
448438 case "upload" :
449439 if req .ID <= 0 {
450440 err = fmt .Errorf ("id must be positive" )
@@ -456,7 +446,6 @@ func handleConn(conn net.Conn, b *browser.Browser, sessionName string, confirmSt
456446 }
457447 err = b .Upload (req .ID , req .Files ... )
458448
459- // Phase 4: Download
460449 case "download" :
461450 if req .ID <= 0 {
462451 err = fmt .Errorf ("id must be positive" )
@@ -472,7 +461,6 @@ func handleConn(conn net.Conn, b *browser.Browser, sessionName string, confirmSt
472461 resp .FilePath = path
473462 }
474463
475- // Phase 4: Mouse operations
476464 case "mouse.move" :
477465 err = b .MouseMove (req .X , req .Y )
478466 case "mouse.down" :
@@ -496,7 +484,6 @@ func handleConn(conn net.Conn, b *browser.Browser, sessionName string, confirmSt
496484 }
497485 err = b .MouseClick (req .X , req .Y , opts )
498486
499- // Phase 5: Tab management
500487 case "tab.list" :
501488 var tabs []browser.TabInfo
502489 tabs , err = b .TabList ()
@@ -510,7 +497,6 @@ func handleConn(conn net.Conn, b *browser.Browser, sessionName string, confirmSt
510497 case "tab.switch" :
511498 err = b .TabSwitch (req .TabIndex )
512499
513- // Phase 5: Network
514500 case "network.route" :
515501 action := browser .RouteBlock
516502 if req .RouteAction == "continue" {
@@ -530,7 +516,6 @@ func handleConn(conn net.Conn, b *browser.Browser, sessionName string, confirmSt
530516 case "network.clear-requests" :
531517 b .NetworkClearRequests ()
532518
533- // Phase 5: Cookies
534519 case "cookies.get" :
535520 var cookies []browser.CookieInfo
536521 cookies , err = b .CookiesGet ()
@@ -544,7 +529,6 @@ func handleConn(conn net.Conn, b *browser.Browser, sessionName string, confirmSt
544529 case "cookies.clear" :
545530 err = b .CookiesClear ()
546531
547- // Phase 5: Storage
548532 case "storage.get" :
549533 var val string
550534 val , err = b .StorageGet (req .StorageType , req .StorageKey )
@@ -564,7 +548,6 @@ func handleConn(conn net.Conn, b *browser.Browser, sessionName string, confirmSt
564548 resp .StorageItems = items
565549 }
566550
567- // Phase 6: Settings
568551 case "set.viewport" :
569552 if req .Width <= 0 || req .Height <= 0 {
570553 err = fmt .Errorf ("width and height must be positive" )
@@ -608,7 +591,6 @@ func handleConn(conn net.Conn, b *browser.Browser, sessionName string, confirmSt
608591 }
609592 err = b .SetColorScheme (req .ColorScheme )
610593
611- // Phase 6: Console/Debug
612594 case "console.start" :
613595 err = b .ConsoleStart ()
614596 case "console.messages" :
@@ -640,7 +622,6 @@ func handleConn(conn net.Conn, b *browser.Browser, sessionName string, confirmSt
640622 case "devtools" :
641623 err = b .OpenDevTools ()
642624
643- // Phase 6: Clipboard
644625 case "clipboard.read" :
645626 var text string
646627 text , err = b .ClipboardRead ()
@@ -650,7 +631,6 @@ func handleConn(conn net.Conn, b *browser.Browser, sessionName string, confirmSt
650631 case "clipboard.write" :
651632 err = b .ClipboardWrite (req .ClipboardText )
652633
653- // Phase 7: Diff
654634 case "diff.snapshot" :
655635 var result * browser.DiffSnapshotResult
656636 result , err = b .DiffSnapshot (browser.DiffSnapshotOptions {
@@ -696,7 +676,6 @@ func handleConn(conn net.Conn, b *browser.Browser, sessionName string, confirmSt
696676 }
697677 }
698678
699- // Phase 7: Trace
700679 case "trace.start" :
701680 if req .Categories != "" {
702681 err = b .TraceStart (req .Categories )
@@ -706,13 +685,11 @@ func handleConn(conn net.Conn, b *browser.Browser, sessionName string, confirmSt
706685 case "trace.stop" :
707686 err = b .TraceStop (req .OutputFile )
708687
709- // Phase 7: Profiler
710688 case "profiler.start" :
711689 err = b .ProfilerStart ()
712690 case "profiler.stop" :
713691 err = b .ProfilerStop (req .OutputFile )
714692
715- // Phase 7: Record
716693 case "record.start" :
717694 err = b .RecordStart (req .OutputFile )
718695 case "record.stop" :
@@ -722,7 +699,6 @@ func handleConn(conn net.Conn, b *browser.Browser, sessionName string, confirmSt
722699 resp .IntResult = frameCount
723700 }
724701
725- // Phase 7: State export/import
726702 case "state.export" :
727703 if req .StatePath == "" {
728704 err = fmt .Errorf ("state file path is required" )
@@ -736,7 +712,6 @@ func handleConn(conn net.Conn, b *browser.Browser, sessionName string, confirmSt
736712 }
737713 err = b .ImportState (req .StatePath )
738714
739- // Phase 8: Missing commands
740715 case "keyboard.inserttext" :
741716 if req .Text == "" {
742717 err = fmt .Errorf ("text is required" )
0 commit comments