Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(397)

Issue 460102: Recognize EMSGSIZE as non-fatal on OS X. (Closed)

Created:
11 years ago by jeremy
Modified:
9 years, 6 months ago
Reviewers:
TVL, agl
CC:
chromium-reviews_googlegroups.com, John Grabowski, Mark Mentovai
Visibility:
Public.

Description

Recognize EMSGSIZE as non-fatal on OS X. BUG=29225 TEST=PageCyclerTest.Intl2File test should succeed. Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=34056

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+15 lines, -9 lines) Patch
M chrome/test/page_cycler/page_cycler_test.cc View 1 chunk +2 lines, -1 line 0 comments Download
M ipc/ipc_channel_posix.cc View 3 chunks +13 lines, -8 lines 0 comments Download

Messages

Total messages: 5 (0 generated)
jeremy
* Also a small fix to page_cycler_tests. I couldn't figure out how to write a ...
11 years ago (2009-12-06 13:31:39 UTC) #1
TVL
Does this mean we spin like a top until the other side reads enough to ...
11 years ago (2009-12-07 00:45:08 UTC) #2
jeremy
The underlying issue is that EMSGSIZE can be returned if the sockets output buffer has ...
11 years ago (2009-12-07 09:01:08 UTC) #3
TVL
My concern is even if we don't spin a tight loop here, are we gonna ...
11 years ago (2009-12-07 14:58:39 UTC) #4
agl
11 years ago (2009-12-07 17:42:27 UTC) #5
This will spin the message loop while it remains the case that there is some
space in the send buffer, but not enough. However, the renderer process handles
messages on a different thread from rendering so hopefully this situation
doesn't persist for long. (At least it did last time I checked.)

Given that I don't have a good solution and that this situation is rare then
it's LGTM.

(Although you should probably add a comment to this effect somewhere in the code
before submitting.)

Powered by Google App Engine
This is Rietveld 408576698