Reland of Add GN build rules to allow java_assertion_enabler to enable Java asserts.

content/public/android/java/src/org/chromium/content/browser/webcontents/WebContentsObserverProxy.java

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 package org.chromium.content.browser.webcontents;
 
 import org.chromium.base.ObserverList;
 import org.chromium.base.ObserverList.RewindableIterator;
 import org.chromium.base.ThreadUtils;
 import org.chromium.base.annotations.CalledByNative;
@@ skipping 230 matching lines @@
     public void destroy() {
         // Super destruction semantics (removing the observer from the
         // Java-based WebContents) are quite different, so we explicitly avoid
         // calling it here.
         ThreadUtils.assertOnUiThread();
         for (mObserversIterator.rewind(); mObserversIterator.hasNext();) {
             mObserversIterator.next().destroy();
         }
         // All observer destroy() implementations should result in their removal
         // from the proxy.
-        assert mObservers.isEmpty();
+        // TODO(crbug.com/667437)
+        // assert mObservers.isEmpty();

[boliu, 2016/11/23 17:20:22]

also is this being fixed?

[agrieve, 2016/11/23 17:55:39]

Looks like there's action on the bug.

         mObservers.clear();
 
         if (mNativeWebContentsObserverProxy != 0) {
             nativeDestroy(mNativeWebContentsObserverProxy);
             mNativeWebContentsObserverProxy = 0;
         }
     }
 
     private native long nativeInit(WebContentsImpl webContents);
     private native void nativeDestroy(long nativeWebContentsObserverProxy);
 }