summaryrefslogtreecommitdiffstats
path: root/courgette
Commit message (Collapse)AuthorAgeFilesLines
...
* Fix undefined behavior in DifferenceEstimator.halyavin2015-03-131-9/+13
| | | | | | | | | | | | It is illegal to use pointers outside of the array. So I added length checks to prevent it. TEST= none BUG= none Review URL: https://codereview.chromium.org/1003643003 Cr-Commit-Position: refs/heads/master@{#320458}
* Add 64-bit absolute address support to courgette.wfh2015-03-138-26/+86
| | | | | | | | | | | | | | | | | | | | Turns out, courgette never really understood 64-bit addresses, and they were all being truncated to 32-bit values. This worked fine, as the high 32-bit part was just serialized as byte data, but would have broken for a 64-bit PE image that straddled the 32-bit boundary. This CL adds a new op code for ABS64 to ensure that 64-bit math is used when assembling them, but re-uses the existing abs32 streams (since RVA are always 32-bit). This always reduces the size of the patch, but only by a tiny amount, since the four bytes in the absolute addresses no longer need to be stored separately. This also fixes static cast by reading the 8-byte absolute address from 64-bit binaries. This change is virtually size neutral. Against test binaries (64-bit chrome.dll 40.0.2214.115->43.0.2317.0) the uncompressed patch sizes were: before: 10,948,152 after: 10,925,425 (0.2% reduction) BUG=419996 TEST=courgette_unittests Review URL: https://codereview.chromium.org/629643002 Cr-Commit-Position: refs/heads/master@{#320449}
* Fix NULL pointer access in AssemblyProgram::FindLabel.halyavin2015-03-121-0/+2
| | | | | | | | | TEST= none BUG= none Review URL: https://codereview.chromium.org/1001603005 Cr-Commit-Position: refs/heads/master@{#320405}
* Fix memory leak in AssemblyProgram::TrimLabels.halyavin2015-03-121-0/+1
| | | | | | | | | | TEST= none BUG= none R=wfh@chromium.org Review URL: https://codereview.chromium.org/982903005 Cr-Commit-Position: refs/heads/master@{#320356}
* Make ParseNonSectionFileRegion a bit simplier and faster.halyavin2015-03-122-16/+8
| | | | | | | | | | TEST=courgette_unittests BUG= none R=wfh@chromium.org Review URL: https://codereview.chromium.org/998443002 Cr-Commit-Position: refs/heads/master@{#320252}
* Restore stress_test_common script.halyavin2015-03-122-1/+17
| | | | | | | | | | | | | | | | | | | This script were present in https://codereview.chromium.org/22935012 but it disappeared when that CL was commited manually. Also fix copy-paste error in analyze_stress_test. TEST= courgette/run_stress_test -s <chrome_dir1> <chrome_dir2> <result_dir> courgette/run_mem_test <result_dir> courgette/analyze_stress_test <result_dir>/log courgette/analyze_mem_test <result_dir> Also create fake bsdiff and bspatch which use courgette executable and add courgette and these utilities to the PATH. BUG= 274054 R=dgarrett@chromium.org Review URL: https://codereview.chromium.org/994883002 Cr-Commit-Position: refs/heads/master@{#320250}
* Fix memory leak in DisassemblerElf32X86::ParseRel32RelocsFromSection.halyavin2015-03-111-0/+1
| | | | | | | | | TEST= none BUG= none Review URL: https://codereview.chromium.org/990803003 Cr-Commit-Position: refs/heads/master@{#320157}
* Fix courgette ELF x86 dissemblerWill Harris2015-03-024-2/+11
| | | | | | | | | | | | | | | | | | | Courgette cannot parse files that have out of order NOBITS sections and instead will crash. The solution to this is to skip them and encode them as raw bytes. Also, courgette has a bug where it incorrectly emits ElfRelocationInstruction instructions even when there are no valid R_386_RELATIVE relocations in the file. Added a test file that exhibits both of these symptoms. BUG=424820,423925 TEST=courgette_unittests R=dgarrett@chromium.org, tommi@chromium.org Review URL: https://codereview.chromium.org/664803002 Cr-Commit-Position: refs/heads/master@{#318764}
* Apply gn format with 'sources' sorting to src/scottmg2015-02-281-13/+13
| | | | | | | | | | | | Application of https://codereview.chromium.org/962003002/. TBR=keybuk@chromium.org R=brettw@chromium.org BUG=456014 Review URL: https://codereview.chromium.org/960413003 Cr-Commit-Position: refs/heads/master@{#318574}
* Update Chomium's build files to work w/ latest GN binaries.dpranke2015-02-201-1/+1
| | | | | | | | | | | | | | The latest GN binaries rename cpu_arch to current_cpu and/or target_cpu as appropriate, build_cpu_arch to host_cpu, and os to current_os and target_os as appropriate. R=brettw@chromium.org TBR=ddorwin@chromium.org EXTRA_TRYBOTS=tryserver.chromium.linux:android_chromium_gn_compile_dbg,android_chromium_gn_compile_rel;tryserver.chromium.win:win8_chromium_gn_rel,win8_chromium_gn_dbg;tryserver.chromium.mac:mac_chromium_gn_rel,mac_chromium_gn_dbg BUG=344767 Review URL: https://codereview.chromium.org/913373002 Cr-Commit-Position: refs/heads/master@{#317223}
* Update existing uses of /wd4267 to use the GN configbrettw2015-02-191-4/+2
| | | | | | | | | | This is the size_t to integer conversion warning. Using the config prevents flag duplication and is clearer to read. Minor updates to Win64 build, including some fixes for size_t to int conversions. Review URL: https://codereview.chromium.org/929793006 Cr-Commit-Position: refs/heads/master@{#317162}
* Remove courgette version tests. Add bsdiff version tests.wfh2015-02-195-16/+16
| | | | | | | | | | | | | | The component updater now only uses bsdiff so courgete diffs no longer have to forwards and backwards compatible, since Chrome always ships the correct version of the courgette patcher with the courgette diff. However, bsdiff generated diffs still have to be version compatible. This CL adds a test for the forwards compatible case. BUG=459505 Review URL: https://codereview.chromium.org/918403003 Cr-Commit-Position: refs/heads/master@{#317117}
* Isolate courgette_unittestsscottmg2015-01-222-0/+35
| | | | | | | | | | | | | | | Adding link to http://www.chromium.org/developers/testing/isolated-testing/for-swes for my future self. R=maruel@chromium.org,jochen@chromium.org TBR=jam@chromium.org BUG=98637, 450912 Committed: https://crrev.com/c9bd3b9a5667a36c05592ec9d16533be91b4567c Cr-Commit-Position: refs/heads/master@{#312531} Review URL: https://codereview.chromium.org/868433002 Cr-Commit-Position: refs/heads/master@{#312665}
* Revert of Revert of Revert of Isolate courgette_unittests (patchset #1 id:1 ↵kochi2015-01-222-31/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | of https://codereview.chromium.org/867523002/) Reason for revert: Sorry for the mess - in the end, the original CL was causing the courgette_unittests failure. Reverting the original again. BUG=450912 Original issue's description: > Revert of Revert of Isolate courgette_unittests (patchset #1 id:1 of https://codereview.chromium.org/865133002/) > > Reason for revert: > The revert might have been wrong: reverting the revert again. > BUG=450912 > > Original issue's description: > > Revert of Isolate courgette_unittests (patchset #1 id:1 of https://codereview.chromium.org/868433002/) > > > > Reason for revert: > > http://crbug.com/450912 > > > > courget_unittests is failing on Win7(dbg) bots since build#34639. > > http://build.chromium.org/p/chromium.win/builders/Win7%20Tests%20%28dbg%29%281%29/builds/34639 > > > > Original issue's description: > > > Isolate courgette_unittests > > > > > > Adding link to http://www.chromium.org/developers/testing/isolated-testing/for-swes for my future self. > > > > > > R=maruel@chromium.org,jochen@chromium.org > > > TBR=jam@chromium.org > > > BUG=98637 > > > > > > Committed: https://crrev.com/c9bd3b9a5667a36c05592ec9d16533be91b4567c > > > Cr-Commit-Position: refs/heads/master@{#312531} > > > > TBR=jam@chromium.org,jochen@chromium.org,maruel@chromium.org,scottmg@chromium.org > > NOPRESUBMIT=true > > NOTREECHECKS=true > > NOTRY=true > > BUG=98637 > > > > Committed: https://crrev.com/1bdeeeb5eae42e95f82806da9351d77a05403e67 > > Cr-Commit-Position: refs/heads/master@{#312571} > > TBR=jam@chromium.org,jochen@chromium.org,maruel@chromium.org,scottmg@chromium.org > NOPRESUBMIT=true > NOTREECHECKS=true > NOTRY=true > BUG=98637 > > Committed: https://crrev.com/217c53ca8128370c64a5d1a4ba15cb4cea5564ac > Cr-Commit-Position: refs/heads/master@{#312577} TBR=jam@chromium.org,jochen@chromium.org,maruel@chromium.org,scottmg@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=450912 Review URL: https://codereview.chromium.org/800333003 Cr-Commit-Position: refs/heads/master@{#312584}
* Revert of Revert of Isolate courgette_unittests (patchset #1 id:1 of ↵kochi2015-01-222-0/+31
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | https://codereview.chromium.org/865133002/) Reason for revert: The revert might have been wrong: reverting the revert again. BUG=450912 Original issue's description: > Revert of Isolate courgette_unittests (patchset #1 id:1 of https://codereview.chromium.org/868433002/) > > Reason for revert: > http://crbug.com/450912 > > courget_unittests is failing on Win7(dbg) bots since build#34639. > http://build.chromium.org/p/chromium.win/builders/Win7%20Tests%20%28dbg%29%281%29/builds/34639 > > Original issue's description: > > Isolate courgette_unittests > > > > Adding link to http://www.chromium.org/developers/testing/isolated-testing/for-swes for my future self. > > > > R=maruel@chromium.org,jochen@chromium.org > > TBR=jam@chromium.org > > BUG=98637 > > > > Committed: https://crrev.com/c9bd3b9a5667a36c05592ec9d16533be91b4567c > > Cr-Commit-Position: refs/heads/master@{#312531} > > TBR=jam@chromium.org,jochen@chromium.org,maruel@chromium.org,scottmg@chromium.org > NOPRESUBMIT=true > NOTREECHECKS=true > NOTRY=true > BUG=98637 > > Committed: https://crrev.com/1bdeeeb5eae42e95f82806da9351d77a05403e67 > Cr-Commit-Position: refs/heads/master@{#312571} TBR=jam@chromium.org,jochen@chromium.org,maruel@chromium.org,scottmg@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=98637 Review URL: https://codereview.chromium.org/867523002 Cr-Commit-Position: refs/heads/master@{#312577}
* Revert of Isolate courgette_unittests (patchset #1 id:1 of ↵kochi2015-01-222-31/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | https://codereview.chromium.org/868433002/) Reason for revert: http://crbug.com/450912 courget_unittests is failing on Win7(dbg) bots since build#34639. http://build.chromium.org/p/chromium.win/builders/Win7%20Tests%20%28dbg%29%281%29/builds/34639 Original issue's description: > Isolate courgette_unittests > > Adding link to http://www.chromium.org/developers/testing/isolated-testing/for-swes for my future self. > > R=maruel@chromium.org,jochen@chromium.org > TBR=jam@chromium.org > BUG=98637 > > Committed: https://crrev.com/c9bd3b9a5667a36c05592ec9d16533be91b4567c > Cr-Commit-Position: refs/heads/master@{#312531} TBR=jam@chromium.org,jochen@chromium.org,maruel@chromium.org,scottmg@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=98637 Review URL: https://codereview.chromium.org/865133002 Cr-Commit-Position: refs/heads/master@{#312571}
* Isolate courgette_unittestsscottmg2015-01-222-0/+31
| | | | | | | | | | | | Adding link to http://www.chromium.org/developers/testing/isolated-testing/for-swes for my future self. R=maruel@chromium.org,jochen@chromium.org TBR=jam@chromium.org BUG=98637 Review URL: https://codereview.chromium.org/868433002 Cr-Commit-Position: refs/heads/master@{#312531}
* Move the test template to //testing/test.gni (part 2)qsr2015-01-211-0/+2
| | | | | | | | | | | | This CL move the test target from build/config/BUILDCONFIG.gn to testing/test.gni It also update the test template to automatically build an apk on Android. R=cjhopman@chromium.org,brettw@chromium.org Review URL: https://codereview.chromium.org/824263004 Cr-Commit-Position: refs/heads/master@{#312396}
* Using "static_assert" in lieu of "COMPILE_ASSERT" in courgete moduleanujk.sharma2015-01-205-9/+10
| | | | | | | | | | All our toolchains support c++'s static_assert now- COMPILE_ASSERT can be removed now. BUG=442514 Review URL: https://codereview.chromium.org/857153003 Cr-Commit-Position: refs/heads/master@{#312203}
* Disable two PagedArrayTests that fail under AddressSanitizer on Windowstimurrrr2015-01-141-0/+4
| | | | | | | | | | | This should make courgette_unittests green in this configuration. BUG=345874 TBR=dgarrett@chromium.org Review URL: https://codereview.chromium.org/850783002 Cr-Commit-Position: refs/heads/master@{#311453}
* Make callers of CommandLine use it via the base:: namespace.avi2014-12-221-3/+4
| | | | | | | | | | | | Covers courgette/, dbus/, device/, extensions/, gin/, google_apis/, and gpu/. BUG=422426 TEST=none TBR=ben@chromium.org Review URL: https://codereview.chromium.org/819193002 Cr-Commit-Position: refs/heads/master@{#309490}
* Cleanup: Get rid of more base::ASCIIToWide usage.thestig2014-12-041-10/+7
| | | | | | | | | | Also do some lint cleanups. BUG=23581 Review URL: https://codereview.chromium.org/777623002 Cr-Commit-Position: refs/heads/master@{#306905}
* gn format // (the rest, except mojo)scottmg2014-12-031-1/+1
| | | | | | | | | | | | | | Excluded mojo because I think that needs to happen on the other side. At gn --version = 306668 for which roll is in CQ. R=brettw@chromium.org TBR=scherkus@chromium.org BUG=348474 Review URL: https://codereview.chromium.org/774353003 Cr-Commit-Position: refs/heads/master@{#306708}
* Add wfh@chromium.org to courgette OWNERS file.dgarrett2014-10-141-0/+1
| | | | | | | | | | | | Will has been the most active Courgette developer for a while, so maybe he should have ownership. R=tommi@chromium.org, sra@chromium.org, dpranke@chromium.org BUG=None Review URL: https://codereview.chromium.org/655893002 Cr-Commit-Position: refs/heads/master@{#299543}
* replace OVERRIDE with override in courgette/mostynb2014-10-141-3/+3
| | | | | | | | | R=dgarrett@chromium.org BUG=417463 Review URL: https://codereview.chromium.org/655553003 Cr-Commit-Position: refs/heads/master@{#299446}
* Re-enable ensemble tests on Windows Release builds.wfh2014-10-141-6/+15
| | | | | | | | | | | This is only taking a long time on Debug, because of the additional iterator validators running on all the stl objects being used. BUG=101614 Review URL: https://codereview.chromium.org/647093002 Cr-Commit-Position: refs/heads/master@{#299437}
* Rewrite line endings from CRLF to LF for a bunch of files.Daniel Cheng2014-10-071-23/+23
| | | | | | | | | | | | | There's a lot of other files with CRLF line endings in test data, third_party, and tools. Out of caution, those files have been left unchanged, in case anything depends on the CRLF being present. BUG=420796 TBR=darin@chromium.org, rsleevi@chromium.org, sky@chromium.org Review URL: https://codereview.chromium.org/630333002 Cr-Commit-Position: refs/heads/master@{#298348}
* Fix more MSVC warnings, courgette/ edition.pkasting2014-10-039-62/+42
| | | | | | | | | | | | This is mostly about changing types and inserting casts so as to avoid implicit value truncations. BUG=81439 TEST=none Review URL: https://codereview.chromium.org/613893002 Cr-Commit-Position: refs/heads/master@{#298069}
* Cleanup: Use base/files/file_util.h instead of base/file_util.h in [c-n]*/thestig2014-09-135-7/+7
| | | | | | | | | 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}
* Make Courgette as much as 5x faster.asvitkine2014-09-121-0/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | Changes the Courgette vector implementation to slightly over-reserve the size of the vector, which makes many future reserve operations no-ops (as they're quite expensive otherwise, since they copy the full contents of the previous buffer). When applying the patch from 35.0.1916.114 to 37.0.2062.120, before and after this change, the runtime goes from 1m10s to 21s on my z620. Slightly higher multipliers produce even better results (e.g. 19s), but this seemed like a reasonable value to chose so that it doesn't result in significant additional memory use by Courgette. BUG=167622 TEST=Build courgette target in Release mode and run it as courgette.exe -apply chrome.7z chrome_patch.diff out.7z (where chrome.7z was from un7zipping a 37.0.2062.94 chrome installer and chrome_patch.diff was from un7zipping a 37.0.2062.94 -> 37.0.2062.120_37 chrome_updater_3stage.exe). With the patch, the operation should run ~5x faster. Review URL: https://codereview.chromium.org/565753002 Cr-Commit-Position: refs/heads/master@{#294592}
* Fixes for re-enabling more MSVC level 4 warnings: courgette/ editionpkasting@chromium.org2014-07-083-3/+3
| | | | | | | | | | | | This contains fixes for the following sorts of issues: * Signedness mismatch BUG=81439 TEST=none Review URL: https://codereview.chromium.org/371153002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@281696 0039d316-1c4b-4281-b951-d872f2087c98
* Minor Fixes for Windows GN build.brettw@chromium.org2014-06-141-1/+1
| | | | | | | | TBR=scottmg Review URL: https://codereview.chromium.org/335973002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@277272 0039d316-1c4b-4281-b951-d872f2087c98
* Add more content subtargets and deps to the GN build.brettw@chromium.org2014-06-131-0/+137
| | | | | | | | | | Includes courgette, libva, and lzma. TBR=scottmg Review URL: https://codereview.chromium.org/334913005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@277132 0039d316-1c4b-4281-b951-d872f2087c98
* Remove {linux|android}_use_tcmalloc and switch to use_allocator in Chromium.dmikurube@chromium.org2014-05-071-2/+1
| | | | | | | | | | | | | | If this change breaks some bots, please try restarting the bot before reverting it. http://crrev.com/264460 may not be effective yet until restarting. See the bug and http://crrev.com/255129 for the details. BUG=345554 Review URL: https://codereview.chromium.org/258433005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@268876 0039d316-1c4b-4281-b951-d872f2087c98
* Remove all uses of toolkit_use_gtk in the gyp files.erg@chromium.org2014-04-171-20/+0
| | | | | | | | | BUG=297026 R=ben@chromium.org, brettw@chromium.org Review URL: https://codereview.chromium.org/238633004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@264609 0039d316-1c4b-4281-b951-d872f2087c98
* Port to x64 Android.anton@chromium.org2014-04-031-1/+1
| | | | | | | | | | | New compiler version complains about used before initialized. Compiler is wrong about this, but there is no harm in initializing. BUG=346626 Review URL: https://codereview.chromium.org/222293002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@261343 0039d316-1c4b-4281-b951-d872f2087c98
* Fix "unreachable code" warnings (MSVC warning 4702), misc. edition.pkasting@chromium.org2014-03-181-2/+0
| | | | | | | | | | | | This CL covers top-level directories that only had one or two modified files. BUG=346399 TEST=none R=darin@chromium.org Review URL: https://codereview.chromium.org/203043002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@257705 0039d316-1c4b-4281-b951-d872f2087c98
* Move WriteFile and WriteFileDescriptor from file_util to base namespace.brettw@chromium.org2014-03-063-3/+3
| | | | | | | | | R=viettrungluu@chromium.org TBR=viettrungluu Review URL: https://codereview.chromium.org/184563006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@255418 0039d316-1c4b-4281-b951-d872f2087c98
* Add use_allocator instead of linux_use_tcmalloc to switch the allocator.dmikurube@chromium.org2014-03-051-1/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This change is to add a new build option 'use_allocator' which will replace 'linux_use_tcmalloc' in the future. It doesn't change the behavior immediately. The migration plan is as follows: 1) (this change) ... Add 'use_allocator' and set its default to "see_use_tcmalloc". ... Change allocator conditions to check use_allocator firstly. ... Use linux_use_tcmalloc if use_allocator=="see_use_tcmalloc". ... NO IMPACT without specifying use_allocator explicitly. 2) Change Blink to accept use_allocator. http://crrev.com/177053003/ 3) Change gyp to accept use_allocator. http://crrev.com/178643004/ 4) PSA the transition period to chromium-dev@. 5) (after the PSA-ed transition period) ... Make 'use_allocator' to "tcmalloc" or "none" (it depends) by default. ... Remove all linux_use_tcmalloc. ... Assert in gyp_chromium to check if linux_use_tcmalloc is not specified. At the point of this change (1), linux_use_tcmalloc is still used by default because 'use_allocator%': "see_use_tcmalloc". As written in http://crbug.com/345554, linux_use_tcmalloc would be confusing to have more options about allocators. We plan to: A) enable gperftools' heap-profiler with non-tcmalloc allocator, B) add a new memory allocator instead of tcmalloc. BUG=345554, 339604, 341349 R=agl@chromium.org, brettw@chromium.org, dgarrett@chromium.org, jam@chromium.org, jamesr@chromium.org, joi@chromium.org, miket@chromium.org, nick@chromium.org, rsleevi@chromium.org, scherkus@chromium.org, sergeyu@chromium.org, shess@chromium.org, sievers@chromium.org, sky@chromium.org, vitalybuka@chromium.org, willchan@chromium.org Review URL: https://codereview.chromium.org/177353002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@255129 0039d316-1c4b-4281-b951-d872f2087c98
* Remove PlatformFile from courgettervargas@chromium.org2014-02-212-70/+20
| | | | | | | | | BUG=322664 R=tommi@chromium.org Review URL: https://codereview.chromium.org/166273021 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@252570 0039d316-1c4b-4281-b951-d872f2087c98
* Allow Win64 build of base library even in a Win32 build.petewil@chromium.org2014-01-091-2/+2
| | | | | | | | | | | | | | | Some components of a 32 bit build of chrome for Windows need to be built as Win64 Dlls. To allow those components to use base, we make a Win64 build of the base library even when building chrome 32 bit. This is needed by the Chrome Desk Band code. crbug.com/327435. https://codereview.chromium.org/79173004/ BUG=327435 Review URL: https://codereview.chromium.org/103333004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@243936 0039d316-1c4b-4281-b951-d872f2087c98
* Update uses of UTF conversions in courgette/, device/, extensions/, ↵avi@chromium.org2013-12-251-1/+2
| | | | | | | | | | | | google_apis/, gpu/, ipc/, media/, net/ to use the base:: namespace. BUG=330556 TEST=no change TBR=ben@chromium.org Review URL: https://codereview.chromium.org/112963005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@242496 0039d316-1c4b-4281-b951-d872f2087c98
* Move GetFileSize, NormalizeFilePath to base namespacebrettw@chromium.org2013-12-041-1/+1
| | | | | | | | | BUG= R=viettrungluu@chromium.org Review URL: https://codereview.chromium.org/102873002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@238722 0039d316-1c4b-4281-b951-d872f2087c98
* Move temp file functions to base namespace.brettw@chromium.org2013-12-031-1/+1
| | | | | | | | BUG= Review URL: https://codereview.chromium.org/99923002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@238427 0039d316-1c4b-4281-b951-d872f2087c98
* Add PE64 support to courgettewfh@chromium.org2013-09-2524-53/+1148
| | | | | | | | | | | Add tests for PE64 BUG=38784 NOTRY=true Review URL: https://chromiumcodereview.appspot.com/23600063 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@225224 0039d316-1c4b-4281-b951-d872f2087c98
* Fix +x bit on courgette testdatascottmg@chromium.org2013-09-242-0/+0
| | | | | | | | BUG=38784 Review URL: https://codereview.chromium.org/23874026 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@225036 0039d316-1c4b-4281-b951-d872f2087c98
* Commit binary files needed by Courgette tests for PE 64.scottmg@chromium.org2013-09-244-0/+0
| | | | | | | | | | | | | | | These files are referenced by unittests, and not used otherwise. They are in this commit to work around the git-svn binary file brokeness. This is the CL they actually belong to: https://codereview.chromium.org/23600063/ BUG=38784 R=wfh@chromium.org Review URL: https://codereview.chromium.org/24201008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@225025 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 224789 "Add PE64 support to courgette"piman@chromium.org2013-09-2328-1148/+53
| | | | | | | | | | | | | | | | | | Caused failures: http://build.chromium.org/p/chromium.win/builders/Win7%20Tests%20%28dbg%29%281%29/builds/23037/steps/courgette_unittests/logs/stdio > Add PE64 support to courgette > > Add tests for PE64 > > BUG=38784 > > Review URL: https://chromiumcodereview.appspot.com/23600063 TBR=wfh@chromium.org Review URL: https://codereview.chromium.org/23591065 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@224806 0039d316-1c4b-4281-b951-d872f2087c98
* Add PE64 support to courgettewfh@chromium.org2013-09-2328-53/+1148
| | | | | | | | | | Add tests for PE64 BUG=38784 Review URL: https://chromiumcodereview.appspot.com/23600063 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@224789 0039d316-1c4b-4281-b951-d872f2087c98
* Convert most run_all_unittests.cc files to use new unit test launcher.phajdan.jr@chromium.org2013-09-052-10/+1
| | | | | | | | | | | | | Note that the new code is still behind a runtime flag (--brave-new-test-launcher), but compiling tests with support for it will make further testing possible. BUG=236893, 79359 R=akalin@chromium.org, enne@chromium.org, erikwright@chromium.org, joi@chromium.org, keybuk@chromium.org, sky@chromium.org, thestig@chromium.org, tommi@chromium.org, wtc@chromium.org, xhwang@chromium.org, yzshen@chromium.org Review URL: https://codereview.chromium.org/23442019 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@221464 0039d316-1c4b-4281-b951-d872f2087c98