Commit f08b0052 authored by Justin Cohen's avatar Justin Cohen Committed by Commit Bot

Reland "Enable -Wextra-semi in iOS builds."

This reverts commit b89cc617.

Bug: 936211, 926235
Change-Id: Ia62f48168f41302582a607b2695cb1b379529945
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1506880
Auto-Submit: Justin Cohen <justincohen@chromium.org>
Reviewed-by: default avatarNico Weber <thakis@chromium.org>
Reviewed-by: default avatarEugene But <eugenebut@chromium.org>
Commit-Queue: Eugene But <eugenebut@chromium.org>
Commit-Queue: Justin Cohen <justincohen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#638414}
parent 271e639f
...@@ -1592,7 +1592,7 @@ config("chromium_code") { ...@@ -1592,7 +1592,7 @@ config("chromium_code") {
# Fuchsia: https://crbug.com/935588 # Fuchsia: https://crbug.com/935588
# iOS: https://crbug.com/936211 # iOS: https://crbug.com/936211
has_dchecks = is_debug || dcheck_always_on has_dchecks = is_debug || dcheck_always_on
if (has_dchecks && !is_fuchsia && !is_ios) { if (has_dchecks && !is_fuchsia) {
cflags += [ "-Wextra-semi" ] cflags += [ "-Wextra-semi" ]
} }
} }
......
...@@ -149,8 +149,8 @@ class OmniboxViewIOS : public OmniboxView, ...@@ -149,8 +149,8 @@ class OmniboxViewIOS : public OmniboxView,
void EmphasizeURLComponents() override; void EmphasizeURLComponents() override;
private: private:
void SetEmphasis(bool emphasize, const gfx::Range& range) override{}; void SetEmphasis(bool emphasize, const gfx::Range& range) override {}
void UpdateSchemeStyle(const gfx::Range& scheme_range) override{}; void UpdateSchemeStyle(const gfx::Range& scheme_range) override {}
// Calculates text attributes according to |display_text| and // Calculates text attributes according to |display_text| and
// returns them in an autoreleased object. // returns them in an autoreleased object.
......
...@@ -32,7 +32,7 @@ bool TestWebClient::IsAppSpecificURL(const GURL& url) const { ...@@ -32,7 +32,7 @@ bool TestWebClient::IsAppSpecificURL(const GURL& url) const {
return url.SchemeIs(kTestWebUIScheme) || return url.SchemeIs(kTestWebUIScheme) ||
url.SchemeIs(kTestNativeContentScheme) || url.SchemeIs(kTestNativeContentScheme) ||
url.SchemeIs(kTestAppSpecificScheme); url.SchemeIs(kTestAppSpecificScheme);
}; }
base::string16 TestWebClient::GetPluginNotSupportedText() const { base::string16 TestWebClient::GetPluginNotSupportedText() const {
return plugin_not_supported_text_; return plugin_not_supported_text_;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment