Skip to content

Commit

Permalink
clang plugin: Remove enforce-in-pdf toggle.
Browse files Browse the repository at this point in the history
Flipped on in r413689.

Review-Url: https://codereview.chromium.org/2268203002
Cr-Commit-Position: refs/heads/master@{#413772}
  • Loading branch information
leizleiz authored and Commit bot committed Aug 23, 2016
1 parent 34b9df2 commit 30a7849
Show file tree
Hide file tree
Showing 3 changed files with 2 additions and 6 deletions.
4 changes: 0 additions & 4 deletions tools/clang/plugins/ChromeClassTester.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -224,10 +224,6 @@ void ChromeClassTester::BuildBannedLists() {
allowed_directories_.emplace("/third_party/WebKit/");
}

if (!options_.enforce_in_pdf) {
banned_directories_.emplace("/pdf/");
}

banned_directories_.emplace("/third_party/");
banned_directories_.emplace("/native_client/");
banned_directories_.emplace("/breakpad/");
Expand Down
3 changes: 2 additions & 1 deletion tools/clang/plugins/FindBadConstructsAction.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -48,7 +48,8 @@ bool FindBadConstructsAction::ParseArgs(const CompilerInstance& instance,
// TODO(rsleevi): Remove this once http://crbug.com/123295 is fixed.
options_.check_base_classes = true;
} else if (args[i] == "enforce-in-pdf") {
options_.enforce_in_pdf = true;
// TODO(thestig): This is now always on. Remove support for this flag once
// the build no longer passes it in.
} else if (args[i] == "enforce-in-thirdparty-webkit") {
options_.enforce_in_thirdparty_webkit = true;
} else if (args[i] == "check-enum-last-value") {
Expand Down
1 change: 0 additions & 1 deletion tools/clang/plugins/Options.h
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,6 @@ namespace chrome_checker {

struct Options {
bool check_base_classes = false;
bool enforce_in_pdf = false;
bool enforce_in_thirdparty_webkit = false; // Use in Blink code itself
bool check_enum_last_value = false;
// This is needed for some distributed build-sytems to respect banned
Expand Down

0 comments on commit 30a7849

Please sign in to comment.