summaryrefslogtreecommitdiffstats
path: root/ipc
Commit message (Collapse)AuthorAgeFilesLines
* Implement ManifestManager to handle manifest in content/.mlamouri2014-09-161-0/+1
| | | | | | | | | | | | | | | | | | | This can be used from the renderer process or the browser process. Requesting the manifest can be done via one call, a callback has to be passed and will run with the manifest passed as parameter. A failure will return the empty manifest. Some more logic could be added like caching the manifest in the browser process or keeping track of the manifest dirty state in the browser process but those things can be added transparently later. BUG=366145 Review URL: https://codereview.chromium.org/537053002 Cr-Commit-Position: refs/heads/master@{#295085}
* ChannelMojo: Handle errors in pending message processing.morrita2014-09-162-1/+132
| | | | | | | | | | | | | | | | | ChannelMojo::OnConnect() ignores errors in Send() but the error results deleting |message_readrer_| which causes null access. This CL add an error check for that. This also adds some hooks to make this testable by faking lower level API. TEST=ipc_channel_mojo_unittest.cc BUG=410813 R=yzshen@chromium.org, viettrungluu@chromium.org Review URL: https://codereview.chromium.org/554363004 Cr-Commit-Position: refs/heads/master@{#294997}
* IPC: Add more debug info.rvargas2014-09-162-6/+21
| | | | | | | | | | | | | | | | | We're still seeing crashes that appear to come from multiple simultaneous writes. Given that writing_ is false when we crash, but the operation matches the current thread, file handle and IPC Channel, it could be that for some reason a failed Write is still genarating a notification. So this CL stores two error codes (one for the current operation and another for a previous failure), and the write size. BUG=387876 R=cpu@chromium.org Review URL: https://codereview.chromium.org/564863003 Cr-Commit-Position: refs/heads/master@{#294993}
* Cleanup: Use base/files/file_util.h instead of base/file_util.h in [c-n]*/thestig2014-09-133-3/+3
| | | | | | | | | TBR=cpu@chromium.org,jochen@chromium.org,piman@chromium.org,rockot@chromium.org,rsleevi@chromium.org,vrk@chromium.org ,zea@chromium.org Review URL: https://codereview.chromium.org/563173002 Cr-Commit-Position: refs/heads/master@{#294712}
* WebCacheManager::ClearCacheOnNavigation() is used by webrequest api, which ↵hanxi2014-09-111-0/+1
| | | | | | | | | | | | | | | | | will be moved to //extensions. So we need to resolve its the dependency on WebCacheManager by moving WebCacheManager to //components. There are two ipc messages sent in WebCacheManger: -ChromeViewMsg_SetCacheCapacities -ChromeViewMsg_ClearCache Both ipc handlers were in ChromeRenderProcessObserver. These two ipcs are renamed as WebCacheMsg_XXX, and we introduce WebCacheRenderProcessObserver and handle these ipcs in //components. BUG=352293,411437 TBR=agl@chromium.org Review URL: https://codereview.chromium.org/528363002 Cr-Commit-Position: refs/heads/master@{#294475}
* Refactoring: Move MessagePipeReader subclasess out from ChannelMojomorrita2014-09-116-382/+524
| | | | | | | | | | | | | | | | | There are a few MessagePipeReader sublcasses defined as inter-classes of ChannelMojo. This makes it harder to hook them for unit testing. This CL moves these classes their own file. Now unittest can define subclasses of these to hook some behavor for testing. This is a preparation for a crash fix. BUG=410813 TEST=none R=viettrungluu@chromium.org,yzshen@chromium.org Review URL: https://codereview.chromium.org/559723002 Cr-Commit-Position: refs/heads/master@{#294439}
* Add ipc_mojo_perftestsmorrita2014-09-0915-370/+575
| | | | | | | | | | | | | | | | This change adds ipc_mojo_perftests that runs the same benchmark as of ipc_perftests. Now head-to-head comparison becomes possible. For this change, whole ipc_perftests logic is extracted to ipc_perftest_support.cc to make it reusable by ipc_mojo_perftests. TEST=none BUG=none R=jam@chromium.org, darin@chromium.org, yuzhu@chromium.org Review URL: https://codereview.chromium.org/536213002 Cr-Commit-Position: refs/heads/master@{#293988}
* Mark gtest and gmock as testonly in GN.Brett Wilson2014-09-081-0/+1
| | | | | | | | | | | This updates the targets that depend on these to also be test-only. BUG=http://crbug.com/412064 R=jamesr@chromium.org, tfarina@chromium.org Review URL: https://codereview.chromium.org/551933002 Cr-Commit-Position: refs/heads/master@{#293808}
* Add wfh@chromium.org to ipc OWNERS filesWill Harris2014-09-081-0/+1
| | | | | | | | | BUG=None R=thestig@chromium.org Review URL: https://codereview.chromium.org/550503003 Cr-Commit-Position: refs/heads/master@{#293783}
* Fix a DCHECK violation in ipc_perftestsyzshen2014-09-053-27/+21
| | | | | | | | | | | | | | | | | We only allow one MessageLoop instance alive at any given time on a thread. If we use IPCTestBase::set_message_loop(), we need to do it in two steps: first set the message loop to NULL, and then set it to a newly-created message loop. Therefore, I changed the interface of IPCTestBase to make it a little more friendly. BUG=None TEST=ipc_perftests should not crash in debug build. Review URL: https://codereview.chromium.org/538593002 Cr-Commit-Position: refs/heads/master@{#293439}
* ipc/ fixups for scoped_refptr operator T* removal.dcheng2014-09-033-4/+5
| | | | | | | | BUG=110610 Review URL: https://codereview.chromium.org/535723002 Cr-Commit-Position: refs/heads/master@{#293170}
* Remove cevans@ from OWNERS files.thestig2014-08-291-1/+0
| | | | | | | | NOTRY=true Review URL: https://codereview.chromium.org/524433003 Cr-Commit-Position: refs/heads/master@{#292549}
* Add ChannelProxy benchmark to ipc_perftests.morrita2014-08-294-18/+81
| | | | | | | | | | | | | | | | This CL add a ChannelProxy equivalent of the Channel benchmark to ipc_perftests. To use it in the benchmark, TestIOThread is extracted from mojo/system/ to base/test TEST=IPChannelPerfTest.ChannelProxyPingPong R=viettrungluu@chromium.org,brettw@chromium.org,darin@chromium.org BUG=none Review URL: https://codereview.chromium.org/488003003 Cr-Commit-Position: refs/heads/master@{#292518}
* Manual fixups for scoped_refptr operator T* removal in ipc/dcheng2014-08-284-25/+31
| | | | | | | | BUG=110610 Review URL: https://codereview.chromium.org/512153002 Cr-Commit-Position: refs/heads/master@{#292404}
* Declaring the weak_ptr_factory in proper order.anujk.sharma2014-08-283-7/+6
| | | | | | | | | | | | Cleaning up weak_ptr_factory destruction order in "src/ipc" module. WeakPtrFactory should remain the last member so it'll be destroyed and invalidate its weak pointers before any other members are destroyed. BUG=303818 Review URL: https://codereview.chromium.org/508903002 Cr-Commit-Position: refs/heads/master@{#292334}
* pdf: Create a separate component for using the pdf pepper plugin.Sadrul Habib Chowdhury2014-08-281-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Create a component necessary for showing PDF in a content-based client. Much of the relevant code currently lives in //chrome/, and is usable by chrome. Moving this code into a separate component in //components/pdf/ allows it to be easily used by other content-clients (e.g. app-shell, athena, etc.). This patch moves PPB_PDF_Impl (implementation for the PPB_PDF interface in ppapi) and the relevant IPC messages in the pdf component. A short summary of the changes in this patch: . Move ppb_pdf_impl.cc|h into //components/pdf from //chrome/renderer/pepper . Put this code in the 'pdf' namespace. This code lives in 'pdf_renderer' target. 'chrome_renderer' depends on this target. . Move the following IPC messages from render_messages.h to pdf_messages.h: - PDFUpdateContentRestrictions - PDFHasUnsupportedFeature - PDFSaveURLAs - PDFModalPromptForPassword Change the prefix of these messages from ChromeViewHostMsg_ to PDFHostMsg_ . Move PDFTabHelper into //components/pdf from //chrome/browser/ui/pdf. Put this code in the 'pdf' namespace. This code lives in 'pdf_browser' target. 'chrome_browser' depends on this target. BUG=401242 R=blundell@chromium.org, raymes@chromium.org, thestig@chromium.org, tsepez@chromium.org TBR=darin@chromium.org for DEPS Review URL: https://codereview.chromium.org/477263003 Cr-Commit-Position: refs/heads/master@{#292313}
* Remove implicit conversions from scoped_refptr to T* in ipc/dcheng2014-08-272-2/+2
| | | | | | | | | | | This patch was generated by running the rewrite_scoped_refptr clang tool on a Linux build. BUG=110610 Review URL: https://codereview.chromium.org/507773002 Cr-Commit-Position: refs/heads/master@{#292130}
* Credential Manager: Renderer-side implementation.mkwst@chromium.org2014-08-221-0/+1
| | | | | | | | | | | | | | | | | | | | | | | This patch does three things: 1. Implements the Blink platform interface WebCredentialManagerClient by adding a CredentialManagerClient class to the password manager component. 2. Wires that implementation up to Blink's platform layer by creating an instance of the new client, held in ChromeContentRendererClient, and setting it as each new RenderView's client. 3. Stubs out IPCs for the renderer to pass messages up to the browser in order to do the heavy lifting of actually answering Blink's requests by generating and delivering Credential objects. BUG=400674 TBR=tkent@chromium.org Review URL: https://codereview.chromium.org/464883002 Cr-Commit-Position: refs/heads/master@{#291390} git-svn-id: svn://svn.chromium.org/chrome/trunk/src@291390 0039d316-1c4b-4281-b951-d872f2087c98
* Mojo: Add PlatformSupport argument to mojo::embedder::Init().viettrungluu@chromium.org2014-08-201-2/+2
| | | | | | | | | | | | Also add InitWithSimplePlatformSupport() to test_embedder.h. Use that for all tests, for now (this won't be right in the long run). R=darin@chromium.org Review URL: https://codereview.chromium.org/489593004 Cr-Commit-Position: refs/heads/master@{#290856} git-svn-id: svn://svn.chromium.org/chrome/trunk/src@290856 0039d316-1c4b-4281-b951-d872f2087c98
* Convert IPC::ChannelMojo initialization to use new ↵viettrungluu@chromium.org2014-08-122-40/+25
| | | | | | | | | | | | | mojo::embedder::CreateChannelOnIOThread(). (Also simplify things a bit.) R=morrita@chromium.org Review URL: https://codereview.chromium.org/459613009 Cr-Commit-Position: refs/heads/master@{#289071} git-svn-id: svn://svn.chromium.org/chrome/trunk/src@289071 0039d316-1c4b-4281-b951-d872f2087c98
* GN: Make content_shell and webkit_unit_tests work in component buildjamesr@chromium.org2014-08-121-3/+4
| | | | | | | | | | | | Except for webrtc, this produces content_shell and webkit_unit_tests binaries that link and run successfully on linux. R=brettw@chromium.org Review URL: https://codereview.chromium.org/462983002 Cr-Commit-Position: refs/heads/master@{#289007} git-svn-id: svn://svn.chromium.org/chrome/trunk/src@289007 0039d316-1c4b-4281-b951-d872f2087c98
* IPC::ChannelMojo: Dup and own fds to transmitmorrita@chromium.org2014-08-091-2/+14
| | | | | | | | | | | | | | | | | IPC::ChannelMojo::Send() overly closed given handles as it took the lifecycle model incorrectly. This change properly delegates the ownership of the files by using dup(). TEST=browser_tests (with ChannelMojo enabled) BUG=none R=viettrungluu@chromium.org, darin@chromium.org Review URL: https://codereview.chromium.org/452293003 Cr-Commit-Position: refs/heads/master@{#288532} git-svn-id: svn://svn.chromium.org/chrome/trunk/src@288532 0039d316-1c4b-4281-b951-d872f2087c98
* IPC::ChannelMojo: Don't supress MOJO_RESULT_FAILED_PRECONDITIONmorrita@chromium.org2014-08-082-8/+16
| | | | | | | | | | | | | | | The error should be reported to IPC::Listener as it is a signal of the dead peer. Without this, the browser cannot detect renderer crashes. TEST=browser_tests (with ChannelMojo enabled) R=viettrungluu@chromium.org,darin@chromium.org BUG=none Review URL: https://codereview.chromium.org/453643003 Cr-Commit-Position: refs/heads/master@{#288442} git-svn-id: svn://svn.chromium.org/chrome/trunk/src@288442 0039d316-1c4b-4281-b951-d872f2087c98
* Add ipc/mojo/OWNERSmorrita@chromium.org2014-08-081-0/+2
| | | | | | | | | | | | | | Don't bother IPC owners with mojo changes. TEST=none BUG=none R=darin@chromium.org,jam@chromium.org NOTRY=true Review URL: https://codereview.chromium.org/451073002 Cr-Commit-Position: refs/heads/master@{#288395} git-svn-id: svn://svn.chromium.org/chrome/trunk/src@288395 0039d316-1c4b-4281-b951-d872f2087c98
* Move the rest of test_file_util to base namespace.brettw@chromium.org2014-08-072-2/+2
| | | | | | | | R=rvargas@chromium.org Review URL: https://codereview.chromium.org/444163003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@288158 0039d316-1c4b-4281-b951-d872f2087c98
* Introduce ChannelMojomorrita@chromium.org2014-08-0436-100/+1740
| | | | | | | | | | | | | | | | | | This CL introduces ChannelMojo IPC::Channel implementation and optionally applies it for renderer-browser IPC channel. Current stability is like 5-seconds browser and There are rough edges. It often closes the channel so needs to be more robust. Even though the level of stability, having it in the tree will helps team to try and improve it. BUG=377980 R=darin@chromium.org,jam@chromium.org,viettrungluu@chromium.org TEST=ipc_channel_mojo_unittest.cc Review URL: https://codereview.chromium.org/382333002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@287402 0039d316-1c4b-4281-b951-d872f2087c98
* IPC: Add more debug logic (crash tracking)rvargas@chromium.org2014-07-312-2/+12
| | | | | | | | | BUG=387876 R=cpu@chromium.org Review URL: https://codereview.chromium.org/427693004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@286759 0039d316-1c4b-4281-b951-d872f2087c98
* Refactor Web Notification permission requests.peter@chromium.org2014-07-291-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | As part of supporting Web Notifications in workers, only the ability to request permission will be done through a WebFrame, all other operations will be routed through Platform. This patch introduces the NotificationPermissionDispatcher, (lazy) member of RenderFrameImpl, handling that sole responsibility. The NPD will replace the existing NotificationProvider in its entirety. However, while moving functionality around, we need to maintain the existing code paths and thus use this opportunity to rename DesktopNotifications to a more generic PlatformNotifications, since they'll also be usable on Android. When a website currently requests permission to display Web Notifications, we show an infobar to the user, then send a message (without result!) to the renderer process, where a synchronous IPC to the browser process will fire in order to find out whether permission was actually granted. This patch includes whether permission was granted in the first message. BUG=392187, 398045 Review URL: https://codereview.chromium.org/381633005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@286321 0039d316-1c4b-4281-b951-d872f2087c98
* Make IPC macros work with -Wunused-local-typedefs.thakis@chromium.org2014-07-291-6/+6
| | | | | | | | | | | | | | | The typedef can be used by the IPC_MESSAGE_HANDLER macro, but that macro isn't always used when IPC_BEGIN_MESSAGE_MAP is used. The alternatives are to either pass ClassName::HanderFunction to IPC_MESSAGE_HANDLER (like what one has to do to use IPC_MESSAGE_FORWARD), or to mark the typedef in IPC_BEGIN_MESSAGE_MAP as ALLOW_UNUSED so that it isn't warned on. BUG=321833 TBR=brettw Review URL: https://codereview.chromium.org/422083002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@286115 0039d316-1c4b-4281-b951-d872f2087c98
* Improve content test GN buildbrettw@chromium.org2014-07-231-3/+3
| | | | | | | | | | | | | | Fix libxml clang warnings. Add ppapi_unittest_shared target. Rename ipc:test_support_ipc to ipc:test_support and use this target when needed (the places that wanted this incorrectly specified only "ipc". R=hclam@chromium.org Review URL: https://codereview.chromium.org/418453002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@285027 0039d316-1c4b-4281-b951-d872f2087c98
* GN Windows build fixesbrettw@chromium.org2014-07-221-1/+0
| | | | | | | | | | Reland of https://codereview.chromium.org/404603002/ but without the problematic extensions.gypi changes. R=agl@chromium.org, dalecurtis@chromium.org Review URL: https://codereview.chromium.org/400233002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@284703 0039d316-1c4b-4281-b951-d872f2087c98
* Rename IPC::ChannelFactory to UnixDomainSocketAcceptor.morrita@chromium.org2014-07-184-154/+1
| | | | | | | | | | | | | | ChannelFactory doesn't create any channel and it just listens on and accepts from a unix socket. So this change renames it to represent what it actually does. R=darin@chromium.org,cpu@chromium.org,jeremya@chromium.org,tapted@chromium.org TEST=none BUG=377980 Review URL: https://codereview.chromium.org/305973002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@284231 0039d316-1c4b-4281-b951-d872f2087c98
* Fixes for re-enabling more MSVC level 4 warnings: misc edition #2pkasting@chromium.org2014-07-181-15/+11
| | | | | | | | | | | | | | | | | | This contains fixes for the following sorts of issues: * Assignment inside conditional * Taking the address of a temporary * Octal escape sequence terminated by decimal number * Signedness mismatch * Possibly-uninitialized local variable This also contains a small number of cleanups to nearby code (e.g. no else after return). BUG=81439 TEST=none Review URL: https://codereview.chromium.org/382673002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@283967 0039d316-1c4b-4281-b951-d872f2087c98
* Remove unused IPC_MESSAGE_START enum constants.tsepez@chromium.org2014-07-011-6/+0
| | | | | | Review URL: https://codereview.chromium.org/358023002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@280728 0039d316-1c4b-4281-b951-d872f2087c98
* [DeviceLight] Add renderer+common partsrijubrata.bhaumik@intel.com2014-06-271-0/+1
| | | | | | | | | | | | | | | | | | | | | | Adding the Renderer and Common part in the content side for the DeviceLight API. Add Unit tests for deviceLight pump Add DeviceLight support in renderer_webkitplatformsupport_impl.h|cc The default polling time interval of DeviceLightEventPump is right now set as of 200 milliseconds. (device motion/orientation = 50ms) In Firefox (Android) implementation, DeviceLight's frequency of update is set as SensorManager.SENSOR_DELAY_NORMAL which in Android parlance translates to 200 milliseconds delay which is 5 times a second. Ofcourse the delay is only a suggested delay, system typically uses a smaller delay. BUG=336424 Review URL: https://codereview.chromium.org/286793002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@280306 0039d316-1c4b-4281-b951-d872f2087c98
* Extensions: Split extensions utility IPC messages into its own file.thestig@chromium.org2014-06-271-1/+2
| | | | | | | | | | Disable extensions code on Android so they do not use these messages. BUG=349436 Review URL: https://codereview.chromium.org/345693002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@280242 0039d316-1c4b-4281-b951-d872f2087c98
* IPC ChannelWin: Add temporary debug info.rvargas@chromium.org2014-06-272-3/+53
| | | | | | | | | | Add some state tracking info to the channel object while working on a crash. BUG=387876 Review URL: https://codereview.chromium.org/355943002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@280241 0039d316-1c4b-4281-b951-d872f2087c98
* The LOG initialization code would clear the last error. Use PLOG instead.weitaosu@chromium.org2014-06-261-3/+2
| | | | | | | | | | LOG(...) needs to perform log initialization when called for the first time. It may call Windows APIs that clears the last error in which case the message written to the log will contain the wrong last error code. PLOG does the right thing by getting the last error code before initialing the log file. BUG= Review URL: https://codereview.chromium.org/331883009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@280103 0039d316-1c4b-4281-b951-d872f2087c98
* ozone: Add GPU message for cursor updatesspang@chromium.org2014-06-261-0/+1
| | | | | | | | | | | | This message will be sent from the browser to the gpu move the hardware cursor around under some platforms (particularly gbm). BUG=377497 TEST=chrome --ozone-platform=gbm Review URL: https://codereview.chromium.org/348603004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@279935 0039d316-1c4b-4281-b951-d872f2087c98
* This patch defines three mechanisms for acquiring CLD2's data:andrewhayden@chromium.org2014-06-241-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 1. Static linking (the way it is done today) 2. Standalone dynamic data file bundled with the application 3. Dynamic data file downloaded via the Component Updater This change does NOT switch any implementation to a different cld data source. Android/iOS remain pinned to CLD1, and all platforms continue to statically link the data with cld2_data_source="static". This change has several important side effects: 1. The gyp variables "cld2_dynamic" and "cld2_is_component" have been removed. There is now a single variable, "cld2_data_source", that defines which implementation is to be used. 2. cld_component_installer.[h,cc] and cld_component_installer_unittest.cc are now conditionally built if and only if cld2_data_source=="component" and have direct dependencies upon components/translate/content/browser. 3. Almost all preprocesser checks for the CLD data source have been removed, greatly simplifying code flow. The logic previously gated by these checks has been split into separate implementation classes whose inclusion is controlled by the cld2_data_source gyp variable. For more information, refer to crbug/383769. BUG=383769 Review URL: https://codereview.chromium.org/333603002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@279415 0039d316-1c4b-4281-b951-d872f2087c98
* Support multiple files for AEC dump.grunell@chromium.org2014-06-181-0/+1
| | | | | | | | | | | | | | | * This adds support for AEC dump files in multiple tabs and if multiple getUserMedia calls are made in one tab. * Each AEC dump consumer registers with the browser to recieve a file handle when AEC dump is enabled. * Add dedicated aec dump message filter an messages file. * Works with and without track-processing. NOTRY=true BUG=350347 Review URL: https://codereview.chromium.org/334743006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@278148 0039d316-1c4b-4281-b951-d872f2087c98
* Fix "inlcude" typo in comments.tsepez@chromium.org2014-06-181-2/+2
| | | | | | | | NOTRY=true Review URL: https://codereview.chromium.org/343473006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@278113 0039d316-1c4b-4281-b951-d872f2087c98
* Split printing utility IPC messages into its own file.thestig@chromium.org2014-06-181-0/+1
| | | | | | Review URL: https://codereview.chromium.org/323693002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@278042 0039d316-1c4b-4281-b951-d872f2087c98
* Enable resource prefetch from the browser process.tburkard@chromium.org2014-06-171-0/+1
| | | | | | | | R=jam@chromium.org, jochen@chromium.org, jschuh@chromium.org Review URL: https://codereview.chromium.org/334483004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@277761 0039d316-1c4b-4281-b951-d872f2087c98
* Decouple IPC::MessageFilter from IPC::Channelmorrita@chromium.org2014-06-136-16/+16
| | | | | | | | | | | | | This change gets rid of Channel dependency from MessageFilter so that it depends only on IPC::Sender. TEST=none BUG=377980 R=jam@chromium.org, darin@chromium.org Review URL: https://codereview.chromium.org/324143002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@276939 0039d316-1c4b-4281-b951-d872f2087c98
* Build fix with gcc 4.8 and C++11 enabledch.dumez@samsung.com2014-06-101-1/+1
| | | | | | | | | | Build fix with gcc 4.8 and C++11 enabled. gcc was complaining because it requires 'override' to be specified *before" __attribute__ (WARN_UNUSED_RESULT in this case). Review URL: https://codereview.chromium.org/322833002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@276021 0039d316-1c4b-4281-b951-d872f2087c98
* Remove base::kInvalidPlatformFileValue from components, ipc and ppapi.rvargas@chromium.org2014-06-101-2/+1
| | | | | | | | | BUG=322664 TBR=mseaborn@chromium.org Review URL: https://codereview.chromium.org/316363003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@276003 0039d316-1c4b-4281-b951-d872f2087c98
* Remove cdn from OWNERS for IPC security review.palmer@chromium.org2014-06-081-1/+0
| | | | | | | | | | | Since he is no longer on the project. Sniff. :'( BUG=none R=jln@chromium.org Review URL: https://codereview.chromium.org/317243005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@275756 0039d316-1c4b-4281-b951-d872f2087c98
* IPC::Channel: Reduce POSIX specific API surfacemorrita@chromium.org2014-06-078-97/+55
| | | | | | | | | | | | | | | This change is a simple cleanup: It removes some Channel API that is only used only in POSIX specific tests. These funtions still live in ChannelPosix. POSIX speicific tests now touch ChannelPosix instead of its superclass Channel. TEST=none R=darin@chromium.org, jam@chromium.org BUG=377980 Review URL: https://codereview.chromium.org/320433002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@275632 0039d316-1c4b-4281-b951-d872f2087c98
* Make IPC::Channel polymorphicmorrita@chromium.org2014-06-0617-272/+257
| | | | | | | | | | | | | | This change makes each platform specific ChannelImpl into a subclass of Channel: ChannelPosix, ChannelWin, ChannelNacl. delegated functions are now virtual. TEST=none BUG=377980 R=darin@chromium.org, jam@chromium.org Review URL: https://codereview.chromium.org/310293002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@275505 0039d316-1c4b-4281-b951-d872f2087c98
generated by cgit v1.1 at 2025-02-15 07:21:15 +0000