Closed Bug 913953 Opened 6 years ago Closed 6 years ago

Remove more dead Chromium code

Categories

(Core :: IPC, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla26

People

(Reporter: Ms2ger, Assigned: Ms2ger)

References

Details

Attachments

(28 files)

5.32 KB, patch
ehsan
: review+
Details | Diff | Splinter Review
2.56 KB, patch
ehsan
: review+
Details | Diff | Splinter Review
1.98 KB, patch
ehsan
: review+
Details | Diff | Splinter Review
5.56 KB, patch
ehsan
: review+
Details | Diff | Splinter Review
3.79 KB, patch
ehsan
: review+
Details | Diff | Splinter Review
9.35 KB, patch
ehsan
: review+
Details | Diff | Splinter Review
3.32 KB, patch
ehsan
: review+
Details | Diff | Splinter Review
11.18 KB, patch
ehsan
: review+
Details | Diff | Splinter Review
6.69 KB, patch
ehsan
: review+
Details | Diff | Splinter Review
5.43 KB, patch
ehsan
: review+
Details | Diff | Splinter Review
4.41 KB, patch
ehsan
: review+
Details | Diff | Splinter Review
1.46 KB, patch
ehsan
: review+
Details | Diff | Splinter Review
2.75 KB, patch
ehsan
: review+
Details | Diff | Splinter Review
2.21 KB, patch
ehsan
: review+
Details | Diff | Splinter Review
2.24 KB, patch
ehsan
: review+
Details | Diff | Splinter Review
2.14 KB, patch
ehsan
: review+
Details | Diff | Splinter Review
1.29 KB, patch
ehsan
: review+
Details | Diff | Splinter Review
2.72 KB, patch
ehsan
: review+
Details | Diff | Splinter Review
2.62 KB, patch
ehsan
: review+
Details | Diff | Splinter Review
4.72 KB, patch
ehsan
: review+
Details | Diff | Splinter Review
2.42 KB, patch
ehsan
: review+
Details | Diff | Splinter Review
4.44 KB, patch
ehsan
: review+
Details | Diff | Splinter Review
3.49 KB, patch
ehsan
: review+
Details | Diff | Splinter Review
2.37 KB, patch
ehsan
: review+
Details | Diff | Splinter Review
3.99 KB, patch
ehsan
: review+
Details | Diff | Splinter Review
2.11 KB, patch
ehsan
: review+
Details | Diff | Splinter Review
2.40 KB, patch
ehsan
: review+
Details | Diff | Splinter Review
2.48 KB, patch
ehsan
: review+
Details | Diff | Splinter Review
No description provided.
 ipc/chromium/moz.build                              |    1 -
 ipc/chromium/src/base/foundation_utils_mac.h        |   37 ---------
 ipc/chromium/src/base/mac_util.h                    |   24 ------
 ipc/chromium/src/base/mac_util.mm                   |   35 ---------
 ipc/chromium/src/base/process.h                     |   14 ---
 ipc/chromium/src/base/process_posix.cc              |   10 --
 ipc/chromium/src/base/process_util.h                |  110 ----------------------------
 ipc/chromium/src/base/process_util_posix.cc         |  155 ---------------------------------------
 ipc/chromium/src/base/process_util_win.cc           |  308 -------------------------------------------------------------------------------
 ipc/chromium/src/base/process_win.cc                |   63 ----------------
 ipc/chromium/src/base/string_escape.cc              |   98 -------------------------
 ipc/chromium/src/base/string_escape.h               |   35 ---------
 ipc/chromium/src/base/string_util.h                 |   31 -------
 ipc/chromium/src/base/sys_string_conversions.h      |   38 ---------
 ipc/chromium/src/base/sys_string_conversions_mac.mm |   52 -------------
 ipc/chromium/src/base/win_util.cc                   |  346 -----------------------------------------------------------------------------------------
 ipc/chromium/src/base/win_util.h                    |   71 ------------------
 ipc/chromium/src/chrome/common/child_process_info.h |    1 -
 18 files changed, 0 insertions(+), 1429 deletions(-)

https://tbpl.mozilla.org/?tree=Try&rev=9b46a16ece55
Attachment #801434 - Flags: review?(ehsan)
Feel free to review anything here too.
Flags: needinfo?(bent.mozilla)
IMO these don't need to be reviewed. rs=me for removing anything in ipc/chromium that we don't need (the only two requirements are that it builds and passes tests)!
Flags: needinfo?(bent.mozilla)
Attachment #801432 - Flags: review?(ehsan) → review+
Attachment #801433 - Flags: review?(ehsan) → review+
Attachment #801434 - Flags: review?(ehsan) → review+
Attachment #801435 - Flags: review?(ehsan) → review+
Attachment #801436 - Flags: review?(ehsan) → review+
Attachment #801437 - Flags: review?(ehsan) → review+
Attachment #801439 - Flags: review?(ehsan) → review+
Attachment #801440 - Flags: review?(ehsan) → review+
Attachment #801441 - Flags: review?(ehsan) → review+
Attachment #801442 - Flags: review?(ehsan) → review+
(In reply to ben turner [:bent] (needinfo? encouraged) from comment #31)
> IMO these don't need to be reviewed. rs=me for removing anything in
> ipc/chromium that we don't need (the only two requirements are that it
> builds and passes tests)!

What Ben said in the future!
Attachment #801443 - Flags: review?(ehsan) → review+
Attachment #801444 - Flags: review?(ehsan) → review+
Attachment #801445 - Flags: review?(ehsan) → review+
Attachment #801446 - Flags: review?(ehsan) → review+
Attachment #801447 - Flags: review?(ehsan) → review+
Attachment #801448 - Flags: review?(ehsan) → review+
Attachment #801449 - Flags: review?(ehsan) → review+
Attachment #801451 - Flags: review?(ehsan) → review+
Attachment #801452 - Flags: review?(ehsan) → review+
Attachment #801453 - Flags: review?(ehsan) → review+
Attachment #801454 - Flags: review?(ehsan) → review+
Attachment #801455 - Flags: review?(ehsan) → review+
Attachment #801456 - Flags: review?(ehsan) → review+
Attachment #801450 - Flags: review?(ehsan) → review+
Attachment #801457 - Flags: review?(ehsan) → review+
Attachment #801458 - Flags: review?(ehsan) → review+
Attachment #801459 - Flags: review?(ehsan) → review+
Attachment #801460 - Flags: review?(ehsan) → review+
https://hg.mozilla.org/mozilla-central/rev/e59f853e6c26
https://hg.mozilla.org/mozilla-central/rev/3d5ec64ea71d
https://hg.mozilla.org/mozilla-central/rev/e6eadf6177a9
https://hg.mozilla.org/mozilla-central/rev/5316beb6adb0
https://hg.mozilla.org/mozilla-central/rev/a5f01ace41b1
https://hg.mozilla.org/mozilla-central/rev/22000376331f
https://hg.mozilla.org/mozilla-central/rev/22c979082c03
https://hg.mozilla.org/mozilla-central/rev/02d635ad7b7c
https://hg.mozilla.org/mozilla-central/rev/2e9624b9aec1
https://hg.mozilla.org/mozilla-central/rev/c00ba1829ba4
https://hg.mozilla.org/mozilla-central/rev/734c93da9910
https://hg.mozilla.org/mozilla-central/rev/d5040d300dba
https://hg.mozilla.org/mozilla-central/rev/af2f22246cd7
https://hg.mozilla.org/mozilla-central/rev/7eaf68ee0c25
https://hg.mozilla.org/mozilla-central/rev/e66af6d88af5
https://hg.mozilla.org/mozilla-central/rev/9d8cdcdc2c0b
https://hg.mozilla.org/mozilla-central/rev/ad53eef2175a
https://hg.mozilla.org/mozilla-central/rev/424b6a293ad6
https://hg.mozilla.org/mozilla-central/rev/1bdcbc9cd60c
https://hg.mozilla.org/mozilla-central/rev/3f44070cab66
https://hg.mozilla.org/mozilla-central/rev/f3fbd500d476
https://hg.mozilla.org/mozilla-central/rev/3c5cf284cbb2
https://hg.mozilla.org/mozilla-central/rev/ad2cf63c716f
https://hg.mozilla.org/mozilla-central/rev/1b5ecb6ba471
https://hg.mozilla.org/mozilla-central/rev/f4f37a20c5e5
https://hg.mozilla.org/mozilla-central/rev/7daf2ec505cd
https://hg.mozilla.org/mozilla-central/rev/6d3ca3b81353
https://hg.mozilla.org/mozilla-central/rev/6c061b718a6d
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Flags: in-testsuite-
Resolution: --- → FIXED
Target Milestone: --- → mozilla26
You need to log in before you can comment on or make changes to this bug.