Comment #11 on issue 31170 by [email protected]: eBay extension
browser action popup crashes browser
http://code.google.com/p/chromium/issues/detail?id=31170
The following revision refers to this bug:
http://src.chromium.org/viewvc/chrome?view=rev&revision=35479
------------------------------------------------------------------------
r35479 | [email protected] | 2010-01-04 13:27:59 -0800 (Mon, 04 Jan
2010) | 5 lines
Changed paths:
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/extensions/extension_host.cc?r1=35479&r2=35478
Fixes bug where recreating the document element via altering |
documentElement.innerHTML| would cause
ExtensionHost::DocumentAvailableInMainFrame to be called more than once for
a single RenderViewHost. Since setup is performed within that function that
does not need to be done more than once (and cannot be since it registers
for notifications from all sources), we check to see if setup has been
performed and bail if so.
BUG=31170
TEST=Install the eBay extension (id khhckppjhonfmcpegdjdibmngahahhck ) and
click on the browser action button in Debug mode. It should not hit the
DCHECK from registering redundant notifications.
Review URL: http://codereview.chromium.org/519036
------------------------------------------------------------------------
--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings
--
Automated mail from issue updates at http://crbug.com/
Subscription options: http://groups.google.com/group/chromium-bugs
