- Feb 15, 2017
-
-
mark authored
Reapply and regenerate all local patches to upstream zlib 1.2.11 Explicitly specify 9 as the minimum windowBits value (representing a 512-byte window) during compression in net/websockets even when 8 (representing 256) is received. This was previously silently done during compression. Because of how zlib's deflate is implemented, when windowBits is 9, it will produce a stream that can be decompressed with a 250-byte or larger window. Changes in 1.2.9 (31 Dec 2016) - Fix contrib/minizip to permit unzipping with desktop API [Zouzou] - Improve contrib/blast to return unused bytes - Assure that gzoffset() is correct when appending - Improve compress() and uncompress() to support large lengths - Fix bug in test/example.c where error code not saved - Remedy Coverity warning [Randers-Pehrson] - Improve speed of gzprintf() in transparent mode - Fix inflateInit2() bug when windowBits is 16 or 32 - Change DEBUG macro to ZLIB_DEBUG - Avoid uninitialized access by gzclose_w() - Allow building zlib outside of the source directory - Fix bug that accepted invalid zlib header when windowBits is zero - Fix gzseek() problem on MinGW due to buggy _lseeki64 there - Loop on write() calls in gzwrite.c in case of non-blocking I/O - Add --warn (-w) option to ./configure for more compiler warnings - Reject a window size of 256 bytes if not using the zlib wrapper - Fix bug when level 0 used with Z_HUFFMAN or Z_RLE - Add --debug (-d) option to ./configure to define ZLIB_DEBUG - Fix bugs in creating a very large gzip header - Add uncompress2() function, which returns the input size used - Assure that deflateParams() will not switch functions mid-block - Dramatically speed up deflation for level 0 (storing) - Add gzfread(), duplicating the interface of fread() - Add gzfwrite(), duplicating the interface of fwrite() - Add deflateGetDictionary() function - Use snprintf() for later versions of Microsoft C - Fix *Init macros to use z_ prefix when requested - Replace as400 with os400 for OS/400 support [Monnerat] - Add crc32_z() and adler32_z() functions with size_t lengths - Update Visual Studio project files [AraHaan] Changes in 1.2.10 (2 Jan 2017) - Avoid warnings on snprintf() return value - Fix bug in deflate_stored() for zero-length input - Fix bug in gzwrite.c that produced corrupt gzip files - Remove files to be installed before copying them in Makefile.in - Add warnings when compiling with assembler code Changes in 1.2.11 (15 Jan 2017) - Fix deflate stored bug when pulling last block from window - Permit immediate deflateParams changes before any deflate input BUG=691074, 691075 Review-Url: https://codereview.chromium.org/2690623003 Cr-Original-Commit-Position: refs/heads/master@{#450585} Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src Cr-Mirrored-Commit: 6d9a6251dfe87183075dc16cfa134e41dc4cee0d
-
- Feb 11, 2017
-
-
hshi authored
BUG=None TBR=satorux Review-Url: https://codereview.chromium.org/2688993002 Cr-Original-Commit-Position: refs/heads/master@{#449851} Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src Cr-Mirrored-Commit: 54ea02a8d32ed453b6e1fd7976f71b545d9f64f5
-
- Feb 10, 2017
-
-
msarett authored
NOTRY=true BUG= Review-Url: https://codereview.chromium.org/2688893003 Cr-Original-Commit-Position: refs/heads/master@{#449782} Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src Cr-Mirrored-Commit: 318473ffcb37f2ec016d76c2907eb65570aff682
-
- Feb 02, 2017
-
-
davidben authored
With SPDY gone, this is no longer necessary. This change was done by reversing simd.patch, reversing mixed-source.patch, reapplying simd.patch, and then recreating simd.patch. BUG=none Review-Url: https://codereview.chromium.org/2669053004 Cr-Original-Commit-Position: refs/heads/master@{#447853} Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src Cr-Mirrored-Commit: f2095b9da969dd7decf3224fc96ffe8ef5f5455b
-
- Nov 22, 2016
-
-
meacer authored
CRX files are extracted twice: Once for only manifest.json, once for the rest of the contents. Extracting manifest.json causes a lot of noise in debug builds since all other files are skipped. Suppress the log statements when extracting manifest.json. BUG=645263 Review-Url: https://codereview.chromium.org/2523803002 Cr-Original-Commit-Position: refs/heads/master@{#433984} Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src Cr-Mirrored-Commit: a98b44daed1683b35f0b5b638f5693aebf0dab5a
-
alemate authored
Debug version of Chrome is not buildable because of inconsistent #ifdefs on NDEBUG/ASSERT/IS_DCHECK_ON . This change makes them consistemt in zlib. BUG=666660 Review-Url: https://codereview.chromium.org/2512933002 Cr-Original-Commit-Position: refs/heads/master@{#433729} Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src Cr-Mirrored-Commit: 90a667a18ad56d14f7068e8e6cbfb5668d3b55f3
-
- Sep 30, 2016
-
-
fdoray authored
This CL makes the following replacements: MessageLoop::current()->task_runner() becomes ThreadTaskRunnerHandle::Get() MessageLoopForUIorIO::current()->task_runner() becomes DCHECK(MessageLoopForUIorIO::IsCurrent()); (or ASSERT_TRUE) ThreadTaskRunnerHandle::Get() TBR=gavinp@chromium.org BUG=650723 Review-Url: https://codereview.chromium.org/2375663002 Cr-Original-Commit-Position: refs/heads/master@{#422211} Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src Cr-Mirrored-Commit: 19e49e924e88e50546ec1a81dbc7b43ea4b172ef
-
- Sep 28, 2016
-
-
vabr authored
path() is being deprecated, GetPath() has better checking against wrong use. For more context, see https://codereview.chromium.org/2275553005/. BUG=640599 CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.win:win10_chromium_x64_rel_ng Review-Url: https://codereview.chromium.org/2317123003 Cr-Original-Commit-Position: refs/heads/master@{#421483} Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src Cr-Mirrored-Commit: a98198273f744d4e045e759734bbaa69962fa94d
-
- Sep 14, 2016
-
-
brettw authored
This removes all .gyp files not called "compiled_resources" in the main Chromium repository. CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.win:win10_chromium_x64_rel_ng Review-Url: https://codereview.chromium.org/2332843004 Cr-Original-Commit-Position: refs/heads/master@{#418654} Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src Cr-Mirrored-Commit: 315e4cfd4b1a120c3e2b28812afc5dbdc292bebe
-
- Sep 09, 2016
-
-
meacer authored
BUG=468355 Review-Url: https://codereview.chromium.org/2321823002 Cr-Original-Commit-Position: refs/heads/master@{#417494} Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src Cr-Mirrored-Commit: b040d3a050ac90bdae599ced55b2f8f7ac84bc30
-
- Jul 27, 2016
-
-
altimin authored
Review-Url: https://codereview.chromium.org/2121343002 Cr-Original-Commit-Position: refs/heads/master@{#408157} Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src Cr-Mirrored-Commit: 3ebc568312f7a2c48dac5a5a7118e257456af405
-
- Jun 30, 2016
-
-
maksim.sisov authored
Use new time conversion API in accordance with https://codereview.chromium.org/1988663002/ BUG=601905 Review-Url: https://codereview.chromium.org/2091673002 Cr-Original-Commit-Position: refs/heads/master@{#403134} Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src Cr-Mirrored-Commit: 3d0a1b68b407f0535052b62819f9e7bb68ee2a94
-
- Jun 29, 2016
-
-
fdoray authored
This CL makes the following replacements in third_party: Before After ---------------------------------------------------------- x.PostTask() x.task_runner()->PostTask() PostDelayedTask() PostDelayedTask() ReleaseSoon() ReleaseSoon() DeleteSoon() DeleteSoon() x->PostTask() y->task_runner()->PostTask() PostDelayedTask() PostDelayedTask() ReleaseSoon() ReleaseSoon() DeleteSoon() DeleteSoon() x.Run() RunLoop().Run() x.RunUntilIdle() RunLoop().RunUntilIdle() x->Run() RunLoop().Run() x->RunUntilIdle() RunLoop().RunUntilIdle() If |y| isn't MessageLoopForUI::current() or MessageLoopForIO::current() y.message_loop()->task_runner() y.task_runner() y->message_loop()->task_runner() y->task_runner() ---------------------------------------------------------- |x| is a base::MessageLoop(ForUI|ForIO) or a pointer to a base::MessageLoop(ForUI|ForIO). |y| is a base::Thread or a pointer to a base::Thread. This CL was generated using the MessageLoopDeprecatedMethods clang-tidy fix available on the associated bug. Only files that compile on Mac are affected. Follow-up CLs will make these replacements for other platforms. This CL doesn't change code behavior. TBR=dglazkov@chromium.org,estade@chromium.org,agl@chromium.org BUG=616447 Review-Url: https://codereview.chromium.org/2107163003 Cr-Original-Commit-Position: refs/heads/master@{#402847} Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src Cr-Mirrored-Commit: b37b2e3abbe6eee8bad666bd543b23570f3ca7c2
-
- Jun 23, 2016
-
-
jiadong.zhu authored
merge the latest open source zlib into chromium external projects Version 1.2.8 fixes a very rare bug in decompression. All users are encouraged to upgrade immediately. Version 1.2.8 also has these improvements: - Add new inflateGetDictionary() function - Fix bug where gzopen() immediately followed by gzclose() would write an empty file instead of an empty gzip stream. - Fix bug in gzclose() when gzwrite() runs out of memory Version 1.2.7 has many portability improvements over 1.2.6, and has these enhancements: - Fix bug in gzclose_w() when gzwrite() fails to allocate memory - Add "x" (O_EXCL) and "e" (O_CLOEXEC) modes support to gzopen() - Add gzopen_w() in Windows for wide character path names - Fix type mismatch between get_crc_table() and crc_table Version 1.2.6 has many changes over 1.2.5, including these improvements: - gzread() can now read a file that is being written concurrently - gzgetc() is now a macro for increased speed - Added a 'T' option to gzopen() for transparent writing (no compression) - Added deflatePending() to return the amount of pending output - Allow deflateSetDictionary() and inflateSetDictionary() at any time in raw mode - deflatePrime() can now insert bits in the middle of the stream - ./configure now creates a configure.log file with all of the results - Added a ./configure --solo option to compile zlib with no dependency on any libraries - Fixed a problem with large file support macros - Fixed a bug in contrib/puff - Many portability improvements BUG=610615 R=gavinp@chromium.org Review-Url: https://codereview.chromium.org/2084863002 Cr-Original-Commit-Position: refs/heads/master@{#401538} Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src Cr-Mirrored-Commit: ca1c569cff20aa43a42816791744d847c7d788de
-
- Jun 20, 2016
-
-
jmadill authored
Revert of Update Zlib to version 1.2.8 (patchset #10 id:220001 of https://codereview.chromium.org/1955383002/ ) Reason for revert: Broke the Windows Debug GPU FYI builder: https://build.chromium.org/p/chromium.gpu.fyi/builders/GPU%20Win%20Builder%20%28dbg%29/builds/41421 fx_zlib.gzlib.obj : error LNK2005: _gzopen_w already defined in zlib.gzlib.obj initial\chrome.dll : fatal error LNK1169: one or more multiply defined symbols found Possibly because they use shared components with a GYP build. Original issue's description: > Update Zlib to version 1.2.8 > > merge the latest open source zlib into chromium external projects > > Version 1.2.8 fixes a very rare bug in decompression. All users are encouraged to upgrade immediately. Version 1.2.8 also has these improvements: > - Add new inflateGetDictionary() function > - Fix bug where gzopen() immediately followed by gzclose() would write an empty file instead of an empty gzip stream. > - Fix bug in gzclose() when gzwrite() runs out of memory > Version 1.2.7 has many portability improvements over 1.2.6, and has these enhancements: > - Fix bug in gzclose_w() when gzwrite() fails to allocate memory > - Add "x" (O_EXCL) and "e" (O_CLOEXEC) modes support to gzopen() > - Add gzopen_w() in Windows for wide character path names > - Fix type mismatch between get_crc_table() and crc_table > Version 1.2.6 has many changes over 1.2.5, including these improvements: > - gzread() can now read a file that is being written concurrently > - gzgetc() is now a macro for increased speed > - Added a 'T' option to gzopen() for transparent writing (no compression) > - Added deflatePending() to return the amount of pending output > - Allow deflateSetDictionary() and inflateSetDictionary() at any time in raw mode > - deflatePrime() can now insert bits in the middle of the stream > - ./configure now creates a configure.log file with all of the results > - Added a ./configure --solo option to compile zlib with no dependency on any libraries > - Fixed a problem with large file support macros > - Fixed a bug in contrib/puff > - Many portability improvements > > BUG=610615 > R=agl@chromium.org > R=gavinp@chromium.org > > Committed: https://crrev.com/a50849ae9ba1b2fad47905db521463b3bf085883 > Cr-Commit-Position: refs/heads/master@{#400647} TBR=agl@chromium.org,gavinp@chromium.org,khasim.mohammed@linaro.org,jiadong.zhu@linaro.org # Skipping CQ checks because original CL landed less than 1 days ago. NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=610615 Review-Url: https://codereview.chromium.org/2079313002 Cr-Original-Commit-Position: refs/heads/master@{#400670} Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src Cr-Mirrored-Commit: 43770253ccc0425c4edde966e56b3e68955cc4d3
-
jiadong.zhu authored
merge the latest open source zlib into chromium external projects Version 1.2.8 fixes a very rare bug in decompression. All users are encouraged to upgrade immediately. Version 1.2.8 also has these improvements: - Add new inflateGetDictionary() function - Fix bug where gzopen() immediately followed by gzclose() would write an empty file instead of an empty gzip stream. - Fix bug in gzclose() when gzwrite() runs out of memory Version 1.2.7 has many portability improvements over 1.2.6, and has these enhancements: - Fix bug in gzclose_w() when gzwrite() fails to allocate memory - Add "x" (O_EXCL) and "e" (O_CLOEXEC) modes support to gzopen() - Add gzopen_w() in Windows for wide character path names - Fix type mismatch between get_crc_table() and crc_table Version 1.2.6 has many changes over 1.2.5, including these improvements: - gzread() can now read a file that is being written concurrently - gzgetc() is now a macro for increased speed - Added a 'T' option to gzopen() for transparent writing (no compression) - Added deflatePending() to return the amount of pending output - Allow deflateSetDictionary() and inflateSetDictionary() at any time in raw mode - deflatePrime() can now insert bits in the middle of the stream - ./configure now creates a configure.log file with all of the results - Added a ./configure --solo option to compile zlib with no dependency on any libraries - Fixed a problem with large file support macros - Fixed a bug in contrib/puff - Many portability improvements BUG=610615 R=agl@chromium.org R=gavinp@chromium.org Review-Url: https://codereview.chromium.org/1955383002 Cr-Original-Commit-Position: refs/heads/master@{#400647} Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src Cr-Mirrored-Commit: a50849ae9ba1b2fad47905db521463b3bf085883
-
- May 12, 2016
-
-
gab authored
Changes made by tools/git/move_source_file.py Also updated DEPS file (script didn't do it automatically) BUG=610438 TBR=klobag Review-Url: https://codereview.chromium.org/1971743003 Cr-Original-Commit-Position: refs/heads/master@{#393166} Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src Cr-Mirrored-Commit: 3ff438d691ee7deaa20f24aa6a126acbbbbd1add
-
- Apr 22, 2016
-
-
dcheng authored
BUG=554298 Review URL: https://codereview.chromium.org/1911823002 Cr-Original-Commit-Position: refs/heads/master@{#389307} Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src Cr-Mirrored-Commit: 91cdd51340b8f6a0cc13ec9f5199defbea6cd49c
-
- Apr 19, 2016
-
-
jam authored
Otherwise patches can land in the CQ which trigger DLOG(FATAL) or assert but then break the main waterfall. BUG=603166 Review URL: https://codereview.chromium.org/1885933002 Cr-Original-Commit-Position: refs/heads/master@{#388086} Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src Cr-Mirrored-Commit: 121900aaaac3ee9407ad40dd05e54b6d50b859c9
-
- Feb 26, 2016
-
-
robert.bradford authored
XP didn't have InitOnceExecuteOnce() so it was necessary to implement an equivalent using atomic compare-and-swap. With XP support no longer necessary the CPU feature detection code can now use InitOnceExecuteOnce() directly. TEST=net_unittests(GZipUnitTest) BUG=587964 Review URL: https://codereview.chromium.org/1711683003 Cr-Original-Commit-Position: refs/heads/master@{#377928} Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src Cr-Mirrored-Commit: 18ad5f3a40ceab583961ca5dc064e01900514c57
-
- Jan 14, 2016
-
-
hajimehoshi authored
We are now implementing CompressibleString in third_party/WebKit/Source/ wtf/text and want to use compression algorithm, but wtf/text can't depend on components. This CL moves components/compression to third_party and enables wtf/text to use the compression utility. The purpose of this CL is similar to crrev.com/14021015. BUG=574317 TEST=n/a Review URL: https://codereview.chromium.org/1564773002 Cr-Original-Commit-Position: refs/heads/master@{#369491} Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src Cr-Mirrored-Commit: 1e1cc1a7060ab8eb95a62da608f333e31aaba475
-
- Dec 28, 2015
-
-
dcheng authored
❆(੭ु ◜◡‾)੭ु⁾
☃️ ❆ BUG=557422 R=avi@chromium.org TBR=jam@chromium.org Review URL: https://codereview.chromium.org/1550693002 Cr-Original-Commit-Position: refs/heads/master@{#366956} Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src Cr-Mirrored-Commit: e486004502c17ab80953ef324926a536b8030d1e
-
- Dec 27, 2015
-
-
avi authored
BUG=138542 TBR=agl@chromium.org Review URL: https://codereview.chromium.org/1552513002 Cr-Original-Commit-Position: refs/heads/master@{#366935} Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src Cr-Mirrored-Commit: 7159e7ac95455689cd1075cd841ab449222ff74b
-
- Nov 03, 2015
-
-
tfarina authored
This patch was generated using the following command line: $ git ls-files "*.gn" | grep -v format_test_data | sed -e "s/^/gn format --in-place /" >x.sh $ chmod +x x.sh $ ./x.sh BUG=None R=dpranke@chromium.org,scottmg@chromium.org TBR=brettw@chromium.org Review URL: https://codereview.chromium.org/1415753003 Cr-Original-Commit-Position: refs/heads/master@{#357493} Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src Cr-Mirrored-Commit: f7698e039267b2d07401506006c0f68bfbfe2453
-
- Oct 13, 2015
-
-
brucedawson authored
The recent change to crc_folding.c was added to google.patch but this was inappropriate because crc_folding.c is not part of the original zlib distribution. This was noticed by scottmg@ R=gavinp@chromium.org BUG=440500 Review URL: https://codereview.chromium.org/1397813003 Cr-Original-Commit-Position: refs/heads/master@{#353760} Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src Cr-Mirrored-Commit: 93a8d6c6a00e690fc4fcdc55d016d0ac8c6fab7c
-
- Oct 07, 2015
-
-
brucedawson authored
VC++ 2015 64-bit builds were giving this warning: crc_folding.c(286): warning C4311: 'type cast': pointer truncation from 'const unsigned char *' to 'unsigned long' Converting from unsigned char* to long is normally dodgy but is safe in this case because of the masking with 0xF. Casting through uintptr_t is sufficient to allay VC++'s fears that we are making a mistake. R=pkasting@chromium.org,gavinp@chromium.org BUG=440500 Review URL: https://codereview.chromium.org/1384773002 Cr-Original-Commit-Position: refs/heads/master@{#352900} Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src Cr-Mirrored-Commit: b474ca01fcc5376eb538f71e524351e52319d396
-
- Sep 02, 2015
-
-
brettw authored
People sometimes nest targets or configs, usually with the assumption that this limits the visibility of a config to within a target. But this nesting provides no visibility restrictions over declaring it outside of a block. Un-nest for clarity. Re-land of http://crrev.com/1318823008 while preserving config ordering. Review URL: https://codereview.chromium.org/1307223010 Cr-Original-Commit-Position: refs/heads/master@{#346964} Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src Cr-Mirrored-Commit: 6dd952c77209619a445205998173fc40fb817be1
-
- Sep 01, 2015
-
-
thakis authored
Revert of Un-nest configs in GN files. (patchset #3 id:40001 of https://codereview.chromium.org/1318823008/ ) Reason for revert: This breaks why these configs exist in the first place and seriously breaks the clang/win gn build (which is what these configs exist for). Original issue's description: > Un-nest configs in GN files. > > People sometimes nest targets or configs, usually with the assumption that this limits the visibility of a config to within a target. But this nesting provides no visibility restrictions over declaring it outside of a block. > > Un-nest for clarity. > > Committed: https://crrev.com/4af2eac8e84692d94f88504ab6e0b244b88dddcb > Cr-Commit-Position: refs/heads/master@{#346461} TBR=dpranke@chromium.org,brettw@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true Review URL: https://codereview.chromium.org/1316843006 Cr-Original-Commit-Position: refs/heads/master@{#346695} Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src Cr-Mirrored-Commit: 0cb5abbf230f3c30e3499d4e787f059ed723dd3c
-
- Aug 31, 2015
-
-
brettw authored
People sometimes nest targets or configs, usually with the assumption that this limits the visibility of a config to within a target. But this nesting provides no visibility restrictions over declaring it outside of a block. Un-nest for clarity. Review URL: https://codereview.chromium.org/1318823008 Cr-Original-Commit-Position: refs/heads/master@{#346461} Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src Cr-Mirrored-Commit: 4af2eac8e84692d94f88504ab6e0b244b88dddcb
-
- Aug 07, 2015
-
-
mostynb authored
Review URL: https://codereview.chromium.org/1279843003 Cr-Original-Commit-Position: refs/heads/master@{#342346} Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src Cr-Mirrored-Commit: 7ea99fda67bf56e494f372c6b3a278f00bed364f
-
- Jul 16, 2015
-
-
brettw authored
This replaces it with base::StartsWith and the appropriate case flag. Since the existing version only ever did case-insensitive tests in ASCII, there should be no behavior change. BUG=506255 TBR=jam Review URL: https://codereview.chromium.org/1242023005 Cr-Original-Commit-Position: refs/heads/master@{#339175} Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src Cr-Mirrored-Commit: 955093110ad64d5ec6f5426991efaa4a680b5d6f
-
brettw authored
This just replaces true -> base::CompareCase::SENSITIVE false -> base::CompareCase::INSENSITIVE_ASCII I checked the insensitive cases to make sure they're not doing anything suspicious. The old version is a sometimes-correct Unicode comparison so converting to INSENSTITIVE_ASCII isn't a no-op. However, generally the prefix/suffix checking is done against a hardcoded string so there were very few cases to actually look at. extensions/browser/api/declarative_webrequest/webrequest_condition_attribute.cc has a not-quite search-and-replace change where I changed the type of a class variable. BUG=506255 TBR=jam Reland of http://crrev.com/1239493005 Review URL: https://codereview.chromium.org/1233043003 Cr-Original-Commit-Position: refs/heads/master@{#339071} Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src Cr-Mirrored-Commit: a7ff1b291cd328e9584779a4bb9101fa9a3c01ad
-
yosin authored
Revert of Remove some legacy versions of StartsWith and EndsWith. (patchset #6 id:100001 of https://codereview.chromium.org/1239493005/) Reason for revert: Compilation error on Windows: FAILED: ninja -t msvc -e environment.x86 -- C:\b\build\goma/gomacc "C:\b\depot_tools\win_toolchain\vs2013_files\VC\bin\amd64_x86\cl.exe" /nologo /showIncludes /FC @obj\chrome\installer\gcapi\gcapi_lib.gcapi.obj.rsp /c ..\..\chrome\installer\gcapi\gcapi.cc /Foobj\chrome\installer\gcapi\gcapi_lib.gcapi.obj /Fdobj\chrome\gcapi_lib.cc.pdb c:\b\build\slave\win-latest-rel\build\src\chrome\installer\gcapi\gcapi.cc(365) : error C3083: 'StartsWith': the symbol to the left of a '::' must be a type c:\b\build\slave\win-latest-rel\build\src\chrome\installer\gcapi\gcapi.cc(365) : error C2039: 'INSENSITIVE_ASCII' : is not a member of 'base' c:\b\build\slave\win-latest-rel\build\src\chrome\installer\gcapi\gcapi.cc(365) : error C2065: 'INSENSITIVE_ASCII' : undeclared identifier ninja: build stopped: subcommand failed. Original issue's description: > Remove some legacy versions of StartsWith and EndsWith. > > This just replaces > true -> base::CompareCase::SENSITIVE > false -> base::CompareCase::INSENSITIVE_ASCII > > I checked the insensitive cases to make sure they're not doing anything suspicious. The old version is a sometimes-correct Unicode comparison so converting to INSENSTITIVE_ASCII isn't a no-op. However, generally the prefix/suffix checking is done against a hardcoded string so there were very few cases to actually look at. > > extensions/browser/api/declarative_webrequest/webrequest_condition_attribute.cc has a not-quite search-and-replace change where I changed the type of a class variable. > > BUG=506255 > TBR=jam > > Committed: https://crrev.com/edce9a33027cc5f73c4866d70e34f690f6720a56 > Cr-Commit-Position: refs/heads/master@{#338996} TBR=jam@chromium.org,brettw@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=506255 Review URL: https://codereview.chromium.org/1233453011 Cr-Original-Commit-Position: refs/heads/master@{#338998} Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src Cr-Mirrored-Commit: ddbde8ff8310abbd529ad5cc43e3991e79106a52
-
brettw authored
This just replaces true -> base::CompareCase::SENSITIVE false -> base::CompareCase::INSENSITIVE_ASCII I checked the insensitive cases to make sure they're not doing anything suspicious. The old version is a sometimes-correct Unicode comparison so converting to INSENSTITIVE_ASCII isn't a no-op. However, generally the prefix/suffix checking is done against a hardcoded string so there were very few cases to actually look at. extensions/browser/api/declarative_webrequest/webrequest_condition_attribute.cc has a not-quite search-and-replace change where I changed the type of a class variable. BUG=506255 TBR=jam Review URL: https://codereview.chromium.org/1239493005 Cr-Original-Commit-Position: refs/heads/master@{#338996} Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src Cr-Mirrored-Commit: edce9a33027cc5f73c4866d70e34f690f6720a56
-
- Jul 10, 2015
-
-
sammc authored
It defines YY_USE_CONST when building mesa to fix this warning, disables -Wincompatible-pointer-types for the other third-party libraries that trigger it and fixes some underspecified dependencies in third_party/libwebp. BUG=504696 Review URL: https://codereview.chromium.org/1226583002 Cr-Original-Commit-Position: refs/heads/master@{#338227} Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src Cr-Mirrored-Commit: c98d446e81bcc18cf4241325c8719ccdc0b7abf3
-
- Jul 01, 2015
-
-
dcheng authored
The Cloud Print Windows service uses WTL, and one of the headers uses an assignment inside a conditional (atlgdi.h), which triggers this clang warning. BUG=505302 TBR=scottbyer@chromium.org,vitalybuka@chromium.org Review URL: https://codereview.chromium.org/1214113010 Cr-Original-Commit-Position: refs/heads/master@{#336964} Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src Cr-Mirrored-Commit: 700bce216cb9399ce306850ee66d3ce5a68c4cfa
-
- Jun 24, 2015
-
-
brettw authored
Moves ReplaceFirstSubstringAfterOffset. It also changes the find/replace arguments to be string pieces (normally these are constants). TBR=gauravsh@chromium.org for chromeos/network Review URL: https://codereview.chromium.org/1200393002 Cr-Original-Commit-Position: refs/heads/master@{#335999} Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src Cr-Mirrored-Commit: e6dae46fc681a8ad62f6937a6ac1cab3d51938a9
-
- Jun 12, 2015
-
-
brettw authored
TBR=jschuh@chromium.org Review URL: https://codereview.chromium.org/1182183003 Cr-Original-Commit-Position: refs/heads/master@{#334284} Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src Cr-Mirrored-Commit: 17b32adf2de02e5a4cdf1c0195bb3b06c721b2ca
-
brettw authored
NOPRESUBMIT=true (no presubmit due to removing base:: from a ScopedAllowIO) Review URL: https://codereview.chromium.org/1172183002 Cr-Original-Commit-Position: refs/heads/master@{#334108} Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src Cr-Mirrored-Commit: 44ce0ec5b05184c4e33d597d4e5a32d9f38b15c2
-
- Jun 09, 2015
-
-
skyostil authored
BUG=465354 Review URL: https://codereview.chromium.org/1176473004 Cr-Original-Commit-Position: refs/heads/master@{#333545} Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src Cr-Mirrored-Commit: 971ecb9a7180e599fa97b5c475f72d5e3f4a713a
-