Skip to content

Commit

Permalink
build: Enable auto raw pointer deduction check everywhere on clang.
Browse files Browse the repository at this point in the history
This patch enables the raw pointer deduction check on every platform.

R=danakj@chromium.org, thakis@chromium.org
BUG=554600

Review-Url: https://codereview.chromium.org/2723253003
Cr-Commit-Position: refs/heads/master@{#454453}
  • Loading branch information
vmpstr authored and Commit bot committed Mar 3, 2017
1 parent f4a6dbf commit 27c7894
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 12 deletions.
14 changes: 4 additions & 10 deletions build/config/clang/BUILD.gn
Original file line number Diff line number Diff line change
Expand Up @@ -35,18 +35,12 @@ config("find_bad_constructs") {
"-add-plugin",
"-Xclang",
"find-bad-constructs",
"-Xclang",
"-plugin-arg-find-bad-constructs",
"-Xclang",
"check-auto-raw-pointer",
]

# TODO(vmpstr): We need to enable this everywhere. crbug.com/554600
if (is_linux && !is_chromeos && !is_chromecast) {
cflags += [
"-Xclang",
"-plugin-arg-find-bad-constructs",
"-Xclang",
"check-auto-raw-pointer",
]
}

if ((is_linux || is_android) && !is_chromecast) {
cflags += [
"-Xclang",
Expand Down
2 changes: 1 addition & 1 deletion ios/chrome/browser/tabs/tab.mm
Original file line number Diff line number Diff line change
Expand Up @@ -1397,7 +1397,7 @@ - (void)webWillFinishHistoryNavigationFromEntry:(CRWSessionEntry*)fromEntry {
- (void)webState:(web::WebState*)webState
didFinishNavigation:(web::NavigationContext*)navigation {
if (navigation->IsSamePage()) {
auto faviconDriver = favicon::WebFaviconDriver::FromWebState(webState);
auto* faviconDriver = favicon::WebFaviconDriver::FromWebState(webState);
if (faviconDriver) {
// Fetch the favicon for the new URL.
faviconDriver->FetchFavicon(navigation->GetUrl());
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -100,7 +100,7 @@ bool IsObservingSources() {
EXPECT_FALSE(GetBlockedPopupTabHelper()->ShouldBlockPopup(source_url));

// Verify that child window was open.
auto open_url_request = web_state_delegate_.last_open_url_request();
auto* open_url_request = web_state_delegate_.last_open_url_request();
ASSERT_TRUE(open_url_request);
EXPECT_EQ(web_state(), open_url_request->web_state);
WebState::OpenURLParams params = open_url_request->params;
Expand Down

0 comments on commit 27c7894

Please sign in to comment.