avoid unnecessary api calls

atom/browser/api/atom_api_window.cc

@@ -125,11 +125,11 @@ void Window::OnWindowRestore() {
 }
 
 void Window::OnWindowResize() {
-  Emit("resize", GetBounds().size());
+  Emit("resize");
 }
 
 void Window::OnWindowMove() {
-  Emit("move", window_->GetPosition());
+  Emit("move");
 }
 
 void Window::OnWindowEnterFullScreen() {

docs/api/browser-window.md

@@ -173,20 +173,10 @@ Emitted when window is restored from minimized state.
 
 ### Event: 'resize'
 
-* `event` Event
-* `value` Object
-  * `width` Integer
-  * `height` Integer
-
 Emitted when window is resized.
 
 ### Event: 'move'
 
-* `event` Event
-* `value` Object
-  * `x` Integer
-  * `y` Integer
-
 Emitted when window is moved.
 
 ### Event: 'enter-full-screen'

spec/api-browser-window-spec.coffee

@@ -92,18 +92,20 @@ describe 'browser-window module', ->
   describe 'BrowserWindow.setSize(width, height)', ->
     it 'sets the window size', (done) ->
       size = [20, 400]
-      w.on 'resize', (e, value) ->
+      w.on 'resize', ->
-        assert.equal value.width, size[0]
+        newSize = w.getSize()
-        assert.equal value.height, size[1]
+        assert.equal newSize[0], size[0]
+        assert.equal newSize[1], size[1]
         done()
       w.setSize size[0], size[1]
 
   describe 'BrowserWindow.setPosition(x, y)', ->
     it 'sets the window position', (done) ->
       pos = [10, 10]
-      w.on 'move', (e, value) ->
+      w.on 'move', ->
-        assert.equal value.x, pos[0]
+        newPos = w.getPosition()
-        assert.equal value.y, pos[1]
+        assert.equal newPos[0], pos[0]
+        assert.equal newPos[1], pos[1]
         done()
       w.setPosition pos[0], pos[1]