classList.contains 本身不会报错，但是有兼容性要求。IE10+。
还有就是你那个判断根本不需要==true

http://caniuse.com/#search=cl...
你可以参考下, 你的浏览器是否在支持列表中.

if(this.classList.contains("on")==true)

在循环里面绑定事件，而且你绑定事件的处理方法使用了循环index的话，你需要在闭包里实现，不然index会变成循环结束时的index。搜索一下闭包。

 for(var i = 0;i<pBox.length;i++){
          pBox[i].addEventListener("mouseover",function(){
                  (function(i){
                      if(pBox[i].classList.contains("on")==true){  //这个地方提示错误。。。。
                          this.classList.remove("on");
                      }else{
                          this.className = "on";
                      }

                  })(i)               
          })
    }