chromium_src.git - central chromium sources

	Commit message (Collapse)	Author	Age	Files	Lines
*	Remove the presubmit warning when changing code in third_party/mojo/	danakj	2015-11-25	1	-0/+2
\| \| \| \| \| \| \| \| \| \| \| \|	This code is part of the chromium tree, not pulled in by DEPS. So we should let people change the code freely. Currently every time you change this code, a presubmit warning is given. Let's not do that. R=jam, jochen, sky Review URL: https://codereview.chromium.org/1476893004 Cr-Commit-Position: refs/heads/master@{#361745}
*	Don't flag third_party/boringssl as needing README.chromium changes.	davidben	2015-11-19	1	-2/+4
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Unlike most things in third_party and like Blink, BoringSSL is unversioned and maintained by us. It is rolled via DEPS in third_party/boringssl/src, but still has Chromium-specific build files checked into the third_party/boringssl directory. Shush presubmit complaining on DEPS rolls when those files change. The one time we might wish to update README.chromium is if something about the license changes, but that would be signaled via a DEPS change and not picked up by this PRESUBMIT rule anyway. BUG=none Review URL: https://codereview.chromium.org/1454303002 Cr-Commit-Position: refs/heads/master@{#360609}
*	Exempt WebKit from third_party PRESUBMIT check for README updates	Primiano Tucci	2015-09-23	1	-1/+3
\| \| \| \| \| \| \| \| \|	BUG=none R=primiano@chromium.org Review URL: https://codereview.chromium.org/1365653002 . Cr-Commit-Position: refs/heads/master@{#350327}
*	Add checking for license formats to third_party/PRESUBMIT	mnaganov@chromium.org	2014-02-10	1	-3/+43
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Since we expect a certain format for the "License:" field, it's better to check it earlier -- during the presubmit stage, so people won't get surprises later. Also, updated the license checking code to accept multiple licenses if one of them is compatible with Android. BUG=168324 R=dannyb@google.com, mkosiba@chromium.org TBR=darin@chromium.org Review URL: https://codereview.chromium.org/146803006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@250068 0039d316-1c4b-4281-b951-d872f2087c98
*	Fix 2 bugs in third_party/PRESUBMIT.py	tony@chromium.org	2013-07-31	1	-2/+19
\| \| \| \| \| \| \| \| \| \| \| \| \| \|	1) Removing a third_party directory raises an exception since it tries to open the deleted README.chromium file. Instead, try to make sure files are being deleted. 2) Editing PRESUBMIT.py gives errors about not having edited README.chromium. Ignore the files in third_party that are not directories. TBR=cdn@chromium.org Review URL: https://codereview.chromium.org/21217004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@214759 0039d316-1c4b-4281-b951-d872f2087c98
*	Make 'License' field in third-party metadata required	steveblock@chromium.org	2012-08-01	1	-2/+11
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	This will simplify the addition of a tool to check licenses for the purpose of the Android WebView build. See also http://codereview.chromium.org/10827099 Also adds other missing fields to these README.chromium files as required by presubmit checks and fixes a regex used to enforce this. BUG=138921 Review URL: https://chromiumcodereview.appspot.com/10821103 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149423 0039d316-1c4b-4281-b951-d872f2087c98
*	Make license presubmit check less strict	thakis@chromium.org	2011-04-29	1	-2/+2
\| \| \| \| \| \| \| \| \| \| \|	Allow spaces in the "Name:" field and colors in "Version:". BUG=none TEST=none Review URL: http://codereview.chromium.org/6893125 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@83543 0039d316-1c4b-4281-b951-d872f2087c98
*	Remove DOS line endings from third_party/PRESUBMIT.py	thakis@chromium.org	2011-04-12	1	-67/+67
\| \| \| \| \| \| \| \| \| \| \|	No behavior change. BUG=none TEST=none Review URL: http://codereview.chromium.org/6825059 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@81184 0039d316-1c4b-4281-b951-d872f2087c98
*	Fix third_party PRESUBMIT script	thakis@chromium.org	2011-04-11	1	-12/+7
\| \| \| \| \| \| \| \| \| \| \| \| \|	It looked at the wrong file contents for its checks, and it had a redundant check for name. The "Security Critical" text had a different spelling than in README.chromium.template. BUG=none TEST=none Review URL: http://codereview.chromium.org/6820055 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@81181 0039d316-1c4b-4281-b951-d872f2087c98
*	Fixing third_party presubmit rules to recognize \r\n as newlines	cdn@chromium.org	2011-04-07	1	-4/+4
\| \| \| \| \| \| \| \|	BUG=N/A TEST=N/A Review URL: http://codereview.chromium.org/6816003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@80727 0039d316-1c4b-4281-b951-d872f2087c98
*	Add presubmit checks to verify that README.chromium files have been updated ↵	cdn@chromium.org	2011-03-31	1	-0/+72
	and contain the proper meta data when files within third_party are modified. Also fixed _CheckNoInterfacesInBase() which was broken on win32 as the path seperator was hardcoded. BUG=None TEST=N/A Review URL: http://codereview.chromium.org/6713009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@80051 0039d316-1c4b-4281-b951-d872f2087c98