summaryrefslogtreecommitdiffstats
path: root/chrome/renderer/extensions
Commit message (Collapse)AuthorAgeFilesLines
* Allow platform apps to get writable FileEntry objects from readable ones.benwells@chromium.org2012-06-133-12/+14
| | | | | | | | | | | | | This propts the user to choose a path to save to, using the file entry passed in as a reference of where to save to. BUG=130196 TEST=Manual testing performed Review URL: https://chromiumcodereview.appspot.com/10539101 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@141878 0039d316-1c4b-4281-b951-d872f2087c98
* Implement the activeTab permission, so that extensions with the permissionkalman@chromium.org2012-06-138-17/+168
| | | | | | | | | | | | | that get tab-specific permissions granted (currently, only via clicking a browser action, but this will expand) can executeScript and captureVisibleTab on that tab only, until the next page navigation or refresh. BUG=93903 Review URL: https://chromiumcodereview.appspot.com/10443105 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@141801 0039d316-1c4b-4281-b951-d872f2087c98
* Move all the feature related stuff into extensions/featuresmitchellwrosen@chromium.org2012-06-113-3/+3
| | | | | | | | | | BUG=120090 TEST=manual Review URL: https://chromiumcodereview.appspot.com/10537103 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@141470 0039d316-1c4b-4281-b951-d872f2087c98
* Double dollar ($$) parsing in messages.json brokenmitchellwrosen@chromium.org2012-06-081-6/+2
| | | | | | | | | | | | Fixed the above behavior. $$ is now parsed properly. BUG=127243 TEST=browser_tests --gtest_filter=ExtensionApiTest.I18N Review URL: https://chromiumcodereview.appspot.com/10544066 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@141280 0039d316-1c4b-4281-b951-d872f2087c98
* Change platform app api getDisplayPath to take FileEntry.benwells@chromium.org2012-06-081-0/+2
| | | | | | | | | | | | | | | Previously it has been taking a file system name and relative file. Now the API takes a FileEntry which is much nicer to use, and is transformed into a file system name and relative file path for the implementation via a custom binding. BUG=130452 TEST=The existing automated test has been updated Review URL: https://chromiumcodereview.appspot.com/10544069 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@141186 0039d316-1c4b-4281-b951-d872f2087c98
* Implement extension API chrome.fileSystem.getDisplayPath().benwells@chromium.org2012-06-082-7/+3
| | | | | | | | | | | | | This allows platform apps to get the full path of file entries in isolated file systems for display purposes. BUG=130452 TEST=Test added Review URL: https://chromiumcodereview.appspot.com/10477009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@141119 0039d316-1c4b-4281-b951-d872f2087c98
* Allow updateArgumentsPostValidate to support callbacks and added / removed ↵benwells@chromium.org2012-06-072-0/+2
| | | | | | | | | | | | | | arguments. This also pulls the normalization code out of schema_generated_bindings.js. BUG=None TEST=Existing tests Review URL: https://chromiumcodereview.appspot.com/10535030 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@140980 0039d316-1c4b-4281-b951-d872f2087c98
* Add sandboxed_pages to allow extension/app pages to be served in amihaip@chromium.org2012-06-061-2/+21
| | | | | | | | | | | | sandboxed, unique origin. This allows manifest_version 2 extensions to have pages that are exempt from their Content Security Policy (but these pages can't call extension APIs either). Depends on http://webkit.org/b/88014 Review URL: https://chromiumcodereview.appspot.com/10458063 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@140689 0039d316-1c4b-4281-b951-d872f2087c98
* Coverity NULL_RETURNS issue.scr@chromium.org2012-06-011-1/+4
| | | | | | | | | | | CID=104100 BUG=None TEST= Review URL: https://chromiumcodereview.appspot.com/10442077 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@139980 0039d316-1c4b-4281-b951-d872f2087c98
* Media Gallery: Implement a basic version of GetMediaFileSystems().thestig@chromium.org2012-05-311-8/+36
| | | | | | | | | BUG=none TEST=none Review URL: https://chromiumcodereview.appspot.com/10409084 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@139766 0039d316-1c4b-4281-b951-d872f2087c98
* Allow "null" to mean optional in extension APIs (for realz), but normalise itkalman@chromium.org2012-05-301-1/+7
| | | | | | | | | | | | | | | | | | for browser implementations by stripping it from objects. The following are all now exactly the same, where previously they weren't and would either not work or crash the browser/renderer: chrome.tabs.create({}); chrome.tabs.create({windowId: undefined}); chrome.tabs.create({windowId: null}); BUG=129411 TEST=as in bug Review URL: https://chromiumcodereview.appspot.com/10456010 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@139510 0039d316-1c4b-4281-b951-d872f2087c98
* Make ScriptBadgeExecutor only show extension script badges after they'rekalman@chromium.org2012-05-303-4/+31
| | | | | | | | | | | | | | | | | definitely running, and for frames within pages (i.e. show a badge on foo.com if it has a bar.com iframe with a content script running on it). Achieve this by (1) only showing for executeScript when the response comes with the correct URL, and (2) only showing for content scripts when the renderer informs the browser that the script is running. BUG=127988 Review URL: https://chromiumcodereview.appspot.com/10446054 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@139487 0039d316-1c4b-4281-b951-d872f2087c98
* Pass command line arguments onto platform apps which provide the right intent.benwells@chromium.org2012-05-263-0/+88
| | | | | | | | | | | | | Any command line arguments which are file names are passed through in launchData.intent BUG=None TEST=New tests added Review URL: https://chromiumcodereview.appspot.com/10332071 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@139195 0039d316-1c4b-4281-b951-d872f2087c98
* Allow platform apps to create shell windows with a script connection. jeremya@chromium.org2012-05-253-0/+130
| | | | | | | | | | | | | | | | e.g. chrome.appWindow.create('main.html', {}, function (win) { console.log(win.innerWidth); }); This patch is an intermediate step towards a cleaner approach to creating app windows (described in the review). BUG=123963 TEST=example works Review URL: https://chromiumcodereview.appspot.com/10391199 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@138986 0039d316-1c4b-4281-b951-d872f2087c98
* Revert "Revert 137734 - Select theme resources from ResourceBundle at ↵flackr@chromium.org2012-05-243-5/+9
| | | | | | | | | | | | | | | requested scale factor." Relanding original CL 137734, http://codereview.chromium.org/10387010/. TBR=aa,abodenha,ben,sail,tony BUG=123611 TEST=All try bots pass and aura runs without errors. Review URL: https://chromiumcodereview.appspot.com/10412004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@138769 0039d316-1c4b-4281-b951-d872f2087c98
* Making webRequest.addEventListener internalvabr@chromium.org2012-05-231-0/+2
| | | | | | | | | | | | Changes required to hide addEventListener, moving it from webRequest to webRequestInternal. BUG=115467 TEST=loading extensions from the ticket 115467 and observing the behaviour Review URL: https://chromiumcodereview.appspot.com/10310028 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@138481 0039d316-1c4b-4281-b951-d872f2087c98
* Get rid of the RenderViewType concept in content, since it was only used by ↵jam@chromium.org2012-05-235-18/+18
| | | | | | | | | Chrome. Store the enum value in the WebContents' property bag. BUG=98716 Review URL: https://chromiumcodereview.appspot.com/10409088 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@138434 0039d316-1c4b-4281-b951-d872f2087c98
* Move Extension into extensions namespaceaa@chromium.org2012-05-2117-22/+49
| | | | | | | | | BUG=117262 TBR=aa@chromium.org Review URL: https://chromiumcodereview.appspot.com/10375021 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@138074 0039d316-1c4b-4281-b951-d872f2087c98
* Add schema chrome-extension-resource:// for extension resources. It may be ↵penghuang@chromium.org2012-05-182-2/+31
| | | | | | | | | | | | | | | | | | used for bundle JS libraries in future like Google Analytics JS library. Mark a test case flaky for OSX, and reland it. BUG=122987 TEST=unit_tests --gtest_filter=ExtensionFileUtil.* TEST=browser_tests --gtest_filter=ExtensionResourceRequestPolicyTest* Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=137794 Review URL: https://chromiumcodereview.appspot.com/9909019 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@137918 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 137794 - Add schema chrome-extension-resource:// for extension ↵rsleevi@chromium.org2012-05-182-31/+2
| | | | | | | | | | | | | | | | | resources. It may be used for bundle JS libraries in future like Google Analytics JS library. BUG=122987 TEST=unit_tests --gtest_filter=ExtensionFileUtil.* TEST=browser_tests --gtest_filter=ExtensionResourceRequestPolicyTest* Review URL: https://chromiumcodereview.appspot.com/9909019 TBR=penghuang@chromium.org Review URL: https://chromiumcodereview.appspot.com/10409019 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@137808 0039d316-1c4b-4281-b951-d872f2087c98
* Add schema chrome-extension-resource:// for extension resources. It may be ↵penghuang@chromium.org2012-05-182-2/+31
| | | | | | | | | | | | | | used for bundle JS libraries in future like Google Analytics JS library. BUG=122987 TEST=unit_tests --gtest_filter=ExtensionFileUtil.* TEST=browser_tests --gtest_filter=ExtensionResourceRequestPolicyTest* Review URL: https://chromiumcodereview.appspot.com/9909019 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@137794 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 137734 - Select theme resources from ResourceBundle at requested ↵fischman@chromium.org2012-05-173-9/+5
| | | | | | | | | | | | | | | | | | scale factor. Return the best match for a requested scale factor when fetching raw image data from ResourceBundle. TBR=aa,abodenha BUG=123611 TEST=ResourceBundle.LoadImageResourceBytes Review URL: https://chromiumcodereview.appspot.com/10387010 TBR=flackr@chromium.org git-svn-id: svn://svn.chromium.org/chrome/trunk/src@137738 0039d316-1c4b-4281-b951-d872f2087c98
* Select theme resources from ResourceBundle at requested scale factor.flackr@chromium.org2012-05-173-5/+9
| | | | | | | | | | | | | | Return the best match for a requested scale factor when fetching raw image data from ResourceBundle. TBR=aa,abodenha BUG=123611 TEST=ResourceBundle.LoadImageResourceBytes Review URL: https://chromiumcodereview.appspot.com/10387010 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@137734 0039d316-1c4b-4281-b951-d872f2087c98
* Hack to restrict runtime API to dev channel rather than experimental.mpcomplete@chromium.org2012-05-162-5/+34
| | | | | | | | | BUG=124167 TEST=no Review URL: https://chromiumcodereview.appspot.com/10310182 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@137566 0039d316-1c4b-4281-b951-d872f2087c98
* Support optional arguments in SendRequestNatives::StartRequestbattre@chromium.org2012-05-161-0/+1
| | | | | | | | | | | | | | | Extensions send calls via SendRequestNatives::StartRequest. It is necessary to support undefined v8 values to express optional parameters. This CL exposes the functionality to configure support for Undefined, Date and Regex data types in content::V8ValueConverter and uses it to enable Undefined values in SendRequestNatives::StartRequest. BUG=127533 TEST=error messages on console are gone, see bug report Review URL: https://chromiumcodereview.appspot.com/10387055 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@137452 0039d316-1c4b-4281-b951-d872f2087c98
* Move declarative API into events APIbattre@chromium.org2012-05-164-5/+37
| | | | | | | | | | | | | | | | | | | | The motivation of this CL is to bring the declarative API and the rest of the events mechanism closer together in the API documentation. To achive this, we move the content of declarative.json and everything related to events from extension.json into the events.json API definition. The new "events" namespace is automatically provided to all extensions. This is a second attempt after http://src.chromium.org/viewvc/chrome?view=rev&revision=137388 failed in ExecuteScriptApiTest.ExecuteScriptInFrame. TBR'ing as this is a tiny change from the previous CL. BUG=126497,112155 TEST=no TBR=aa@chromium.org, kalman@chromium.org Review URL: https://chromiumcodereview.appspot.com/10392127 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@137406 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 137388 - Move declarative API into events APIbattre@chromium.org2012-05-164-37/+5
| | | | | | | | | | | | | | | | | | The motivation of this CL is to bring the declarative API and the rest of the events mechanism closer together in the API documentation. To achive this, we move the content of declarative.json and everything related to events from extension.json into the events.json API definition. The new "events" namespace is automatically provided to all extensions. BUG=126497,112155 TEST=no Review URL: https://chromiumcodereview.appspot.com/10392008 TBR=battre@chromium.org Review URL: https://chromiumcodereview.appspot.com/10389168 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@137394 0039d316-1c4b-4281-b951-d872f2087c98
* Move declarative API into events APIbattre@chromium.org2012-05-164-5/+37
| | | | | | | | | | | | | | | The motivation of this CL is to bring the declarative API and the rest of the events mechanism closer together in the API documentation. To achive this, we move the content of declarative.json and everything related to events from extension.json into the events.json API definition. The new "events" namespace is automatically provided to all extensions. BUG=126497,112155 TEST=no Review URL: https://chromiumcodereview.appspot.com/10392008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@137388 0039d316-1c4b-4281-b951-d872f2087c98
* Don't crash the renderer if v8 throws an exception when dispatching anmpcomplete@chromium.org2012-05-141-0/+12
| | | | | | | | | | | | extension port message. It looks like this can happen when V8 hits a stack overflow. BUG=127932 TEST=no Review URL: https://chromiumcodereview.appspot.com/10399012 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@137016 0039d316-1c4b-4281-b951-d872f2087c98
* Change when extensions are allowed to load other extension resources.mihaip@chromium.org2012-05-141-23/+28
| | | | | | | | | | | | | | | | | | | | The previous check was inconsistent (extensions could load other extensions' resources, even if they weren't accessible, except for iframes). The new logic is that extensions can load their own resources (via iframes or otherwise), but can only load resources from other extensions if they're listed in web_accessible_resources. In theory this might break some manifest_version 2 extensions, but in practice we don't think any extensions were relying on this behavior. BUG=127832 R=cdn@chromium.org Review URL: https://chromiumcodereview.appspot.com/10391076 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@136988 0039d316-1c4b-4281-b951-d872f2087c98
* Remove VIEW_TYPE_WEB_CONTENTS and make default view type INVALID.aa@chromium.org2012-05-141-1/+1
| | | | | | | | | | | | | | The view type enum is supposed to represent the 'type of view' in the user sense, so 'tab', 'background page', 'popup', etc. A 'web contents' is not a type of view, it's an implementation detail of many different view types. This was essentially resulting in too many WebContents instances being treated as tabs. Review URL: https://chromiumcodereview.appspot.com/10387110 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@136835 0039d316-1c4b-4281-b951-d872f2087c98
* Initial extension bindings for Media Gallery API.vandebo@chromium.org2012-05-113-0/+97
| | | | | | | | | | BUG=110823 TEST=NONE Review URL: https://chromiumcodereview.appspot.com/10256015 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@136677 0039d316-1c4b-4281-b951-d872f2087c98
* Add unit tests for event.jskoz@chromium.org2012-05-101-0/+89
| | | | | | | | | | BUG=121479 TEST=supplied unit tests Review URL: https://chromiumcodereview.appspot.com/10383035 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@136200 0039d316-1c4b-4281-b951-d872f2087c98
* Allow serialization of ArrayBuffer params in extension/apps API methodsasargent@chromium.org2012-05-091-10/+11
| | | | | | | | | | | | | | | | | | | | | This changes the V8ValueConverter to do ArrayBuffer<->BinaryValue conversions, as well as supporting ArrayBufferView subclasses as request parameters (but not response ones, since that's unnecessary). Also adds an experimental API for testing ArrayBuffers in request/response parameters. (This is re-landing https://chromiumcodereview.appspot.com/10161038, which ran into some test failures fixed by crrev.com/135933) TBR=asargent@chromium.org BUG=122675 TEST=Included browser tests should pass Review URL: https://chromiumcodereview.appspot.com/10377063 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@136114 0039d316-1c4b-4281-b951-d872f2087c98
* Move Declarative Web Request API out of experimental and make it a featurebattre@chromium.org2012-05-091-4/+4
| | | | | | | | | | | | | This CL moves the declarative web request API and the declarative API out of the experimental namespace. Instead the APIs become features that are only enabled on the dev channel. This CL introduces also empty permissions for the APIs, we need to improve this so that host permissions are respected. BUG=112155 TEST=no Review URL: https://chromiumcodereview.appspot.com/10257006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@136006 0039d316-1c4b-4281-b951-d872f2087c98
* Refactor the socket API to remove onEvent callback in socket.create() function.penghuang@chromium.org2012-05-073-59/+0
| | | | | | | | | | BUG=None TEST=browser_tests --gtest_filter=SocketApiTest.* TEST=unit_tests --gtest_filter=SocketTest.* Review URL: http://codereview.chromium.org/10273016 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@135709 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 135602 - Allow serialization of ArrayBuffer params in extension/apps ↵dmazzoni@google.com2012-05-071-11/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | API methods Caused these three tests to time out - apparently only in debug mode, which is perhaps why they were not caught by try runs: INCOMPLETE (5): TtsApiTest.RegisterEngine INCOMPLETE (6): TtsApiTest.EngineError INCOMPLETE (7): TtsApiTest.EngineWordCallbacks Original description: This changes the V8ValueConverter to do ArrayBuffer<->BinaryValue conversions, as well as supporting ArrayBufferView subclasses as request parameters (but not response ones, since that's unnecessary). Also adds an experimental API for testing ArrayBuffers in request/response parameters. This depends on WebKit changes in https://bugs.webkit.org/show_bug.cgi?id=84899 BUG=122675 TEST=Included browser tests should pass Review URL: http://codereview.chromium.org/10161038 TBR=asargent@chromium.org Review URL: https://chromiumcodereview.appspot.com/10388002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@135618 0039d316-1c4b-4281-b951-d872f2087c98
* Allow serialization of ArrayBuffer params in extension/apps API methodsasargent@chromium.org2012-05-061-10/+11
| | | | | | | | | | | | | | | | | | This changes the V8ValueConverter to do ArrayBuffer<->BinaryValue conversions, as well as supporting ArrayBufferView subclasses as request parameters (but not response ones, since that's unnecessary). Also adds an experimental API for testing ArrayBuffers in request/response parameters. This depends on WebKit changes in https://bugs.webkit.org/show_bug.cgi?id=84899 BUG=122675 TEST=Included browser tests should pass Review URL: http://codereview.chromium.org/10161038 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@135602 0039d316-1c4b-4281-b951-d872f2087c98
* Move input.ime back to experimental until privacy review is complete.zork@chromium.org2012-05-041-1/+2
| | | | | | | | BUG=chromium-os:30418 Review URL: http://codereview.chromium.org/10316003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@135338 0039d316-1c4b-4281-b951-d872f2087c98
* Experimental USB APIgdk@chromium.org2012-04-283-0/+52
| | | | | | | | | | | | This change adds a new API for low-level access to USB devices. The change depends on https://chromiumcodereview.appspot.com/10161035/ BUG=none TEST=none Review URL: http://codereview.chromium.org/10224009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@134423 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 134390 - Experimental USB APIjamesr@chromium.org2012-04-271-6/+0
| | | | | | | | | | | | | | | This change adds a new API for low-level access to USB devices. The change depends on https://chromiumcodereview.appspot.com/10161035/ BUG=none TEST=none Review URL: https://chromiumcodereview.appspot.com/10224009 TBR=miket@chromium.org Review URL: https://chromiumcodereview.appspot.com/10137013 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@134392 0039d316-1c4b-4281-b951-d872f2087c98
* Experimental USB APImiket@chromium.org2012-04-271-0/+6
| | | | | | | | | | | | This change adds a new API for low-level access to USB devices. The change depends on https://chromiumcodereview.appspot.com/10161035/ BUG=none TEST=none Review URL: https://chromiumcodereview.appspot.com/10224009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@134390 0039d316-1c4b-4281-b951-d872f2087c98
* Add an API for hosted apps to check their install and running states.jstritar@chromium.org2012-04-242-0/+75
| | | | | | | | | BUG=107216 TEST=ChromeAppAPITest.* Review URL: http://codereview.chromium.org/10174001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@133722 0039d316-1c4b-4281-b951-d872f2087c98
* Back out r132850 from the trunk. It caused crashes, documented at bug 124208.mark@chromium.org2012-04-203-91/+97
| | | | | | | | | | TBR=koz BUG=121479, 124208 TEST=Browse around for a few minutes with an extension such as cfhdojbkjhnklbpkdaibdccddilifddb. Expect no sad tabs. Review URL: https://chromiumcodereview.appspot.com/10171011 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@133253 0039d316-1c4b-4281-b951-d872f2087c98
* Use Value objects instead of JSON strings in extension API response IPCs.asargent@chromium.org2012-04-206-8/+24
| | | | | | | | | | | | | | | | | | When we're sending the result of an extension API method call, we currently convert a Value object into a JSON string in the browser process, and then call JSON.parse in javascript on the renderer side. This CL changes that to pass the Value object in the IPC, and uses V8ValueConverter to turn the Value into the appropriate v8 native object, avoiding the need to parse JSON. This is a first step towards supporting binary data for API request parameters and responses. BUG=122675 TEST=No new functionality ; existing unit and browser tests should still pass. Review URL: https://chromiumcodereview.appspot.com/10041047 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@133133 0039d316-1c4b-4281-b951-d872f2087c98
* Fix bug where transient pages would miss events dispatched while it wasmpcomplete@chromium.org2012-04-194-99/+138
| | | | | | | | | | | | | | | | | unloading. The fix is a 2-parter: Part 1: lazy_background_queue.* now checks if the background page is unloading before allowing a task to run. If it is unloading, we wait until the page unloads, then reload it. Part 2: Part 1 exposed the problem which I had previously outlined in extension_process_manager.cc:403 - after we reload the page, we occasionally received messages from the renderer to decrement the keepalive count for the old page, throwing the count out of balance. The fix is to make sure we decrement keepalive counts only if the host we incremented them for is still active. BUG=123243 TEST=no Review URL: https://chromiumcodereview.appspot.com/10114015 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@133079 0039d316-1c4b-4281-b951-d872f2087c98
* Refactor EventBindings to be a ChromeV8Extension.koz@chromium.org2012-04-183-97/+91
| | | | | | | | | | BUG=121479 TEST=existing tests Review URL: http://codereview.chromium.org/10035033 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@132850 0039d316-1c4b-4281-b951-d872f2087c98
* TabContents -> WebContentsImpl, part 15.avi@chromium.org2012-04-171-1/+1
| | | | | | | | | | | | | Switch VIEW_TYPE_TAB_CONTENTS to VIEW_TYPE_WEB_CONTENTS. Also removed all references in comments to TAB_CONTENTS_WEB. TAB_CONTENTS_WEB was a value of the enum TabContentsType, which was removed in r13730. BUG=105875 TEST=no change Review URL: https://chromiumcodereview.appspot.com/10091006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@132567 0039d316-1c4b-4281-b951-d872f2087c98
* Fix top renderer crash. Extension can go away before Context. Duh.aa@chromium.org2012-04-131-2/+2
| | | | | | | | | BUG=123148 Review URL: http://codereview.chromium.org/10084016 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@132236 0039d316-1c4b-4281-b951-d872f2087c98
* Make lazy field access handle the case where ModuleSystem has been deleted.koz@chromium.org2012-04-131-1/+2
| | | | | | | | | | BUG=122562 TEST= Review URL: http://codereview.chromium.org/10050029 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@132233 0039d316-1c4b-4281-b951-d872f2087c98
generated by cgit v1.1 at 2025-02-10 20:40:55 +0000