Remove _onStore() function

This function isn't useful anymore because store verification happens
after the 'ready' event.

lib/torrent.js

@@ -534,6 +534,8 @@ Torrent.prototype._onMetadata = function (metadata) {
     self._onWireWithMetadata(wire)
   })
 
+  self.emit('metadata')
+
   self._debug('verifying existing torrent data')
   if (self._fileModtimes && self._store === FSChunkStore) {
     // don't verify if the files haven't been modified since we last checked
@@ -550,6 +552,9 @@ Torrent.prototype._onMetadata = function (metadata) {
         for (var index = 0; index < self.pieces.length; index++) {
           self._markInStore(index)
         }
+        self._checkDone()
+
+        self.updateSelections()
       } else {
         self._verifyPieces()
       }
@@ -558,8 +563,8 @@ Torrent.prototype._onMetadata = function (metadata) {
     self._verifyPieces()
   }
 
-  self.emit('metadata')
-  self._onStore()
+  self.ready = true
+  self.emit('ready')
 }
 
 /*
@@ -655,24 +660,6 @@ Torrent.prototype._markInStore = function (index) {
   this.bitfield.set(index, true)
 }
 
-/**
- * Called when the metadata, listening server, and underlying chunk store is initialized.
- */
-Torrent.prototype._onStore = function () {
-  var self = this
-  if (self.destroyed) return
-  self._debug('on store')
-
-  self.ready = true
-  self.emit('ready')
-
-  // Files may start out done if the file was already in the store
-  self._checkDone()
-
-  // In case any selections were made before torrent was ready
-  self._updateSelections()
-}
-
 Torrent.prototype.destroy = function (cb) {
   var self = this
   self._destroy(null, cb)