summaryrefslogtreecommitdiffstats
path: root/webkit
Commit message (Collapse)AuthorAgeFilesLines
* Update WebPermissionClient implementation to support image URLsjochen@chromium.org2011-09-192-3/+6
| | | | | | | | | | BUG=none TEST=still works Review URL: http://codereview.chromium.org/7931020 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@101722 0039d316-1c4b-4281-b951-d872f2087c98
* Aura under Linux is enabled with:saintlou@chromium.org2011-09-181-0/+10
| | | | | | | | | | | 'use_aura': 1, BUG=none TEST=none Review URL: http://codereview.chromium.org/7850026 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@101704 0039d316-1c4b-4281-b951-d872f2087c98
* Fix a comment in cpp_binding_example.hsteveblock@chromium.org2011-09-171-2/+2
| | | | | | Review URL: http://codereview.chromium.org/7886005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@101656 0039d316-1c4b-4281-b951-d872f2087c98
* Add layout test exception.levin@chromium.org2011-09-171-0/+3
| | | | | | | | | | TBR=keishi@chromium.org r95343 Review URL: http://codereview.chromium.org/7912006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@101618 0039d316-1c4b-4281-b951-d872f2087c98
* Rewriting FFmpegAudioDecoder and eliminating DecoderBase.scherkus@chromium.org2011-09-171-0/+1
| | | | | | | | | BUG=93379 TEST=media_unittests, layout tests Review URL: http://codereview.chromium.org/6901135 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@101607 0039d316-1c4b-4281-b951-d872f2087c98
* Roll WebKit DEPS 95307:95316.levin@chromium.org2011-09-161-11/+0
| | | | | | | | TBR=keishi@chromium.org Review URL: http://codereview.chromium.org/7922025 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@101596 0039d316-1c4b-4281-b951-d872f2087c98
* Remove generation of .rc files in locale grd files.tony@chromium.org2011-09-161-54/+1
| | | | | | | | | | | | | | Since we use pak files on all platforms, we don't use the .rc files anymore. This should speed up the build a tiny amount. On my Z600, grit on generated_resources.grd goes from 38s to 25s. BUG=None TEST=compiles Review URL: http://codereview.chromium.org/7922002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@101590 0039d316-1c4b-4281-b951-d872f2087c98
* chrome.experimental.downloads.download() implementationbenjhayden@chromium.org2011-09-161-39/+1
| | | | | | | | | | | | | | Ownership: (Done) Asanka: content/browser/download/* (Done) Brett: webkit/plugins/ppapi/ppb_url_request_info_impl.cc (Done) Chris: net/* (Done) John: content/browser/renderer_host/* (Done) Mihai: extensions/* Review URL: http://codereview.chromium.org/7647028 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@101583 0039d316-1c4b-4281-b951-d872f2087c98
* Add a new error code for a null callback on the main thread.brettw@chromium.org2011-09-1615-30/+54
| | | | | | | | Convert the NaCl and ChromeIPC proxies to use the new value. Review URL: http://codereview.chromium.org/7885014 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@101556 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 101545 - Moved code not relating to GPU scheduling out of ↵rsimha@chromium.org2011-09-161-60/+53
| | | | | | | | | | | | | | | | | | | | GpuScheduler and into GpuCommandBufferStub. Reason for revert: Compile failure on Arm. This was mostly a refactor because the code was awkward. I also deleted the original gles2_demo since we have the gles2 book demos now. THings still to do are a common way of setting up the few objects involved in initializing a command buffer that is now more-or-less duplicated in the gles2 conformance tests, the gles2 demos, the command buffer stub and the in-process webgl context. I also want to completely remove the reference to the decoder from the scheduler. I tested WebGL on both windows and mac and saw no regressions. I checked the conformance tests, the gpu tests and am running by the try bots now. The gles2 demos still work. Review URL: http://codereview.chromium.org/7782041 TBR=apatrick@chromium.org Review URL: http://codereview.chromium.org/7920015 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@101547 0039d316-1c4b-4281-b951-d872f2087c98
* Moved code not relating to GPU scheduling out of GpuScheduler and into ↵apatrick@chromium.org2011-09-161-53/+60
| | | | | | | | | | | | | | | GpuCommandBufferStub. This was mostly a refactor because the code was awkward. I also deleted the original gles2_demo since we have the gles2 book demos now. THings still to do are a common way of setting up the few objects involved in initializing a command buffer that is now more-or-less duplicated in the gles2 conformance tests, the gles2 demos, the command buffer stub and the in-process webgl context. I also want to completely remove the reference to the decoder from the scheduler. I tested WebGL on both windows and mac and saw no regressions. I checked the conformance tests, the gpu tests and am running by the try bots now. The gles2 demos still work. Review URL: http://codereview.chromium.org/7782041 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@101545 0039d316-1c4b-4281-b951-d872f2087c98
* Add messages/handlers for the invocation sequence. Copy from ↵gbillock@chromium.org2011-09-163-0/+54
| | | | | | | | | | | | | | geolocation_dispatcher. Get return path serialization working. R=groby@chromium.org BUG=None TEST=None Review URL: http://codereview.chromium.org/7823002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@101536 0039d316-1c4b-4281-b951-d872f2087c98
* Don't try to call setDefersLoading when the loader is null. This does a bit ↵brettw@google.com2011-09-162-10/+34
| | | | | | | | | | of cleanup so the checks for the null-ness of this variable is centralized, and I filed a bug for not having this implemented for main document loads. TEST=manual BUG=96727 Review URL: http://codereview.chromium.org/7920001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@101524 0039d316-1c4b-4281-b951-d872f2087c98
* Declarations for Pepper IME API.kinaba@chromium.org2011-09-161-0/+8
| | | | | | | | | | | | | | BUG=59425 TEST=Check that ppapi_tests compile. This change list is the first part for adding IME support for PPAPI. For effectiveness of reviewing, I'll split the rather large change into three parts: (*1) Header files declaring IME API. (2) Boilerplate code for proxy & thunk stuff. (3) Actual implementation in webkit/plugin/ppapi/* and content/renderer/*. Review URL: http://codereview.chromium.org/7882004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@101458 0039d316-1c4b-4281-b951-d872f2087c98
* Convert file request to http request for layout test for Android.zhenghao@google.com2011-09-163-4/+153
| | | | | | | | | | | | | | | | | | | | | | | | | For the limited environment of Android, we cannot host all layout test material on Android. We implemented a 'forwarder', which can forward all http request on Android to the same port on the host machine running Linux/Mac. However, most layout tests are file-based tests, so we need to convert file request to http request for layout tests. We enable file-over-http to bridge the file protocol to http protocol in here, which can (1) run the layout tests on android target device, but never need to push the test files and corresponding resources to device, which saves huge running time. (2) still run non-http layout (tests not under LayoutTests/http) tests via file protocol without breaking test environment / convention of webkit layout tests, which are followed by current all webkit ports. BUG= TEST= Review URL: http://codereview.chromium.org/7901009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@101441 0039d316-1c4b-4281-b951-d872f2087c98
* Included ppb_opengles2.h instead of ppb_opengles.h.alokp@chromium.org2011-09-161-1/+1
| | | | | | Review URL: http://codereview.chromium.org/7903014 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@101434 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 101418 - Fix aspect ratio and clarify video frame dimensionsvrk@google.com2011-09-163-11/+11
| | | | | | | | | | | | BUG=18941,94861 TEST=video-aspect-ratio.html Review URL: http://codereview.chromium.org/7864009 TBR=vrk@google.com Review URL: http://codereview.chromium.org/7919006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@101420 0039d316-1c4b-4281-b951-d872f2087c98
* Fix aspect ratio and clarify video frame dimensionsvrk@google.com2011-09-163-11/+11
| | | | | | | | | BUG=18941,94861 TEST=video-aspect-ratio.html Review URL: http://codereview.chromium.org/7864009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@101418 0039d316-1c4b-4281-b951-d872f2087c98
* Relay server support for P2P Transport API.sergeyu@chromium.org2011-09-153-11/+58
| | | | | | | | | BUG=41776 TEST=None Review URL: http://codereview.chromium.org/7791008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@101396 0039d316-1c4b-4281-b951-d872f2087c98
* add some flaky tests for nowdpranke@chromium.org2011-09-151-0/+7
| | | | | | | | TBR=none Review URL: http://codereview.chromium.org/7909014 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@101365 0039d316-1c4b-4281-b951-d872f2087c98
* Remove support for the old scrollbar interfaces.brettw@chromium.org2011-09-153-101/+0
| | | | | | | | These appear to no longer be used by PDF or NaCl, so we can remove them. Review URL: http://codereview.chromium.org/7889036 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@101364 0039d316-1c4b-4281-b951-d872f2087c98
* Remove old cruft from test_expectaions which isn't needed anymore.levin@chromium.org2011-09-151-2/+0
| | | | | | | | TBR=keishi@chromium.org Review URL: http://codereview.chromium.org/7904020 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@101341 0039d316-1c4b-4281-b951-d872f2087c98
* Roll WebKit DEPS 95087:95187.levin@chromium.org2011-09-151-0/+3
| | | | | | | | TBR=keishi@chromium.org Review URL: http://codereview.chromium.org/7903011 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@101320 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 101310 - Roll WebKit DEPS 95087:95189.levin@chromium.org2011-09-151-3/+0
| | | | | | | | | | | TBR=keishi@chromium.org Review URL: http://codereview.chromium.org/7909006 TBR=levin@chromium.org Review URL: http://codereview.chromium.org/7913001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@101314 0039d316-1c4b-4281-b951-d872f2087c98
* Roll WebKit DEPS 95087:95189.levin@chromium.org2011-09-151-0/+3
| | | | | | | | TBR=keishi@chromium.org Review URL: http://codereview.chromium.org/7909006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@101310 0039d316-1c4b-4281-b951-d872f2087c98
* Reland r101269: Store plug-in enabled/disabled state in PluginPrefs instead ↵bauerb@chromium.org2011-09-1516-933/+54
| | | | | | | | | | | | | | | | | of WebPluginInfo, to allow different sets of enabled/disabled plug-ins to be specified per profile. The relanded version fixes GViewRequestInterceptorTest on ChromeOS and a memory leak in PluginListTest.GetPluginGroup. Original review URL: http://codereview.chromium.org/7848025 TBR=jam@chromium.org BUG=80794 TEST=Open two profiles, disable different plugins in them. Review URL: http://codereview.chromium.org/7904009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@101297 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 101269 - Store plug-in enabled/disabled state in PluginPrefs instead ↵bauerb@chromium.org2011-09-1516-53/+932
| | | | | | | | | | | | | | of WebPluginInfo, to allow different sets of enabled/disabled plug-ins to be specified per profile. BUG=80794 TEST=Open two profiles, disable different plugins in them. Review URL: http://codereview.chromium.org/7848025 TBR=bauerb@chromium.org Review URL: http://codereview.chromium.org/7901015 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@101272 0039d316-1c4b-4281-b951-d872f2087c98
* Store plug-in enabled/disabled state in PluginPrefs instead of ↵bauerb@chromium.org2011-09-1516-932/+53
| | | | | | | | | | | WebPluginInfo, to allow different sets of enabled/disabled plug-ins to be specified per profile. BUG=80794 TEST=Open two profiles, disable different plugins in them. Review URL: http://codereview.chromium.org/7848025 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@101269 0039d316-1c4b-4281-b951-d872f2087c98
* Add Android platform for webkit/support.zhenghao@google.com2011-09-154-10/+17
| | | | | | | | | | | | | | | Add platform_support_android.cc, which is a copy of platform_support_gtk.cc now. Will add Android specific code in subsequent changes. And use default MessageLoop instead of MessageLoopForUI, as Android's MessageLoopForUI goes through Java. BUG= TEST= Review URL: http://codereview.chromium.org/7795049 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@101243 0039d316-1c4b-4281-b951-d872f2087c98
* Move the V8 benchmarking_extension.cc/.h files out of webkit/extensions into ↵ananta@chromium.org2011-09-156-251/+0
| | | | | | | | | | | | | | | | | | | chrome as the functionality exposed by this extension is chrome specific. The IPCs used by this extension have all been moved to chrome to a new header file benchmarking_messages.h in chrome\common. Added a new message filter object in chrome\browser which filters the benchmarking IPCs. Most of the changes in this CL are centered around ensuring that the benchmarking stuff gets initialized in the same order as before. Continuing changes to get rid of the pattern of IPC messags spanning across content and chrome. BUG=87335 Review URL: http://codereview.chromium.org/7885013 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@101214 0039d316-1c4b-4281-b951-d872f2087c98
* Use enum instead of string to specify transport type.sergeyu@chromium.org2011-09-154-20/+17
| | | | | | | | | TEST=None BUG=41776 Review URL: http://codereview.chromium.org/7891014 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@101211 0039d316-1c4b-4281-b951-d872f2087c98
* Fixing dropped seeks.serya@chromium.org2011-09-152-1/+19
| | | | | | | | | | BUG=92251,chromium-os:19888 TEST=None Review URL: http://codereview.chromium.org/7890012 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@101197 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 101189 - Use enum instead of string to specify transport type.sergeyu@chromium.org2011-09-144-17/+20
| | | | | | | | | | | | TEST=None BUG=41776 Review URL: http://codereview.chromium.org/7891014 TBR=sergeyu@chromium.org Review URL: http://codereview.chromium.org/7901002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@101191 0039d316-1c4b-4281-b951-d872f2087c98
* Use enum instead of string to specify transport type.sergeyu@chromium.org2011-09-144-20/+17
| | | | | | | | | TEST=None BUG=41776 Review URL: http://codereview.chromium.org/7891014 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@101189 0039d316-1c4b-4281-b951-d872f2087c98
* Convert dev interfaces to use the macro system.brettw@chromium.org2011-09-149-131/+36
| | | | | | | | | | | | This converts the non-"weird" dev interfaces to using the macro system. The trusted and private ones are still remaining. This moves the find implementation to the instance API which saves some code and routing. Review URL: http://codereview.chromium.org/7887021 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@101177 0039d316-1c4b-4281-b951-d872f2087c98
* Check for a crashed plugin before creating resources to send to the pluginbrettw@google.com2011-09-142-0/+14
| | | | | | | BUG=95710 Review URL: http://codereview.chromium.org/7839040 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@101119 0039d316-1c4b-4281-b951-d872f2087c98
* Cleaned up DEPS under webkit.alokp@chromium.org2011-09-143-12/+0
| | | | | | Review URL: http://codereview.chromium.org/7876006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@101116 0039d316-1c4b-4281-b951-d872f2087c98
* Eliminate headers from precompile.h that define 'interface' to avoid compile ↵joi@chromium.org2011-09-141-96/+96
| | | | | | | | | | | failures on Windows that would not be seen on Mac or Linux. I didn't do an exact measurement but I think we lose very little in terms of build time from this; Windows.h is the big Windows-specific header and it's still precompiled. BUG=none TEST=it builds Review URL: http://codereview.chromium.org/7888030 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@101059 0039d316-1c4b-4281-b951-d872f2087c98
* Remove deprecated STR-resource-based Mac plugin info loadingstuartmorgan@chromium.org2011-09-141-148/+8
| | | | | | | | | | | This method of declaring resources has been deprecated for quite some time, and almost all plugins now support the new Info.plist approach. This code occasionally leads to browser crashes, so removing it should improve stability. BUG=96422 TEST=Most plugins should still show up in about:plugins on the Mac. A handful (see bug) will not. Review URL: http://codereview.chromium.org/7886008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@101036 0039d316-1c4b-4281-b951-d872f2087c98
* Prepare appcache to build as a DLL in the component build.dpranke@chromium.org2011-09-149-13/+59
| | | | | | | | | | | | | | | | This is needed to break the circular dependencies between appcache, chrome, and content (once we fix the messageloopproxy issue as seen in http://codereview.chromium.org/7841052/ ). R=michaeln@chromium.org, jam@chromium.org BUG=none TEST=none Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=100831 Review URL: http://codereview.chromium.org/7859021 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@101017 0039d316-1c4b-4281-b951-d872f2087c98
* s patch tries to remove most of the manual registration for Pepper ↵brettw@chromium.org2011-09-137-189/+81
| | | | | | | | | | | | | | | | | | | | | | | | interfaces, and replaces it with a list of macros. When files want to know which Pepper interface names and structs there are, they define what they want to do with the macros, and then include the relevant files for the classes of interfaces they want (stable, private, dev). This re-lands my previous change. Original Review URL: http://codereview.chromium.org/7874002 This does not convert all the dev interfaces. I just did a few to keep the patch smaller. So there is still a lot of manual registration. This fixes the previous design problem where we assumed one *_Proxy object == one interface. We have been hacking around this lately with duplicate GetInfo calls, but this doesn't work for PPP interfaces. Now, a _Proxy object is just there to help keep things organized. One proxy can handle zero, one, or many interfaces, and this mapping is controlled by just one line in the interfaces file. So for example, to add a new function to a new version of an interface with backward compatibility, you would add that function to the _api.h file, and write a thunk for the new interface. Then you only need to add one line to the interfaces_ppb_public_stable.h file and that will be hooked up with the proxy and the implementation. This removes some _proxy objects/files that were used only to declare that the interfaces existed, since they're no longer necessary. I folded Console into the Instance API which removed a bunch of code. I removed FileChooser 0.4. I think everybody has converted to the new one, and I think parts of it weren't even hooked up properly anymore. Review URL: http://codereview.chromium.org/7887001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@100936 0039d316-1c4b-4281-b951-d872f2087c98
* Third try at committing this.erikwright@chromium.org2011-09-131-3/+11
| | | | | | | | | | | | | | | | | | Patchset 1 is the original patch ( http://codereview.chromium.org/7833042/ ). Patchset 2 is the original patch plus a fix to a memory leak from the initial commit ( http://codereview.chromium.org/7831056/ ) On the first try there was a memory leak in a test. I fixed that, but made a mistake in the commit (the committed code did not correspond to the reviewed code). Both commits were reverted. I then landed a new CL ( http://codereview.chromium.org/7860039/ ) that contained the correct changes combining the first two CLs. This caused an error in heapchecker for which a suppression has subsequently been defined ( http://codereview.chromium.org/7780010 ). In summary, all of this is reviewed, minus some lint fixes. BUG=68657 TEST=net_unittests / DeferredCookieTaskTest.* and CookieMonsterTest.* TBR=estade Review URL: http://codereview.chromium.org/7891008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@100932 0039d316-1c4b-4281-b951-d872f2087c98
* Revert r100853, 100851. Win_shared build is broken.brettw@chromium.org2011-09-137-81/+189
| | | | git-svn-id: svn://svn.chromium.org/chrome/trunk/src@100854 0039d316-1c4b-4281-b951-d872f2087c98
* This patch tries to remove most of the manual registration for Pepper ↵brettw@chromium.org2011-09-137-189/+81
| | | | | | | | | | | | | | | | | | | | | | | | interfaces, and replaces it with a list of macros. When files want to know which Pepper interface names and structs there are, they define what they want to do with the macros, and then include the relevant files for the classes of interfaces they want (stable, private, dev). This re-lands my previous change. Original Review URL: http://codereview.chromium.org/7740038 This does not convert all the dev interfaces. I just did a few to keep the patch smaller. So there is still a lot of manual registration. This fixes the previous design problem where we assumed one *_Proxy object == one interface. We have been hacking around this lately with duplicate GetInfo calls, but this doesn't work for PPP interfaces. Now, a _Proxy object is just there to help keep things organized. One proxy can handle zero, one, or many interfaces, and this mapping is controlled by just one line in the interfaces file. So for example, to add a new function to a new version of an interface with backward compatibility, you would add that function to the _api.h file, and write a thunk for the new interface. Then you only need to add one line to the interfaces_ppb_public_stable.h file and that will be hooked up with the proxy and the implementation. This removes some _proxy objects/files that were used only to declare that the interfaces existed, since they're no longer necessary. I folded Console into the Instance API which removed a bunch of code. I removed FileChooser 0.4. I think everybody has converted to the new one, and I think parts of it weren't even hooked up properly anymore. Review URL: http://codereview.chromium.org/7874002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@100851 0039d316-1c4b-4281-b951-d872f2087c98
* Revert "Prepare appcache to build as a DLL in the component build."dpranke@chromium.org2011-09-139-59/+13
| | | | | | | | | | TBR=michaeln@chromium.org BUG= TEST= Review URL: http://codereview.chromium.org/7881006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@100832 0039d316-1c4b-4281-b951-d872f2087c98
* Prepare appcache to build as a DLL in the component build.dpranke@chromium.org2011-09-139-13/+59
| | | | | | | | | | | | | | This is needed to break the circular dependencies between appcache, chrome, and content (once we fix the messageloopproxy issue as seen in http://codereview.chromium.org/7841052/ ). R=michaeln@chromium.org, jam@chromium.org BUG=none TEST=none Review URL: http://codereview.chromium.org/7859021 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@100831 0039d316-1c4b-4281-b951-d872f2087c98
* Coverity: Initialize member variables.jhawkins@chromium.org2011-09-124-2/+8
| | | | | | | | | | | | | CID_COUNT=23 CID=100941,100986,100987,101015,101016,101017,101227,101229,101230,101332,101333,101334,101335,101336,101337,101355,101356,101359,101362,101363,101381,101389,101417 BUG=none TEST=none R=kmadhusu@chromium.org Review URL: http://codereview.chromium.org/7780013 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@100774 0039d316-1c4b-4281-b951-d872f2087c98
* Make the NaCl windows 64 bit binaries not depend on chrome targets. These ↵jam@chromium.org2011-09-121-25/+0
| | | | | | | | | | | | | targets are very simple and used little code from chrome targets. However their dependency on chrome targets was problematic because a lot of code wasn't being built for 64 bit on Windows, and so there were a lot of "dummy" files being added with stub functions and code was also being compiled out in random places for NACL_WIN64. I've made the NaCl 64 bit windows targets self contained. They do use a few files from common, but those files are self-contained. In the future, we could move these to be in the same 64 bit target as the constants from common. However that won't make a maintenance difference since someone could still introduce link dependencies to other files in common. Additionally, since we're not using chrome code anymore, we can avoid having both nacl.exe and nacl.dll. nacl.exe is sufficient, and this saves 1.4MB of uncompresed binaries in the installer. BUG=86322 Review URL: http://codereview.chromium.org/7863024 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@100767 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 100748 - This patch tries to remove most of the manual registration ↵dmazzoni@chromium.org2011-09-127-81/+189
| | | | | | | | | | | | | | | | | | | | | | | | | for Pepper interfaces, and replaces it with a list of macros. When files want to know which Pepper interface names and structs there are, they define what they want to do with the macros, and then include the relevant files for the classes of interfaces they want (stable, private, dev). This does not convert all the dev interfaces. I just did a few to keep the patch smaller. So there is still a lot of manual registration. This fixes the previous design problem where we assumed one *_Proxy object == one interface. We have been hacking around this lately with duplicate GetInfo calls, but this doesn't work for PPP interfaces. Now, a _Proxy object is just there to help keep things organized. One proxy can handle zero, one, or many interfaces, and this mapping is controlled by just one line in the interfaces file. So for example, to add a new function to a new version of an interface with backward compatibility, you would add that function to the _api.h file, and write a thunk for the new interface. Then you only need to add one line to the interfaces_ppb_public_stable.h file and that will be hooked up with the proxy and the implementation. This removes some _proxy objects/files that were used only to declare that the interfaces existed, since they're no longer necessary. I folded Console into the Instance API which removed a bunch of code. I removed FileChooser 0.4. I think everybody has converted to the new one, and I think parts of it weren't even hooked up properly anymore. Review URL: http://codereview.chromium.org/7740038 TBR=brettw@chromium.org Review URL: http://codereview.chromium.org/7844018 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@100754 0039d316-1c4b-4281-b951-d872f2087c98
* This patch tries to remove most of the manual registration for Pepper ↵brettw@chromium.org2011-09-127-189/+81
| | | | | | | | | | | | | | | | | | | | | | interfaces, and replaces it with a list of macros. When files want to know which Pepper interface names and structs there are, they define what they want to do with the macros, and then include the relevant files for the classes of interfaces they want (stable, private, dev). This does not convert all the dev interfaces. I just did a few to keep the patch smaller. So there is still a lot of manual registration. This fixes the previous design problem where we assumed one *_Proxy object == one interface. We have been hacking around this lately with duplicate GetInfo calls, but this doesn't work for PPP interfaces. Now, a _Proxy object is just there to help keep things organized. One proxy can handle zero, one, or many interfaces, and this mapping is controlled by just one line in the interfaces file. So for example, to add a new function to a new version of an interface with backward compatibility, you would add that function to the _api.h file, and write a thunk for the new interface. Then you only need to add one line to the interfaces_ppb_public_stable.h file and that will be hooked up with the proxy and the implementation. This removes some _proxy objects/files that were used only to declare that the interfaces existed, since they're no longer necessary. I folded Console into the Instance API which removed a bunch of code. I removed FileChooser 0.4. I think everybody has converted to the new one, and I think parts of it weren't even hooked up properly anymore. Review URL: http://codereview.chromium.org/7740038 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@100748 0039d316-1c4b-4281-b951-d872f2087c98
generated by cgit v1.1 at 2025-02-12 05:01:53 +0000