Comment #2 on issue 26126 by [email protected]: Change the default location of content script injection http://code.google.com/p/chromium/issues/detail?id=26126
The following revision refers to this bug:
http://src.chromium.org/viewvc/chrome?view=rev&revision=30784
------------------------------------------------------------------------
r30784 | [email protected] | 2009-11-02 17:29:39 -0800 (Mon, 02 Nov 2009) |
10 lines
Changed paths:
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/extensions/user_script_master.cc?r1=30784&r2=30783
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/chrome.gyp?r1=30784&r2=30783
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/common/extensions/docs/content_scripts.html?r1=30784&r2=30783
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/common/extensions/docs/static/content_scripts.html?r1=30784&r2=30783
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/common/extensions/extension.cc?r1=30784&r2=30783
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/common/extensions/extension_constants.cc?r1=30784&r2=30783
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/common/extensions/extension_constants.h?r1=30784&r2=30783
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/common/extensions/user_script.h?r1=30784&r2=30783
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/common/extensions/user_script_unittest.cc?r1=30784&r2=30783
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/renderer/navigation_state.h?r1=30784&r2=30783
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/renderer/render_view.cc?r1=30784&r2=30783
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/renderer/render_view.h?r1=30784&r2=30783
A
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/renderer/user_script_idle_scheduler.cc
A
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/renderer/user_script_idle_scheduler.h
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/test/data/extensions/good/Extensions/bjafgdebaacbbbecmhlhpofkepfkgcpa/1.0/page.js?r1=30784&r2=30783
Add new user script injection point "document_idle" and make it the default.
Semantically, document-idle means "when the DOM is ready and layout has
been idle for awhile", or more loosely, "as soon as we get around to it".
Right now this uses a simple heuristic. It injects scripts 200ms after
DOMContentLoaded, or immediately after onload, whichever happens first.
BUG=26126
TEST=Manual. Extensions with content scripts should work.
Review URL: http://codereview.chromium.org/339064
------------------------------------------------------------------------
--
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
-~----------~----~----~----~------~----~------~--~---
