Merge 188631 "Don't throw exceptions when v8 objects/contexts ar..."

chrome/test/data/extensions/api_test/crazy_extension/background.js

 // Copyright 2013 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.
 
 // This test is for bizarre things that extensions probably will do. We just
 // need to ensure that behaviour is predictable.
 
 chrome.test.runTests([
   function accessNonexistentIframe() {
     var iframe = document.createElement("iframe");
@@ skipping 14 matching lines @@
           // Second test is that it does if accessed before removal.
           var iframeChrome = iframe.contentWindow.chrome;
           var iframeChromeWebstore = iframeChrome.webstore;
           chrome.test.assertTrue(typeof(iframeChromeWebstore) == 'object');
           document.body.removeChild(iframe);
           chrome.test.assertTrue(typeof(iframeChrome.webstore) == 'object');
           document.body.appendChild(iframe);
           break;
         }
         case 2: {
-          // Third test is that calling frame-dependent methods doesn't crash
-          // the renderer if the frame doesn't exist anymore.
+          // Third test is that accessing API methods doesn't crash the
+          // renderer if the frame doesn't exist anymore.
+          // TODO(kalman): this fails. Fix.
           var iframeChromeApp = iframe.contentWindow.chrome.app;
           document.body.removeChild(iframe);
-          var exception = null;
-          try {
-            iframeChromeApp.getDetails();
-          } catch (e) {
-            exception = e;
-          }
-          chrome.test.assertEq("Extension view no longer exists", exception);
+          chrome.test.assertEq(undefined, iframeChromeApp.getDetails);
           chrome.test.succeed();
           break;
         }
       }
       iteration++;
     });
     document.body.appendChild(iframe);
   }
 ]);