Add warning when watchAsset attempts to reuse token symbol

app/_locales/en/messages.json

@@ -746,6 +746,9 @@
   "knownTokenWarning": {
     "message": "This action will edit tokens that are already listed in your wallet, which can be used to phish you. Only approve if you are certain that you mean to change what these tokens represent."
   },
+  "reusedTokenNameWarning": {
+    "message": "A token here reuses a symbol from another token you watch, this can be confusing or deceptive."
+  },
   "invalidAddressRecipientNotEthNetwork": {
     "message": "Not ETH network, set to lowercase"
   },

ui/app/pages/confirm-add-suggested-token/confirm-add-suggested-token.component.js

@@ -37,6 +37,7 @@ export default class ConfirmAddSuggestedToken extends Component {
     const pendingTokenKey = Object.keys(pendingTokens)[0]
     const pendingToken = pendingTokens[pendingTokenKey]
     const hasTokenDuplicates = this.checkTokenDuplicates(pendingTokens, tokens);
+    const reusesName = this.checkNameReuse(pendingTokens, tokens);
 
     return (
       <div className="page-container">
@@ -52,6 +53,11 @@ export default class ConfirmAddSuggestedToken extends Component {
                 { this.context.t('knownTokenWarning') }
               </div> : null
           }
+          { reusesName ?
+              <div className="warning">
+                { this.context.t('reusedTokenNameWarning') }
+              </div> : null
+          }
         </div>
         <div className="page-container__content">
           <div className="confirm-add-token">
@@ -136,4 +142,21 @@ export default class ConfirmAddSuggestedToken extends Component {
     return dupes.length > 0;
   }
 
+  /**
+   * Returns true if any pendingTokens both:
+   * - Share a symbol with an existing `tokens` member.
+   * - Does not share an address with that same `tokens` member.
+   * This should be flagged as possibly deceptive or confusing.
+   */
+  checkNameReuse(pendingTokens, tokens) {
+    const duplicates = Object.keys(pendingTokens)
+      .map((addr) => pendingTokens[addr])
+      .filter((token) => {
+        const dupes = tokens.filter(old => old.symbol === token.symbol)
+          .filter(old => old.address !== token.address);
+        return dupes.length > 0;
+      })
+    return duplicates.length > 0;
+  }
+
 }