Comment #4 on issue 26128 by [email protected]: Add stub functions to  
chrome.* APIs for content scripts
http://code.google.com/p/chromium/issues/detail?id=26128

The following revision refers to this bug:
     http://src.chromium.org/viewvc/chrome?view=rev&revision=31958

------------------------------------------------------------------------
r31958 | [email protected] | 2009-11-13 14:22:55 -0800 (Fri, 13 Nov  
2009) | 16 lines
Changed paths:
    A  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/extensions/stubs_apitest.cc
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/chrome.gyp?r1=31958&r2=31957
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/common/extensions/api/extension_api.json?r1=31958&r2=31957
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/renderer/resources/extension_process_bindings.js?r1=31958&r2=31957
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/renderer/resources/renderer_extension_bindings.js?r1=31958&r2=31957
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/renderer/user_script_slave.cc?r1=31958&r2=31957
    A  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/test/data/extensions/api_test/stubs
    A  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/test/data/extensions/api_test/stubs/background.html
    A  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/test/data/extensions/api_test/stubs/content_script.js
    A  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/test/data/extensions/api_test/stubs/manifest.json

Add stub functions for chrome.* APIs in content scripts.

Most of the extensions API is not supported in content scripts for security
purposes. Instead the typical pattern is to use messaging between content
scripts and a background page which executes them on behalf of the script.
This is not immediately obvious to many extension developers, so this
change adds an error message to help them better understand what's going on.


BUG=26128
TEST=Try writing a content script that uses some of the extensions API stuff
in chrome.tabs, chrome.windows, etc. You should see errors in the console
telling you that it isn't supported in content scripts.


Review URL: http://codereview.chromium.org/366024
------------------------------------------------------------------------


--
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

Reply via email to