| Index: chrome/test/data/extensions/api_test/crazy_extension/background.js
|
| diff --git a/chrome/test/data/extensions/api_test/crazy_extension/background.js b/chrome/test/data/extensions/api_test/crazy_extension/background.js
|
| index ab7b3016badeefc12c62735d2da46ee0f2462b7e..c41519ae388b8cd357f689eaa7b6a50826d0cd45 100644
|
| --- a/chrome/test/data/extensions/api_test/crazy_extension/background.js
|
| +++ b/chrome/test/data/extensions/api_test/crazy_extension/background.js
|
| @@ -32,17 +32,12 @@ chrome.test.runTests([
|
| 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;
|
| }
|
|
|