From c4ed5ece4d3c56ac0fe5a909bf5b151005548309 Mon Sep 17 00:00:00 2001
From: Jackson Tian <shyvo1987@gmail.com>
Date: Mon, 8 Feb 2016 21:12:09 +0800
Subject: [PATCH] tools: enable no-proto rule for linter

Enable `no-proto` in `.eslintrc`.

Use `Object.setPrototypeOf()` and `Object.getPrototypeOf()`
instead of.

PR-URL: https://github.com/nodejs/node/pull/5140
Reviewed-By: Sakthipriyan Vairamani <thechargingvolcano@gmail.com>
Reviewed-By: Roman Reiss <me@silverwind.io>
Reviewed-By: James M Snell <jasnell@gmail.com>
---
 .eslintrc                                | 2 ++
 test/common.js                           | 2 +-
 test/parallel/test-buffer-arraybuffer.js | 4 ++--
 test/parallel/test-buffer-fakes.js       | 4 ++--
 test/parallel/test-buffer-inheritance.js | 5 +++--
 test/parallel/test-child-process-env.js  | 4 ++--
 6 files changed, 12 insertions(+), 9 deletions(-)

diff --git a/.eslintrc b/.eslintrc
index a03bb13ecaf773..6fc8883b41b5c8 100644
--- a/.eslintrc
+++ b/.eslintrc
@@ -17,6 +17,8 @@ ecmaFeatures:
 rules:
   # Possible Errors
   # list: https://github.com/eslint/eslint/tree/master/docs/rules#possible-errors
+  ## Disallow Use of __proto__
+  no-proto: 2
   ## disallow control characters in regular expressions
   no-control-regex: 2
   ## check debugger sentence
diff --git a/test/common.js b/test/common.js
index 43f4362b45cbfe..152e5e5c6f3e45 100644
--- a/test/common.js
+++ b/test/common.js
@@ -190,7 +190,7 @@ exports.hasIPv6 = Object.keys(ifaces).some(function(name) {
 
 function protoCtrChain(o) {
   var result = [];
-  for (; o; o = o.__proto__) { result.push(o.constructor); }
+  for (; o; o = Object.getPrototypeOf(o)) { result.push(o.constructor); }
   return result.join();
 }
 
diff --git a/test/parallel/test-buffer-arraybuffer.js b/test/parallel/test-buffer-arraybuffer.js
index 522d97ece196c1..56811a950e63f0 100644
--- a/test/parallel/test-buffer-arraybuffer.js
+++ b/test/parallel/test-buffer-arraybuffer.js
@@ -40,8 +40,8 @@ assert.equal(dv.getFloat64(8, true), 3.1415);
 
 assert.throws(function() {
   function AB() { }
-  AB.__proto__ = ArrayBuffer;
-  AB.prototype.__proto__ = ArrayBuffer.prototype;
+  Object.setPrototypeOf(AB, ArrayBuffer);
+  Object.setPrototypeOf(AB.prototype, ArrayBuffer.prototype);
   new Buffer(new AB());
 }, TypeError);
 
diff --git a/test/parallel/test-buffer-fakes.js b/test/parallel/test-buffer-fakes.js
index 1a1d9c0c494b84..ca268fa265fb0b 100644
--- a/test/parallel/test-buffer-fakes.js
+++ b/test/parallel/test-buffer-fakes.js
@@ -5,8 +5,8 @@ const assert = require('assert');
 const Buffer = require('buffer').Buffer;
 
 function FakeBuffer() { }
-FakeBuffer.__proto__ = Buffer;
-FakeBuffer.prototype.__proto__ = Buffer.prototype;
+Object.setPrototypeOf(FakeBuffer, Buffer);
+Object.setPrototypeOf(FakeBuffer.prototype, Buffer.prototype);
 
 const fb = new FakeBuffer();
 
diff --git a/test/parallel/test-buffer-inheritance.js b/test/parallel/test-buffer-inheritance.js
index ea61e50bf5f04a..be845c66ccf508 100644
--- a/test/parallel/test-buffer-inheritance.js
+++ b/test/parallel/test-buffer-inheritance.js
@@ -24,8 +24,9 @@ const vals = [new T(4), T(4)];
 
 vals.forEach(function(t) {
   assert.equal(t.constructor, T);
-  assert.equal(t.__proto__, T.prototype);
-  assert.equal(t.__proto__.__proto__, Buffer.prototype);
+  assert.equal(Object.getPrototypeOf(t), T.prototype);
+  assert.equal(Object.getPrototypeOf(Object.getPrototypeOf(t)),
+    Buffer.prototype);
 
   t.fill(5);
   let cntr = 0;
diff --git a/test/parallel/test-child-process-env.js b/test/parallel/test-child-process-env.js
index 89880af1cc5c0d..2ebc653e9a067d 100644
--- a/test/parallel/test-child-process-env.js
+++ b/test/parallel/test-child-process-env.js
@@ -7,9 +7,9 @@ var spawn = require('child_process').spawn;
 var env = {
   'HELLO': 'WORLD'
 };
-env.__proto__ = {
+Object.setPrototypeOf(env, {
   'FOO': 'BAR'
-};
+});
 
 var child;
 if (common.isWindows) {