summaryrefslogtreecommitdiffstats
path: root/chrome/chrome_renderer.gypi
Commit message (Collapse)AuthorAgeFilesLines
* Correct path to JS for MSVC 2010 following http://crrev.com/122953rsleevi@chromium.org2012-02-221-1/+1
| | | | | | | | | | BUG=none TEST=none TBR=koz Review URL: http://codereview.chromium.org/9428041 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@123117 0039d316-1c4b-4281-b951-d872f2087c98
* Introduce a module system for JS based on a require() method with similarkoz@chromium.org2012-02-221-0/+5
| | | | | | | | | | | | | | | semantics to node.js'. This is intended to replace the current ChromeV8Extension-based system of injecting JS into an extension's renderer. BUG=104100 TEST=unit tests provided Review URL: http://codereview.chromium.org/9402003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@122953 0039d316-1c4b-4281-b951-d872f2087c98
* Extensions: convert all extension.* and i18n.* functions/events that arekalman@chromium.org2012-02-221-0/+2
| | | | | | | | | | | | | | hand-written in miscellaneous_bindings.js to use the schema_generated_bindings.js infrastructure. BUG=104099 TEST=browser_tests Review URL: http://codereview.chromium.org/9406034 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@122942 0039d316-1c4b-4281-b951-d872f2087c98
* First check in for password generation.gcasto@chromium.org2012-02-221-0/+2
| | | | | | | | | | | | Create a PasswordGenerationManager that will live in the renderer and control this feature. BUG=114092 TEST=Ran password_generation_manager_browsertest Review URL: http://codereview.chromium.org/9309064 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@122920 0039d316-1c4b-4281-b951-d872f2087c98
* Stop using weak v8 references to keep track of inline install callbacks.mihaip@chromium.org2012-02-111-5/+3
| | | | | | | | | | | | | | | | | | | | | | The callback function(s) need to be kept alive until installation suceeds or fails. Switch to using an explicit map (implemented in the JS bindings) instead. This was a problem from the start, but wasn't uncovered in tests until r120621 landed, since (presumably) it keeps the renderer process alive longer, thus making it more likely to do a GC between chrome.webstore.install being called and the callbacks being invoked. To make implementing all of this easier, change the bindings to use ChromeV8Extension and ChromeV8ExtensionHandler, so that we can use chromeHidden to reach the response function. BUG=113024 R=aa@chromium.org Review URL: http://codereview.chromium.org/9361017 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@121606 0039d316-1c4b-4281-b951-d872f2087c98
* Bindings layer for declarative events APIbattre@chromium.org2012-02-081-0/+1
| | | | | | | | | BUG=112155 TEST=no Review URL: https://chromiumcodereview.appspot.com/9192029 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@120962 0039d316-1c4b-4281-b951-d872f2087c98
* Move handling of debug urls like chrome://crash, chrome://gpuclean to ↵jam@chromium.org2012-02-081-2/+0
| | | | | | | | | | | content. These are for testing the content layer, so they belong there. This allows us to hide GpuProcessHostUIShim from chrome. I cleaned up the about_handler.* files which were overly complicated. I also was able to remove a method from ContentBrowserClient. BUG=98716 Review URL: https://chromiumcodereview.appspot.com/9349010 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@120891 0039d316-1c4b-4281-b951-d872f2087c98
* Move DOMAutomationController code from chrome to content. This is needed by ↵jam@chromium.org2012-02-031-2/+0
| | | | | | | | | | | interstitial pages which are in content, and also browser tests in content. In followup changes, I'll make interstitial pages not have to derive from a concrete class, and instead implement some sort of delegate interface like we did with WebUI. BUG=98716 Review URL: https://chromiumcodereview.appspot.com/9310094 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@120441 0039d316-1c4b-4281-b951-d872f2087c98
* Merge different plug-in placeholders into one class.bauerb@chromium.org2012-02-031-4/+0
| | | | | | | | | | | | | | This allows us to combine functionality from different plug-in placeholders, i.e. installing *and* loading a plug-in. Also refactor ChromeContentRendererClient::CreatePlugin to be (hopefully) simpler: It's just a big switch statement now that configures the plug-in placeholder. BUG=92796 TEST=none Review URL: http://codereview.chromium.org/9309037 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@120310 0039d316-1c4b-4281-b951-d872f2087c98
* Take the IME Extension API out of experimental:zork@chromium.org2012-01-271-1/+1
| | | | | | | | | | | | | | | - s/experimental.input.ime/input.ime/g (file contents and names). - s/EXPERIMENTAL_INPUT_IME/INPUT_IME/g (ditto). - re-sort anything out of order as a result. - regenerate docs. BUG=chromium-os:23646 TEST=browser_tests --gtest_filter=ExtensionApiTest.InputImeApiBasic Review URL: http://codereview.chromium.org/9169081 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@119396 0039d316-1c4b-4281-b951-d872f2087c98
* Fix name of types_custom_bindings.js in gyp file.scottmg@google.com2012-01-251-1/+1
| | | | | | | | | | Avoids warning on VS project generation. See http://codereview.chromium.org/9179006 TBR=aa,koz Review URL: https://chromiumcodereview.appspot.com/9117011 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@119132 0039d316-1c4b-4281-b951-d872f2087c98
* Split logic for defining custom types in schema_generated_bindings.js into ↵koz@chromium.org2012-01-181-0/+1
| | | | | | | | | | | | separate files. BUG=80310 TEST= Review URL: http://codereview.chromium.org/9179006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@118070 0039d316-1c4b-4281-b951-d872f2087c98
* Extension cleanup: move devtools custom bindings out of ↵kalman@chromium.org2012-01-181-0/+1
| | | | | | | | | | | | schema_generated_bindings.js BUG=80310 TEST=browser_tests (win, mac, linux) Review URL: http://codereview.chromium.org/9212029 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@118019 0039d316-1c4b-4281-b951-d872f2087c98
* Extensions cleanup: move the chromePrivate, extension, fileBrowserPrivate, andkalman@chromium.org2012-01-171-3/+15
| | | | | | | | | | | | | | | | pageCapture APIs out of schema_generated_bindings.js. Also, since there would have been naming confusion otherwise, convert the fileBrowserHandler API stuff into the "custom bindings" infrastructure. BUG=80310 TEST=browser_tests (mac, win, linux, linux_chromeos) Review URL: http://codereview.chromium.org/9228002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@117864 0039d316-1c4b-4281-b951-d872f2087c98
* Extensions cleanup: pull the browserAction and pageAction API customisationskalman@chromium.org2012-01-141-0/+2
| | | | | | | | | | | | | out of schema_generated_bindings. BUG=80310 TEST= Review URL: http://codereview.chromium.org/9203003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@117765 0039d316-1c4b-4281-b951-d872f2087c98
* Extensions cleanup: move tabs API cusomisations out of ↵kalman@chromium.org2012-01-131-0/+2
| | | | | | | | | | | | schema_generated_bindings. BUG=80310 TEST=browser_tests --gtest_filter=*Extension* Review URL: http://codereview.chromium.org/9121028 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@117617 0039d316-1c4b-4281-b951-d872f2087c98
* Extensions cleanup: move a bunch of API customisations fromkalman@chromium.org2012-01-121-3/+19
| | | | | | | | | | | | | | | | | | | schema_generated_bindings into their own files: * experimental.input.ime * experimental.socket * omnibox * pageActions * ttsEngine * tts * webRequest * windows BUG=80310 TEST= Review URL: http://codereview.chromium.org/9171001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@117434 0039d316-1c4b-4281-b951-d872f2087c98
* Re-land r117016: Split out the contextMenus schema bindings from ↵kalman@chromium.org2012-01-121-1/+6
| | | | | | | | | | | | | | | schema_generated_bindings.js. This is safe to land again after r117398. TBR=aa@chromium.org BUG=80310 TEST= Review URL: http://codereview.chromium.org/9190036 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@117405 0039d316-1c4b-4281-b951-d872f2087c98
* Revert r117016 "Split out the contextMenus schema bindings from ↵maruel@chromium.org2012-01-101-6/+1
| | | | | | | | | | | | | | schema_generated_bindings.js." browser_tests: IsolatedAppTest.ProcessOverflow started to fail on Win7 Tests (dbg)(6) TBR=kalman@chromium.org BUG= TEST= Review URL: http://codereview.chromium.org/9168011 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@117032 0039d316-1c4b-4281-b951-d872f2087c98
* Split out the contextMenus schema bindings from schema_generated_bindings.js.kalman@chromium.org2012-01-101-1/+6
| | | | | | | | | BUG=80310 TEST= Review URL: http://codereview.chromium.org/9019006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@117016 0039d316-1c4b-4281-b951-d872f2087c98
* Defer loading of audio/video tags while prerendering.shishir@chromium.org2011-12-091-0/+2
| | | | | | | | | | BUG=98690 TEST= Review URL: http://codereview.chromium.org/8095007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@113742 0039d316-1c4b-4281-b951-d872f2087c98
* Implement SetVisuallyDeemphasized in ChromeRenderViewObserver.xiyuan@chromium.org2011-12-021-0/+2
| | | | | | | | | | | | | | | Implement SetVisuallyDeemphasized in ChromeRenderViewObserver - Add a ChromeViewMsg_SetVisuallyDeemphasized and use it to toggle background dimming on web contents via ChromeRenderViewObserver; - Derive WebViewColorOverlay from WebKit::WebPageOverlay and add to web view when need to dimming the background; - Remove RWHV SetVisuallyDeemphasized since it's no longer used; BUG=103386 TEST=Verify fix for 103386, dimming should show up even when accelerated compositing is on. Review URL: http://codereview.chromium.org/8730001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@112787 0039d316-1c4b-4281-b951-d872f2087c98
* Collect profiler stats from all browser child processes.rtenneti@chromium.org2011-12-011-2/+0
| | | | | | | | | | | R=jar Bug=104770 TEST=about:profiler after lauching a plug-in or GPU should show tracking data for plugin-gpu processes Review URL: http://codereview.chromium.org/8588023 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@112571 0039d316-1c4b-4281-b951-d872f2087c98
* Clean up plug-in placeholders:bauerb@chromium.org2011-11-251-4/+9
| | | | | | | | | | | | | | * Add separate class for missing plug-in placeholder, and factor out a common base class * Move plug-in related classes to plugins/ subdirectory * Move custom menu command IDs to a separate header file. TBR=darin@chromium.org BUG=62079 TEST=none Review URL: http://codereview.chromium.org/8461011 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@111589 0039d316-1c4b-4281-b951-d872f2087c98
* CrOs: Remove 2 static initializers and 2 exit time destructors.thakis@chromium.org2011-11-211-0/+1
| | | | | | | | | | | | Turn on -Wexit-time-destructors for (most of) the chrome target. BUG=101600,94925 TEST=none TBR=stevenjb Review URL: http://codereview.chromium.org/8547005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@110954 0039d316-1c4b-4281-b951-d872f2087c98
* Moving media-related files from webkit/glue/ to webkit/media/.scherkus@chromium.org2011-11-171-0/+1
| | | | | | | | Files under webkit/media/ are now under their very own webkit_media namespace. Review URL: http://codereview.chromium.org/8570010 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@110438 0039d316-1c4b-4281-b951-d872f2087c98
* Enable extension APIs for content scripts.aa@chromium.org2011-11-121-6/+6
| | | | | | | | | | | This only enables extension functions. I will do events in a separate change. BUG=81760,80308 Review URL: http://codereview.chromium.org/8540012 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@109812 0039d316-1c4b-4281-b951-d872f2087c98
* Changes to upload tracked_objects data from all rendererrtenneti@chromium.org2011-11-051-0/+2
| | | | | | | | | | | | | | | | | | | | | | | | | processes to browser process. Added the following messages to render_messages. ChromeViewMsg_GetRendererTrackedData - Asks the renderer to send back tracked data (ThreadData in tracked_objects). ChromeViewMsg_SetTrackingStatus - Asks the renderer to set the tracking status. ChromeViewHostMsg_RendererTrackedData - render's Send back tracked data (ThreadData in tracked_objects) as a string. ChromeViewHostMsg_IsTrackingEnabled - Asks Browser process to set the current tracking status from browser in the render. BUG=101933 R=jar TEST=browser tests Review URL: http://codereview.chromium.org/8413009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@108770 0039d316-1c4b-4281-b951-d872f2087c98
* Introduces chromium_resources.gyp to factor out Chrome resource generationdhollowa@chromium.org2011-11-031-2/+2
| | | | | | | | | | | | | | | | | | | | 1. Factors out Chrome/Chromium resource generation into a separate chrome_resources.gyp file. 2. Eliminates repetition between Mac and other platforms. 3. Breaks long "repack" actions out into separate files for greater readability. 4. Eliminates circular dependencies in the Aura shell, the Views components, and the compositor when utilizing Chrome resources. BUG=none TEST=try bots run gyps and build correctly. Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=107967 Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=108466 Review URL: http://codereview.chromium.org/8425002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@108517 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 108466 - Introduces chromium_resources.gyp to factor out Chrome ↵dhollowa@chromium.org2011-11-031-2/+2
| | | | | | | | | | | | | | | | | | | | | | | resource generation 1. Factors out Chrome/Chromium resource generation into a separate chrome_resources.gyp file. 2. Eliminates repetition between Mac and other platforms. 3. Breaks long "repack" actions out into separate files for greater readability. 4. Eliminates circular dependencies in the Aura shell, the Views components, and the compositor when utilizing Chrome resources. BUG=none TEST=try bots run gyps and build correctly. Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=107967 Review URL: http://codereview.chromium.org/8425002 TBR=dhollowa@chromium.org Review URL: http://codereview.chromium.org/8448001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@108468 0039d316-1c4b-4281-b951-d872f2087c98
* Introduces chromium_resources.gyp to factor out Chrome resource generationdhollowa@chromium.org2011-11-031-2/+2
| | | | | | | | | | | | | | | | | | 1. Factors out Chrome/Chromium resource generation into a separate chrome_resources.gyp file. 2. Eliminates repetition between Mac and other platforms. 3. Breaks long "repack" actions out into separate files for greater readability. 4. Eliminates circular dependencies in the Aura shell, the Views components, and the compositor when utilizing Chrome resources. BUG=none TEST=try bots run gyps and build correctly. Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=107967 Review URL: http://codereview.chromium.org/8425002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@108466 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 107967 - Introduces chromium_resources.gyp to factor out Chrome ↵dhollowa@chromium.org2011-10-311-2/+2
| | | | | | | | | | | | | | | | | | | | | resource generation 1. Factors out Chrome/Chromium resource generation into a separate chrome_resources.gyp file. 2. Eliminates repetition between Mac and other platforms. 3. Breaks long "repack" actions out into separate files for greater readability. 4. Eliminates circular dependencies in the Aura shell, the Views components, and the compositor when utilizing Chrome resources. BUG=none TEST=try bots run gyps and build correctly. Review URL: http://codereview.chromium.org/8425002 TBR=dhollowa@chromium.org Review URL: http://codereview.chromium.org/8430011 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@107984 0039d316-1c4b-4281-b951-d872f2087c98
* Introduces chromium_resources.gyp to factor out Chrome resource generationdhollowa@chromium.org2011-10-311-2/+2
| | | | | | | | | | | | | | | | 1. Factors out Chrome/Chromium resource generation into a separate chrome_resources.gyp file. 2. Eliminates repetition between Mac and other platforms. 3. Breaks long "repack" actions out into separate files for greater readability. 4. Eliminates circular dependencies in the Aura shell, the Views components, and the compositor when utilizing Chrome resources. BUG=none TEST=try bots run gyps and build correctly. Review URL: http://codereview.chromium.org/8425002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@107967 0039d316-1c4b-4281-b951-d872f2087c98
* Add a new globals object for PPAPI tracking information.brettw@chromium.org2011-10-201-0/+1
| | | | | | | | | | | | | | | | | | | | [ Reland of 106142: http://codereview.chromium.org/8316008 ] This adds a specialization on the host and plugin side of the proxy. This replaces the ad-hoc singleton tracking done by the resource and var trackers with just being getters on this global object. Most code can use the single PpapiGlobals class. I also allow code to get the host and plugin specializations since some code needs access to some specific features of each side. In a later pass I'll move the other stuff out of TrackerBase and delete it. TEST=none BUG=none Review URL: http://codereview.chromium.org/8344025 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@106537 0039d316-1c4b-4281-b951-d872f2087c98
* Move the java channel code to content. As discussed, we don't want to expose ↵jam@chromium.org2011-10-171-2/+0
| | | | | | | | | the plugin and npobject internals to embedders. BUG=98716 Review URL: http://codereview.chromium.org/8317010 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@105851 0039d316-1c4b-4281-b951-d872f2087c98
* Pull FileBrowserPrivate API out of EventBindings.aa@chromium.org2011-10-131-0/+3
| | | | | | Review URL: http://codereview.chromium.org/8229040 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@105221 0039d316-1c4b-4281-b951-d872f2087c98
* Add a per-context "Handler" concept to ChromeV8Extension andaa@chromium.org2011-10-121-2/+5
| | | | | | | | | use it to update the chrome.app bidings. Review URL: http://codereview.chromium.org/8218004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@104975 0039d316-1c4b-4281-b951-d872f2087c98
* Relanding: Add js api for hosted/packaged apps to request notification ↵asargent@chromium.org2011-10-071-0/+2
| | | | | | | | | | | | | | | | | | authorization (this time with fixed unit test, *and* as a special bonus, fixed compile error) BUG=98145 TEST=None (more pieces will be landing that will make it testable) TBR=aa@chromium.org Original Review URL: http://codereview.chromium.org/8113006 Review URL: http://codereview.chromium.org/8186008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@104493 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 104438 - Relanding: Add js api for hosted/packaged apps to request ↵asargent@chromium.org2011-10-071-2/+0
| | | | | | | | | | | | | | | | | | | | | notification authorization (this time with fixed unit test) BUG=98145 TEST=None (more pieces will be landing that will make it testable) TBR=aa@chromium.org Original Review URL: http://codereview.chromium.org/8113006 Review URL: http://codereview.chromium.org/8194005 TBR=asargent@chromium.org Review URL: http://codereview.chromium.org/8184008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@104439 0039d316-1c4b-4281-b951-d872f2087c98
* Relanding: Add js api for hosted/packaged apps to request notification ↵asargent@chromium.org2011-10-071-0/+2
| | | | | | | | | | | | | | | | | | authorization (this time with fixed unit test) BUG=98145 TEST=None (more pieces will be landing that will make it testable) TBR=aa@chromium.org Original Review URL: http://codereview.chromium.org/8113006 Review URL: http://codereview.chromium.org/8194005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@104438 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 104410 - Add js api for hosted/packaged apps to request notification ↵rsleevi@chromium.org2011-10-071-2/+0
| | | | | | | | | | | | | | | | | | | | | authorization BUG=98145 TEST=None (more pieces will be landing that will make it testable) Review URL: http://codereview.chromium.org/8113006 TBR=asargent@chromium.org Review URL: http://codereview.chromium.org/8188004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@104417 0039d316-1c4b-4281-b951-d872f2087c98
* Add js api for hosted/packaged apps to request notification authorizationasargent@chromium.org2011-10-071-0/+2
| | | | | | | | | | BUG=98145 TEST=None (more pieces will be landing that will make it testable) Review URL: http://codereview.chromium.org/8113006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@104410 0039d316-1c4b-4281-b951-d872f2087c98
* I realized that ExtensionBindingsContext is really just theaa@chromium.org2011-10-061-4/+4
| | | | | | | | | Chrome-side wrapper for v8::Context, so renamed to that. Review URL: http://codereview.chromium.org/8110005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@104238 0039d316-1c4b-4281-b951-d872f2087c98
* Factor the static data and its management out ofaa@chromium.org2011-10-041-0/+2
| | | | | | | | | | | ExtensionBindingsContext into ExtensionBindingsContextSet. Add a unit test (!!). Review URL: http://codereview.chromium.org/8093017 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@103887 0039d316-1c4b-4281-b951-d872f2087c98
* Fix compilation when use_aura is turned ON.vamshi@motorola.com2011-10-031-5/+0
| | | | | | | | | | | BUG=none TEST=none Patch by vamshi@motorola.com Review URL: http://codereview.chromium.org/8109001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@103688 0039d316-1c4b-4281-b951-d872f2087c98
* Rename ExtensionBase to ChromeV8Extension and simplify.aa@chromium.org2011-10-011-6/+2
| | | | | | | | | | Remove JsOnlyV8Extensions since we can just create instances of ChromeV8Extension where necessary. Instead of using made-up names like "chrome/extensionprocessbindings", standardize on the path to the file as the name of an extension. Review URL: http://codereview.chromium.org/8100014 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@103624 0039d316-1c4b-4281-b951-d872f2087c98
* Clean up DEPS rules against content so that not all chrome code can include ↵jam@chromium.org2011-09-301-1/+0
| | | | | | | | | anything from content. I also moved ChromeContentPluginClient to chrome\plugin where it really belongs (i.e. to match what we do with the utility client). Review URL: http://codereview.chromium.org/8095011 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@103569 0039d316-1c4b-4281-b951-d872f2087c98
* More aura build fixesdavemoore@chromium.org2011-09-301-1/+6
| | | | | | | | | BUG=None TEST=None Review URL: http://codereview.chromium.org/8094001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@103473 0039d316-1c4b-4281-b951-d872f2087c98
* Re-re-reland r99689: Refactor the ContextInfo struct from bindings_utils into aaa@chromium.org2011-09-281-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | real class. In this iteration, I have made the following changes to protect against various crashes: * In ExtensionProcessBindings::HandleResponse(), null check ExtensionBindingsContext before using. It can legitimately be NULL in the case where a frame is removed before a response comes back. We see crashes here in the canary. * Delete ExtensionBindingsContext asynchronously. I haven't seen any crashes related to this, but it seems like a call to CallChromeHiddenMethod() could end up immediately deleting any other ExtensionBindingsContext. That would be super hard to reason about, so deleting asynchronously makes more sense. See ExtensionBindingsContext::HandleV8ContextReleased(). BUG=97929 Review URL: http://codereview.chromium.org/8068004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@103175 0039d316-1c4b-4281-b951-d872f2087c98
* Hilariously revert "Re-reland r99689: Refactor theaa@chromium.org2011-09-271-2/+2
| | | | | | | | | | | | ContextInfo struct from bindings_utils into a real class." This reverts commit 5b3c4a3471179c86e0b8f3ce3c7620e69806d173. BUG=97929 Review URL: http://codereview.chromium.org/8051011 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@102889 0039d316-1c4b-4281-b951-d872f2087c98
generated by cgit v1.1 at 2025-02-12 05:39:41 +0000