summaryrefslogtreecommitdiffstats
path: root/ipc
Commit message (Collapse)AuthorAgeFilesLines
* Componentize renderer side of DNS prefetching.gunsch2014-11-181-0/+1
| | | | | | | | BUG=419909 Review URL: https://codereview.chromium.org/644123002 Cr-Commit-Position: refs/heads/master@{#304529}
* IPC: Remove debugging code.rvargas2014-11-142-78/+5
| | | | | | | | | | | This is not needed anymore. BUG=387876 R=cpu@chromium.org Review URL: https://codereview.chromium.org/717193004 Cr-Commit-Position: refs/heads/master@{#304249}
* IPC: a way for security exploit browsertests to simulate the appearance of a ↵nick2014-11-144-2/+69
| | | | | | | | | | | | | | | malicious IPC. Use this to add a SecurityExploitBrowserTest for http://crbug.com/429922 Fix SecurityExploitBrowserTests on Android. Re-enable SecurityExploitBrowserTests on Android, except for two issues (1) the new test, which is actually disabled because of http://crbug.com/432737, discovered while developing this CL and (2) SetWebUIProperty, which is disabled because of http://crbug.com/433068, also discovered while developing this CL. Moral of the story being: never try. BUG=429922,432737,338023,433068 TEST=content_browsertests Review URL: https://codereview.chromium.org/712713002 Cr-Commit-Position: refs/heads/master@{#304170}
* IPC::ChannelMojo: Make IPC handling robust against bad messages.morrita2014-11-132-2/+16
| | | | | | | | | | | | This change replaces some DCHECK()s with real error handling. They happen when it receives broken messages. BUG=428800 R=viettrungluu@chromium.org Review URL: https://codereview.chromium.org/725733002 Cr-Commit-Position: refs/heads/master@{#304102}
* Fix signed integer overflow in GetRefNumUpper24.rickyz2014-11-121-2/+2
| | | | | | Review URL: https://codereview.chromium.org/717793005 Cr-Commit-Position: refs/heads/master@{#303777}
* Update mojo sdk to rev 04a510fb37db10642e156957f9b2c11c2f6442acjamesr2014-11-072-12/+1
| | | | | | | | | | | | | | | This updates the mojo sdk and updates chromium code to match. The most significant change is updating references to the mojo JS bindings code, which reshuffled. This part was reviewed separately by hansmuller@ in https://codereview.chromium.org/693343003/. This also updates ipc/mojo to reflect changes in ChannelInfo. R=morrita@chromium.org for ipc/mojo R=jam@chromium.org for top-level approval of the rest Review URL: https://codereview.chromium.org/703273002 Cr-Commit-Position: refs/heads/master@{#303149}
* Cleanup: Replace base::ASCIIToWide with base::ASCIIToUTF16.thestig2014-11-071-2/+2
| | | | | | | | BUG=23581 Review URL: https://codereview.chromium.org/689063002 Cr-Commit-Position: refs/heads/master@{#303142}
* Non-SFI mode: Enable MSG_TRUNC and MSG_CTRUNC code for nacl_helper_nonsfi.hidehiko2014-11-061-4/+0
| | | | | | | | | | | | Now, the PNaCL toolchain for Non-SFI build supports MSG_TRUNC and MSG_CTRUNC, so this CL enables the related code. BUG=427625, 358465 TEST=Ran trybots. Review URL: https://codereview.chromium.org/703033003 Cr-Commit-Position: refs/heads/master@{#303065}
* Non-SFI mode: Clean up macros of base/ and ipc/ libraries for ↵hidehiko2014-11-051-9/+9
| | | | | | | | | | | | | nacl_helper_nonsfi. Now, OS_NACL_SFI and OS_NACL_NONSFI are introduced to build/build_config.h, and these are preferred to __native_client_nonsfi__ and OS_NACL combination. BUG=358465 TEST=Ran trybots. Review URL: https://codereview.chromium.org/704513003 Cr-Commit-Position: refs/heads/master@{#302907}
* Update mojo sdk to rev cfc99316100efdfa7d53d83f9e07f1d4d3765c21jamesr2014-11-041-3/+1
| | | | | | | | | | | Most changes are updates to the mojom syntax for modules. R=viettrungluu@chromium.org TBR=jam@chromium.org Review URL: https://codereview.chromium.org/698883005 Cr-Commit-Position: refs/heads/master@{#302542}
* Revert of Enable ChannelMojo on Linux (patchset #1 id:1 of ↵yurys2014-10-301-4/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | https://codereview.chromium.org/684323002/) Reason for revert: Caused multiple layout test crashes on Linux See http://test-results.appspot.com/dashboards/flakiness_dashboard.html#tests=css3%2Ffilters%2Feffect-reference-subregion-hidpi-hw.html%2Ccss3%2Ffilters%2Feffect-reference-subregion-hidpi.html%2Ccss3%2Fimages%2Fpixelated-hidpi.html%2Cfast%2Fhidpi%2Fimage-set-as-background.html%2Cfast%2Fhidpi%2Fimage-set-background-dynamic.html%2Cfast%2Fhidpi%2Fimage-set-background-repeat-without-size.html%2Cfast%2Fhidpi%2Fimage-set-background-repeat.html%2Cfast%2Fhidpi%2Fimage-set-border-image-comparison.html%2Cfast%2Fhidpi%2Fimage-set-border-image-dynamic.html%2Cfast%2Fhidpi%2Fimage-set-border-image-simple.html%2Cfast%2Fhidpi%2Fimage-set-in-content-dynamic.html%2Cfast%2Fhidpi%2Fimage-set-out-of-order.html%2Cfast%2Fhidpi%2Fimage-set-simple.html%2Cfast%2Fhidpi%2Fimage-set-without-specified-width.html%2Cfast%2Fhistory%2Freplacestate-nocrash.html%2Chttp%2Ftests%2Finspector%2Fnetwork%2Flong-script-content.html%2Chttp%2Ftests%2Fxmlhttprequest%2Fsimple-cross-origin-progress-events.html%2Csvg%2Fas-image%2Fsvg-as-image-canvas.html%2Csvg%2Fcss%2Fsvg-resource-fragment-identifier-img-src.html Regressions: Unexpected crashes (19) css3/filters/effect-reference-subregion-hidpi-hw.html [ Crash ] css3/filters/effect-reference-subregion-hidpi.html [ Crash ] css3/images/pixelated-hidpi.html [ Crash ] fast/hidpi/image-set-as-background.html [ Crash ] fast/hidpi/image-set-background-dynamic.html [ Crash ] fast/hidpi/image-set-background-repeat-without-size.html [ Crash ] fast/hidpi/image-set-background-repeat.html [ Crash ] fast/hidpi/image-set-border-image-comparison.html [ Crash ] fast/hidpi/image-set-border-image-dynamic.html [ Crash ] fast/hidpi/image-set-border-image-simple.html [ Crash ] fast/hidpi/image-set-in-content-dynamic.html [ Crash ] fast/hidpi/image-set-out-of-order.html [ Crash ] fast/hidpi/image-set-simple.html [ Crash ] fast/hidpi/image-set-without-specified-width.html [ Crash ] fast/history/replacestate-nocrash.html [ Crash ] http/tests/inspector/network/long-script-content.html [ Crash ] http/tests/xmlhttprequest/simple-cross-origin-progress-events.html [ Crash ] svg/as-image/svg-as-image-canvas.html [ Crash ] svg/css/svg-resource-fragment-identifier-img-src.html [ Crash ] Original issue's description: > Enable ChannelMojo on Linux > > This change enables ChannelMojo on Linux. > There could possibly be a perf slowdown, especially > on latency metrics. > > If you're looking for the cause of some perf bot redness, > probably this is that one. Please feel free to revert and > let me know which benchmark was hit. > > BUG=664413003 > R=viettrungluu@chromium.org > > Committed: https://crrev.com/d4d85c640375ea7f9692e53860f4de058a7f0ea1 > Cr-Commit-Position: refs/heads/master@{#301992} TBR=viettrungluu@chromium.org,morrita@chromium.org NOTREECHECKS=true NOTRY=true BUG=664413003 Review URL: https://codereview.chromium.org/673903005 Cr-Commit-Position: refs/heads/master@{#302055}
* Enable ChannelMojo on Linuxmorrita2014-10-301-0/+4
| | | | | | | | | | | | | | | | | This change enables ChannelMojo on Linux. There could possibly be a perf slowdown, especially on latency metrics. If you're looking for the cause of some perf bot redness, probably this is that one. Please feel free to revert and let me know which benchmark was hit. BUG=664413003 R=viettrungluu@chromium.org Review URL: https://codereview.chromium.org/684323002 Cr-Commit-Position: refs/heads/master@{#301992}
* Remove last non-third_party typeofs on Mac.thakis2014-10-291-9/+1
| | | | | | | | BUG=427584 Review URL: https://codereview.chromium.org/684273002 Cr-Commit-Position: refs/heads/master@{#301948}
* Add a few missing overrides found by a new clang warning.thakis2014-10-292-2/+3
| | | | | | | | | | Namely, -Winconsistent-missing-override. No behavior change. BUG=428099 Review URL: https://codereview.chromium.org/687693004 Cr-Commit-Position: refs/heads/master@{#301746}
* Introduce OS_NACL_SFI and OS_NACL_NONSFI macros.hidehiko2014-10-283-7/+4
| | | | | | | | | | | | | | | | | | PNaCl toolchain starts to support "Non-SFI build binary" mode. Some sources are built both for SFI binary and for Non-SFI binary, but, in some cases, there are small diffs which need to be guarded by #ifdef. In "Non-SFI build binary" mode, __native_client_nonsfi__ is defined. This CL introduce OS_NACL_SFI and OS_NACL_NONSFI which can be used among the chrome code, to slightly simplify the #ifdef conditions. This CL is the follow up of crrev.com/659243002 BUG=358465 TEST=Ran trybot. Review URL: https://codereview.chromium.org/664373003 Cr-Commit-Position: refs/heads/master@{#301603}
* ChannelMojo: Replace hand written messsages with mojo.morrita2014-10-276-270/+170
| | | | | | | | | | | | | | | | | This change introduces client_channel.mojom to define internal messages for ChannelMojo. Hand-written HelloMessage routines are no longer needed. This is a preparation for coming fix which needs additional internal messages. TEST=ipc_mojo_unittest R=viettrungluu@chromium.org BUG=377980 Review URL: https://codereview.chromium.org/679453002 Cr-Commit-Position: refs/heads/master@{#301428}
* Move LayoutTestMessageFilter-related IPCs to their own message file.peter2014-10-271-0/+1
| | | | | | | | | | | | This does not yet contain all layout-test only IPC messages, but moves those used in LayoutTestMessageFilter to a separate message file. As refactoring continues, more IPCs will move there from the ShellMessageFilter. BUG=420994 Review URL: https://codereview.chromium.org/677673002 Cr-Commit-Position: refs/heads/master@{#301392}
* Non-SFI Mode: Build ipc/ library by PNaCl toolchain for nacl_helper_nonsfi.hidehiko2014-10-246-13/+82
| | | | | | | | | | | | | | | | | | This CL is to build ipc/ for nacl_helper_nonsfi. The library is similar to ipc_nacl, but slightly different: - The IPC::Channel should use ChannelPosix rather than ChannelNaCl, as it runs under linux directly. - Some features of ChannelPosix cannot be compiled by PNaCl toolchain for Non-SFI build, but these are not necessary for nacl_helper_nonsfi. These are dropped by "ifdef" guard. Note that this library is not used yet, but should be built successfully. BUG=358465 TEST=Ran trybot. Implement nacl_helper_nonsfi on top of this CL, and made sure it works. Review URL: https://codereview.chromium.org/659243002 Cr-Commit-Position: refs/heads/master@{#301037}
* Make IPC_ENUM_TRAITS_MIN_MAX_VALUE() work with c++11 enum class.tsepez2014-10-231-1/+4
| | | | | | | | | | It is a matter of adding explicit casts to make legal comparisions. BUG=425190 Review URL: https://codereview.chromium.org/669713003 Cr-Commit-Position: refs/heads/master@{#300826}
* Standardize usage of virtual/override/final in ipc/dcheng2014-10-2230-208/+197
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The Google C++ style guide states: Explicitly annotate overrides of virtual functions or virtual destructors with an override or (less frequently) final specifier. Older (pre-C++11) code will use the virtual keyword as an inferior alternative annotation. For clarity, use exactly one of override, final, or virtual when declaring an override. To better conform to these guidelines, the following constructs have been rewritten: - if a base class has a virtual destructor, then: virtual ~Foo(); -> ~Foo() override; - virtual void Foo() override; -> void Foo() override; - virtual void Foo() override final; -> void Foo() final; This patch was automatically generated. The clang plugin can generate fixit hints, which are suggested edits when it is 100% sure it knows how to fix a problem. The hints from the clang plugin were applied to the source tree using the tool in https://codereview.chromium.org/598073004. Several formatting edits by clang-format were manually reverted, due to mangling of some of the more complicate IPC macros. BUG=417463 R=agl@chromium.org Review URL: https://codereview.chromium.org/666493005 Cr-Commit-Position: refs/heads/master@{#300623}
* Add a guard in MojoBootstrap::OnClientLaunched()morrita2014-10-222-0/+8
| | | | | | | | | | | | | | There is a race where the IO thread got error after the UI thread launched a process. This CL rejects such a case. This is OK as the error is eventually notified to the UI thread. TEST=chromeos browser_tests with ChannelMojo being on. R=viettrungluu@chromium.org BUG=377980 Review URL: https://codereview.chromium.org/671513011 Cr-Commit-Position: refs/heads/master@{#300589}
* Revert of Standardize usage of virtual/override/final in ipc/ (patchset #1 ↵dcheng2014-10-2130-241/+275
| | | | | | | | | | | | | | | | | | | | | | | | | | | | id:1 of https://codereview.chromium.org/645623006/) Reason for revert: Addressing reviewer comments in a followup patch. Original issue's description: > Standardize usage of virtual/override/final in ipc/ > > This patch was automatically generated by applying clang fixit hints > generated by the plugin to the source tree. > > BUG=417463 > TBR=agl@chromium.org > > Committed: https://crrev.com/39be52ebbd6681c7d79ad63e2a0ab028075af0c2 > Cr-Commit-Position: refs/heads/master@{#300481} TBR=agl@chromium.org NOTREECHECKS=true NOTRY=true BUG=417463 Review URL: https://codereview.chromium.org/669953003 Cr-Commit-Position: refs/heads/master@{#300521}
* Standardize usage of virtual/override/final in ipc/dcheng2014-10-2130-275/+241
| | | | | | | | | | | | This patch was automatically generated by applying clang fixit hints generated by the plugin to the source tree. BUG=417463 TBR=agl@chromium.org Review URL: https://codereview.chromium.org/645623006 Cr-Commit-Position: refs/heads/master@{#300481}
* ChannelMojo: Add ChannelMojo::CanBeUsed() to enable it incrementallymorrita2014-10-212-0/+9
| | | | | | | | | | | | | ChannelMojo::CanBeUsed() let the code enable the feature for specific platform regardless of the command line switch. TEST=none R=viettrungluu@chromium.org BUG=377980 Review URL: https://codereview.chromium.org/665993002 Cr-Commit-Position: refs/heads/master@{#300394}
* Refactoring: Make IPC::Channel::TakeClientFileDescriptor() a ScopedFD.morrita2014-10-2013-23/+21
| | | | | | | | | | | | | By using base::ScopedFD, the ownership of the returned FD becomes clearer. BUG=415294 R=brettw@chromium.org, jam@chromium.org, yzshen@chromium.org, sehr@chromium.org TBR=gene@chromium.org Review URL: https://codereview.chromium.org/621613002 Cr-Commit-Position: refs/heads/master@{#300381}
* Use scoped_ptr::Pass instead of scoped_ptr::PassAs<T>.dcheng2014-10-172-9/+5
| | | | | | | | BUG=423621 Review URL: https://codereview.chromium.org/660913002 Cr-Commit-Position: refs/heads/master@{#300030}
* Modify ALLOW_UNUSED to allow enabling unused local warnings on MSVC.pkasting2014-10-161-6/+6
| | | | | | | | | | | | | | | | | | | | | | This splits the macro into two: int a = 1; ALLOW_UNUSED_LOCAL(a); and typedef Foo Bar ALLOW_UNUSED_TYPE; void foo() ALLOW_UNUSED_TYPE; // ALLOW_UNUSED_TYPE_OR_FUNC seemed too verbose This matches changes that have already been made in Blink. BUG=81439 TEST=none TBR=ben Review URL: https://codereview.chromium.org/650393002 Cr-Commit-Position: refs/heads/master@{#300014}
* MojoChannel: Fail when GetFileHandleForProcess() failed.morrita2014-10-153-4/+77
| | | | | | | | | | | | On Windows, GetFileHandleForProcess() can fail. This CL handles that case instead of just doing CHECK(). R=viettrungluu@chromium.org BUG=419139 Review URL: https://codereview.chromium.org/654113002 Cr-Commit-Position: refs/heads/master@{#299731}
* ipc_mojo_bootstrap.cc: don't export classes in anonymous namespacehans2014-10-151-2/+2
| | | | | | | | | | | | It doesn't make sense since the classes cannot be referred to outside the translation unit, and Clang on Windows gets upset about putting dllexport on symbols with internal linkage. BUG=82385 Review URL: https://codereview.chromium.org/655983002 Cr-Commit-Position: refs/heads/master@{#299718}
* Fix WeakPtrFactory member placementanujk.sharma2014-10-152-5/+5
| | | | | | | | | | | | | Changing in the intialization order of WeakPtrFactory in src/ipc module such that all member variables should appear before the WeakPtrFactory to ensure that any WeakPtrs to Controller are invalidated before its members variable's destructors are executed, rendering them invalid. BUG=303818 Review URL: https://codereview.chromium.org/645123004 Cr-Commit-Position: refs/heads/master@{#299660}
* Misc. cleanup, primarily removing unused locals.pkasting2014-10-132-4/+2
| | | | | | | | | | | Also various other fixes, e.g. condensing code, converting DCHECK_LT(0, a) -> DCHECK_GT(a, 0) (and the like) for readability, inserting a few typecasts. BUG=none TEST=none Review URL: https://codereview.chromium.org/637023002 Cr-Commit-Position: refs/heads/master@{#299362}
* Add mkwst@chromium.org to IPC OWNERS files.mkwst2014-10-091-0/+1
| | | | | | | | | NOTRY=true TBR=darin@chromium.org Review URL: https://codereview.chromium.org/636203003 Cr-Commit-Position: refs/heads/master@{#298800}
* Move mojo/edk gyp targets in to mojo/edk/mojo_edk.gypjamesr2014-10-071-3/+3
| | | | | | | | | R=viettrungluu@chromium.org TBR=darin@chromium.org for updates to various gyp files to point to moved targets Review URL: https://codereview.chromium.org/596053004 Cr-Commit-Position: refs/heads/master@{#298560}
* Define gyp targets for mojo/public/* in mojo/public/mojo_public.gypjamesr2014-10-071-3/+3
| | | | | | | | | | | | | | | The targets in the mojo public SDK, i.e. those in mojo/public/, should be free of dependencies outside of mojo/public/ and usable in a standalone fashion. This moves the gyp definitions from mojo/mojo_base.gyp, which also contains a number of non-public targets, to mojo/public/mojo_public.gyp. R=viettrungluu@chromium.org TBR=darin@chromium.org for updating various gyp entries to point to new target location Review URL: https://codereview.chromium.org/632853002 Cr-Commit-Position: refs/heads/master@{#298422}
* replace OVERRIDE and FINAL with override and final in ipc/mostynb2014-10-0730-187/+187
| | | | | | | | BUG=417463 Review URL: https://codereview.chromium.org/633563002 Cr-Commit-Position: refs/heads/master@{#298413}
* Move mojo edk into mojo/edkjamesr2014-10-039-12/+12
| | | | | | | | | | | | | | | | | | | | This creates a mojo/edk directory which contains the "embedder developer kit" aka the set of code needed to embed mojo code. mojo/edk/embedder = code from mojo/embedder mojo/edk/system = code from mojo/system mojo/edk/test = code used to test the previous two, from mojo/common/test mojo/edk/ can only depend on mojo/public/, base/ and itself. R=viettrungluu@chromium.org TBR=sky@chromium.org for file renames Committed: https://chromium.googlesource.com/chromium/src/+/ee7ff197a98da4636f33bd713de784948b487bd4 Review URL: https://codereview.chromium.org/621153003 Cr-Commit-Position: refs/heads/master@{#297986}
* Revert "Move mojo edk into mojo/edk"Trent Apted2014-10-039-12/+12
| | | | | | | | | | | | | | | This reverts commit ee7ff197a98da4636f33bd713de784948b487bd4. Causing widespread checkdeps failures. ERROR in src\content\browser\webui\web_ui_mojo_browsertest.cc Illegal include: "mojo/edk/test/test_utils.h" Because of no rule applying. TBR=jamesr@chromium.org Review URL: https://codereview.chromium.org/623883002 Cr-Commit-Position: refs/heads/master@{#297966}
* Move mojo edk into mojo/edkJames Robinson2014-10-039-12/+12
| | | | | | | | | | | | | | | | | This creates a mojo/edk directory which contains the "embedder developer kit" aka the set of code needed to embed mojo code. mojo/edk/embedder = code from mojo/embedder mojo/edk/system = code from mojo/system mojo/edk/test = code used to test the previous two, from mojo/common/test mojo/edk/ can only depend on mojo/public/, base/ and itself. R=viettrungluu@chromium.org Review URL: https://codereview.chromium.org/621153003 Cr-Commit-Position: refs/heads/master@{#297958}
* Pass through geofencing API calls to the browser process.mek2014-10-021-0/+1
| | | | | | | | BUG=383125 Review URL: https://codereview.chromium.org/476293002 Cr-Commit-Position: refs/heads/master@{#297892}
* Refactoring: Let ChannelPosix adopt ScopedFD.morrita2014-09-292-94/+91
| | | | | | | | | | | | | | This gets rid of raw close() call usign base::ScopedFD. Ownership of FDs become clearer. This is a preparation for kiling base::FileDescriptor. R=jam@chromium.org, agl@chromium.org BUG=415294 Review URL: https://codereview.chromium.org/602193004 Cr-Commit-Position: refs/heads/master@{#297285}
* IPC: Use ScopedHandle instead of a raw HANDLE for the private members.rvargas2014-09-262-39/+42
| | | | | | | | | BUG=387876 R=cpu@chromium.org Review URL: https://codereview.chromium.org/588383003 Cr-Commit-Position: refs/heads/master@{#297011}
* ChannelMojo: Handle when ChannelMojo outlives ChannelMojoHostmorrita2014-09-266-57/+147
| | | | | | | | | | | | | | | | | | | In some case ChannelMojo outlives ChannelMojoHost because two objects are living in diffent thread. Instead of using lifecycle callbacks, this CL relies on WeakPtr. See comment on ipc_channel_mojo_host.h for more details. This CL also fixes a crash on --single-process mode. R=viettrungluu@chromium.org TBR=jam@chromium.org TEST=content_browsertests (with --enable-renderer-mojo-channel on) BUG=377980 Review URL: https://codereview.chromium.org/599333002 Cr-Commit-Position: refs/heads/master@{#296871}
* Remove IPC::Channel::TakePipeHandle()morrita2014-09-2511-38/+0
| | | | | | | | | | | It doesn't work, and it is no longer used. BUG=416208 R=cpu@chromium.org, jam@chromium.org Review URL: https://codereview.chromium.org/601563002 Cr-Commit-Position: refs/heads/master@{#296820}
* Adding tracking to verify the theory that jankiness of ↵vadimt2014-09-251-4/+4
| | | | | | | | | | ExtensionMessageFilter::OnExtensionAddListener is because of the observer invocation. BUG=401560, 417106 Review URL: https://codereview.chromium.org/600213002 Cr-Commit-Position: refs/heads/master@{#296804}
* Fix IPCChannelMojoErrorTest.SendFailWithPendingMessagesmorrita2014-09-241-46/+11
| | | | | | | | | | | | | | | | The test has been broken since MojoBootstrap was introduced. The change makes it impossible to inject failure from the ChannelMojo subclass. This change uses large data as pending message to trigger failure, instead of overloading IPC::ChannelMojo functions. R=viettrungluu@chromium.org BUG=417439 Review URL: https://codereview.chromium.org/598093006 Cr-Commit-Position: refs/heads/master@{#296565}
* Disable IPCChannelMojoErrorTest.SendFailWithPendingMessagesmorrita2014-09-241-1/+2
| | | | | | | | | | | The test is flaky. Disabling it while investigating the cause. BUG=417439 R=jam@chromium.org Review URL: https://codereview.chromium.org/597003006 Cr-Commit-Position: refs/heads/master@{#296533}
* Move SafeManifestParser to //extensionsrockot2014-09-241-0/+1
| | | | | | | | | | | | | This moves SafeManifestParser to extensions_browser. Also creates a new ExtensionUtility IPC message class in which to define extensions utility process messages. The relevant manifest parsing messages have been moved into this class. BUG=398671 Review URL: https://codereview.chromium.org/464613002 Cr-Commit-Position: refs/heads/master@{#296508}
* IPC: Get rid of FileDescriptor usage from FileDescriptorSet and Messagemorrita2014-09-2412-153/+191
| | | | | | | | | | | | | | | This is a step toward to killing FileDescriptor. This change lets FiileDescriptorSet have both Files (for owning fds) and PlatformFiles (for non-owning fds). Doing this, we no longer need FileDescriptor which provides |auto_close| flag. BUG=415294 TEST=ipc_tests, ipc_mojo_unittests R=agl@chromium.org, jam@hcromium.org, viettrungluu@chromium.org Review URL: https://codereview.chromium.org/583473002 Cr-Commit-Position: refs/heads/master@{#296498}
* Make ServiceWorkerFetchRequest and ServiceWorkerResponse header maps case ↵jkarlin2014-09-241-3/+3
| | | | | | | | | | | | insensitive When storing headers in a map the map should be case insensitive (like HTTPHeaderMap in Blink) so that headers can be easily searched for and coalesced when necessary. BUG=392621 Review URL: https://codereview.chromium.org/576973004 Cr-Commit-Position: refs/heads/master@{#296387}
* IPC::ChannelMojo: Introduce IPC::MojoBootstrap for Windowsmorrita2014-09-2313-134/+675
| | | | | | | | | | | | | | | | | | | | | | | | | | | | ChannelMojo doesn't work on Windows with existing implementaion and this CL fixes it. On Windows, ChannelHandle isn't immediately usable: The handle has to be activated through ConnectNamedPipe() windows API, which is done in its own Connect() handlshaking phase. ChannelMojo didn't Connect() underlying channel and took the ChannelHandle over so the handle wasn't activated. Instead of hijacking underlying ChannelHandle, this CL actually Connect()s underlying channel, creates a pipe on the server side, send one side of the pipe to the client process, and use the pipe for the MessagePipe initialization. These initialization task is encapsulated behind new MojoBootstrap class. ChannelMojo creates MojoBootstrap class to get the PlatformHandle which is already activated and usable. BUG=377980 TEST=ipc_mojo_bootstrap_unittest.cc, ipc_channel_mojo_unittest.cc R=viettrungluu@chromium.org, darin@chromium.org, yzshen@chromium.org Review URL: https://codereview.chromium.org/553283002 Cr-Commit-Position: refs/heads/master@{#296248}
generated by cgit v1.1 at 2025-02-13 23:55:38 +0000