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

Side by Side Diff: tools/gn/setup.cc

Issue 516683002: Add GN variables for controlling header checking. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@desc
Patch Set: Created 6 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright (c) 2013 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "tools/gn/setup.h" 5 #include "tools/gn/setup.h"
6 6
7 #include <stdlib.h> 7 #include <stdlib.h>
8 8
9 #include <algorithm> 9 #include <algorithm>
10 #include <sstream> 10 #include <sstream>
(...skipping 172 matching lines...) Expand 10 before | Expand all | Expand 10 after
183 // TODO(brettw) implement a system of warnings. Until we have a better 183 // TODO(brettw) implement a system of warnings. Until we have a better
184 // system, print the error but don't return failure. 184 // system, print the error but don't return failure.
185 err.PrintToStdout(); 185 err.PrintToStdout();
186 return true; 186 return true;
187 } 187 }
188 } 188 }
189 189
190 if (check_public_headers_) { 190 if (check_public_headers_) {
191 if (!commands::CheckPublicHeaders(&build_settings_, 191 if (!commands::CheckPublicHeaders(&build_settings_,
192 builder_->GetAllResolvedTargets(), 192 builder_->GetAllResolvedTargets(),
193 std::vector<const Target*>())) { 193 std::vector<const Target*>(),
194 false)) {
194 return false; 195 return false;
195 } 196 }
196 } 197 }
197 198
198 // Write out tracing and timing if requested. 199 // Write out tracing and timing if requested.
199 const CommandLine* cmdline = CommandLine::ForCurrentProcess(); 200 const CommandLine* cmdline = CommandLine::ForCurrentProcess();
200 if (cmdline->HasSwitch(kTimeSwitch)) 201 if (cmdline->HasSwitch(kTimeSwitch))
201 PrintLongHelp(SummarizeTraces()); 202 PrintLongHelp(SummarizeTraces());
202 if (cmdline->HasSwitch(kTracelogSwitch)) 203 if (cmdline->HasSwitch(kTracelogSwitch))
203 SaveTraces(cmdline->GetSwitchValuePath(kTracelogSwitch)); 204 SaveTraces(cmdline->GetSwitchValuePath(kTracelogSwitch));
(...skipping 384 matching lines...) Expand 10 before | Expand all | Expand 10 after
588 } 589 }
589 590
590 void DependentSetup::RunPreMessageLoop() { 591 void DependentSetup::RunPreMessageLoop() {
591 CommonSetup::RunPreMessageLoop(); 592 CommonSetup::RunPreMessageLoop();
592 } 593 }
593 594
594 bool DependentSetup::RunPostMessageLoop() { 595 bool DependentSetup::RunPostMessageLoop() {
595 return CommonSetup::RunPostMessageLoop(); 596 return CommonSetup::RunPostMessageLoop();
596 } 597 }
597 598
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698