Make Closure Compiler understand assertNotReached

third_party/closure_compiler/runner/src/org/chromium/closure/compiler/ChromeCodingConvention.java

Index: third_party/closure_compiler/runner/src/org/chromium/closure/compiler/ChromeCodingConvention.java
diff --git a/third_party/closure_compiler/runner/src/org/chromium/closure/compiler/ChromeCodingConvention.java b/third_party/closure_compiler/runner/src/org/chromium/closure/compiler/ChromeCodingConvention.java
index 396ae828c130e8ba4bbc017235d44fbb6b5013c8..7eadc53fbbf043b396dfd90e6114bda8d31c3f1f 100644
--- a/third_party/closure_compiler/runner/src/org/chromium/closure/compiler/ChromeCodingConvention.java
+++ b/third_party/closure_compiler/runner/src/org/chromium/closure/compiler/ChromeCodingConvention.java
@@ -67,4 +67,21 @@ public class ChromeCodingConvention extends CodingConventions.Proxy {
       new AssertInstanceofSpec("cr.ui.decorate")
     );
   }
+
+  // TODO(dbeam): combine this with ClosureCodingConvention?
+  @Override
+  public boolean isFunctionCallThatAlwaysThrows(Node n) {
+    if (n.isExprResult()) {
+      if (!n.getFirstChild().isCall()) {
+        return false;
+      }
+    } else if (!n.isCall()) {
+      return false;
+    }
+    if (n.isExprResult()) {
+      n = n.getFirstChild();
+    }
+    // n is a call
+    return n.getFirstChild().matchesQualifiedName("assertNotReached");
+  }
 }