diff --git a/apps/ui/views/app_window_frame_view.h b/apps/ui/views/app_window_frame_view.h index ff6301356c049a..b9c1490fb8e036 100644 --- a/apps/ui/views/app_window_frame_view.h +++ b/apps/ui/views/app_window_frame_view.h @@ -59,9 +59,7 @@ class AppWindowFrameView : public views::NonClientFrameView, void SetResizeSizes(int resize_inside_bounds_size, int resize_outside_bounds_size, int resize_area_corner_size); - int resize_inside_bounds_size() const { - return resize_inside_bounds_size_; - }; + int resize_inside_bounds_size() const { return resize_inside_bounds_size_; } private: // views::NonClientFrameView implementation. diff --git a/cc/animation/element_animations.h b/cc/animation/element_animations.h index 8d960dcee7b858..e7fc49c5c65b59 100644 --- a/cc/animation/element_animations.h +++ b/cc/animation/element_animations.h @@ -135,10 +135,10 @@ class CC_ANIMATION_EXPORT ElementAnimations KeyframeModel* keyframe_model) override; void NotifyClientSizeAnimated(const gfx::SizeF& size, int target_property_id, - KeyframeModel* keyframe_model) override{}; + KeyframeModel* keyframe_model) override {} void NotifyClientColorAnimated(SkColor color, int target_property_id, - KeyframeModel* keyframe_model) override{}; + KeyframeModel* keyframe_model) override {} void NotifyClientTransformOperationsAnimated( const TransformOperations& operations, int target_property_id, diff --git a/cc/layers/painted_scrollbar_layer.cc b/cc/layers/painted_scrollbar_layer.cc index 214a33ba52cb10..11f22ae1c9f9a1 100644 --- a/cc/layers/painted_scrollbar_layer.cc +++ b/cc/layers/painted_scrollbar_layer.cc @@ -13,7 +13,7 @@ namespace { static constexpr int kMaxScrollbarDimension = 8192; -}; +} namespace cc { diff --git a/cc/paint/paint_op_buffer.cc b/cc/paint/paint_op_buffer.cc index a4b97445137713..a1f407ad8188c2 100644 --- a/cc/paint/paint_op_buffer.cc +++ b/cc/paint/paint_op_buffer.cc @@ -206,13 +206,13 @@ static bool g_has_paint_flags[kNumOpTypes] = {TYPES(M)}; #define M(T) \ static_assert(sizeof(T) <= sizeof(LargestPaintOp), \ #T " must be no bigger than LargestPaintOp"); -TYPES(M); +TYPES(M) #undef M #define M(T) \ static_assert(alignof(T) <= PaintOpBuffer::PaintOpAlign, \ #T " must have alignment no bigger than PaintOpAlign"); -TYPES(M); +TYPES(M) #undef M using AnalyzeOpFunc = void (*)(PaintOpBuffer*, const PaintOp*); diff --git a/cc/trees/image_animation_controller.h b/cc/trees/image_animation_controller.h index ec73e0419a14ae..a3efaee1433054 100644 --- a/cc/trees/image_animation_controller.h +++ b/cc/trees/image_animation_controller.h @@ -99,7 +99,7 @@ class CC_EXPORT ImageAnimationController { size_t GetFrameIndexForImage(PaintImage::Id paint_image_id, WhichTree tree) const; - void set_did_navigate() { did_navigate_ = true; }; + void set_did_navigate() { did_navigate_ = true; } const base::flat_set& GetDriversForTesting( PaintImage::Id paint_image_id) const; diff --git a/chrome/browser/signin/chrome_signin_helper.h b/chrome/browser/signin/chrome_signin_helper.h index 63471d237e47ca..71eccf4838b6e0 100644 --- a/chrome/browser/signin/chrome_signin_helper.h +++ b/chrome/browser/signin/chrome_signin_helper.h @@ -83,6 +83,6 @@ void ProcessAccountConsistencyResponseHeaders(ResponseAdapter* response, const GURL& redirect_url, bool is_off_the_record); -}; // namespace signin +} // namespace signin #endif // CHROME_BROWSER_SIGNIN_CHROME_SIGNIN_HELPER_H_ diff --git a/chrome/browser/ui/profile_chooser_constants.h b/chrome/browser/ui/profile_chooser_constants.h index 92934d72315e8d..8b69c3ef0fd859 100644 --- a/chrome/browser/ui/profile_chooser_constants.h +++ b/chrome/browser/ui/profile_chooser_constants.h @@ -27,6 +27,6 @@ enum BubbleViewMode { BUBBLE_VIEW_MODE_ACCOUNT_REMOVAL, }; -}; // namespace profiles +} // namespace profiles #endif // CHROME_BROWSER_UI_PROFILE_CHOOSER_CONSTANTS_H_ diff --git a/chrome/browser/ui/views/location_bar/location_bar_bubble_delegate_view.h b/chrome/browser/ui/views/location_bar/location_bar_bubble_delegate_view.h index 667caa465bb572..12119148fe0763 100644 --- a/chrome/browser/ui/views/location_bar/location_bar_bubble_delegate_view.h +++ b/chrome/browser/ui/views/location_bar/location_bar_bubble_delegate_view.h @@ -17,7 +17,7 @@ namespace content { class NotificationDetails; class NotificationSource; class WebContents; -}; +} // namespace content // Base class for bubbles that are shown from location bar icons. The bubble // will automatically close when the browser transitions in or out of fullscreen diff --git a/components/contextual_search/core/browser/contextual_search_preference.h b/components/contextual_search/core/browser/contextual_search_preference.h index 69fcced6e77f59..7c99d43669363d 100644 --- a/components/contextual_search/core/browser/contextual_search_preference.h +++ b/components/contextual_search/core/browser/contextual_search_preference.h @@ -36,7 +36,7 @@ bool IsEnabled(const PrefService& prefs); class ContextualSearchPreference { public: // No public constructor, use |GetInstance|. - ~ContextualSearchPreference(){}; + ~ContextualSearchPreference() {} // Returns the singleton instance of this class, created when needed. static ContextualSearchPreference* GetInstance(); diff --git a/components/offline_pages/core/model/clear_digest_task.cc b/components/offline_pages/core/model/clear_digest_task.cc index 3b4ca6ba976b98..e78c0b1d766b2f 100644 --- a/components/offline_pages/core/model/clear_digest_task.cc +++ b/components/offline_pages/core/model/clear_digest_task.cc @@ -31,7 +31,7 @@ ClearDigestTask::ClearDigestTask(OfflinePageMetadataStore* store, DCHECK(store_); } -ClearDigestTask::~ClearDigestTask(){}; +ClearDigestTask::~ClearDigestTask() {} void ClearDigestTask::Run() { store_->Execute(base::BindOnce(&ClearDigestSync, offline_id_), diff --git a/components/offline_pages/core/model/mark_page_accessed_task.cc b/components/offline_pages/core/model/mark_page_accessed_task.cc index 9e3de26906708f..7e25747c1344a8 100644 --- a/components/offline_pages/core/model/mark_page_accessed_task.cc +++ b/components/offline_pages/core/model/mark_page_accessed_task.cc @@ -82,7 +82,7 @@ MarkPageAccessedTask::MarkPageAccessedTask(OfflinePageMetadataStore* store, DCHECK(store_); } -MarkPageAccessedTask::~MarkPageAccessedTask(){}; +MarkPageAccessedTask::~MarkPageAccessedTask() {} void MarkPageAccessedTask::Run() { store_->Execute( diff --git a/components/offline_pages/core/model/update_file_path_task.cc b/components/offline_pages/core/model/update_file_path_task.cc index c96e77a9a26e43..a243a2e0576621 100644 --- a/components/offline_pages/core/model/update_file_path_task.cc +++ b/components/offline_pages/core/model/update_file_path_task.cc @@ -58,7 +58,7 @@ UpdateFilePathTask::UpdateFilePathTask(OfflinePageMetadataStore* store, DCHECK(store_); } -UpdateFilePathTask::~UpdateFilePathTask(){}; +UpdateFilePathTask::~UpdateFilePathTask() {} void UpdateFilePathTask::Run() { store_->Execute(base::BindOnce(&UpdateFilePathSync, file_path_, offline_id_), diff --git a/components/offline_pages/core/offline_page_client_policy.h b/components/offline_pages/core/offline_page_client_policy.h index 08204a4b3e37d2..b403c153b4f363 100644 --- a/components/offline_pages/core/offline_page_client_policy.h +++ b/components/offline_pages/core/offline_page_client_policy.h @@ -105,7 +105,7 @@ struct OfflinePageClientPolicy { : name_space(namespace_val), lifetime_policy(lifetime_policy_val), pages_allowed_per_url(pages_allowed_per_url_val), - feature_policy(feature_policy_val){}; + feature_policy(feature_policy_val) {} OfflinePageClientPolicy(std::string namespace_val, LifetimePolicy lifetime_policy_val, diff --git a/components/query_parser/query_parser.cc b/components/query_parser/query_parser.cc index 0686489b3f837b..5654c858845d84 100644 --- a/components/query_parser/query_parser.cc +++ b/components/query_parser/query_parser.cc @@ -72,7 +72,7 @@ class QueryNodeWord : public QueryNode { const base::string16& word() const { return word_; } - bool literal() const { return literal_; }; + bool literal() const { return literal_; } void set_literal(bool literal) { literal_ = literal; } // QueryNode: diff --git a/components/safe_browsing/db/v4_rice.cc b/components/safe_browsing/db/v4_rice.cc index 1a3f83f8185f47..547a40206f96c7 100644 --- a/components/safe_browsing/db/v4_rice.cc +++ b/components/safe_browsing/db/v4_rice.cc @@ -273,7 +273,7 @@ uint32_t V4RiceDecoder::GetBitsFromCurrentWord( current_word_ = current_word_ >> num_requested_bits; current_word_bit_index_ += num_requested_bits; return x; -}; +} std::string V4RiceDecoder::DebugString() const { // Calculates the total number of bits that we have read from the buffer, diff --git a/components/translate/content/browser/content_translate_driver.h b/components/translate/content/browser/content_translate_driver.h index faeead5e71528d..9b05c7e389578f 100644 --- a/components/translate/content/browser/content_translate_driver.h +++ b/components/translate/content/browser/content_translate_driver.h @@ -41,20 +41,20 @@ class ContentTranslateDriver : public TranslateDriver, class Observer { public: // Handles when the value of IsPageTranslated is changed. - virtual void OnIsPageTranslatedChanged(content::WebContents* source) {}; + virtual void OnIsPageTranslatedChanged(content::WebContents* source) {} // Handles when the value of translate_enabled is changed. - virtual void OnTranslateEnabledChanged(content::WebContents* source) {}; + virtual void OnTranslateEnabledChanged(content::WebContents* source) {} // Called when the page language has been determined. virtual void OnLanguageDetermined( - const translate::LanguageDetectionDetails& details) {}; + const translate::LanguageDetectionDetails& details) {} // Called when the page has been translated. - virtual void OnPageTranslated( - const std::string& original_lang, - const std::string& translated_lang, - translate::TranslateErrors::Type error_type) {}; + virtual void OnPageTranslated(const std::string& original_lang, + const std::string& translated_lang, + translate::TranslateErrors::Type error_type) { + } protected: virtual ~Observer() {} diff --git a/components/translate/core/browser/translate_client.h b/components/translate/core/browser/translate_client.h index 23f139a0ea6baf..67986394de2528 100644 --- a/components/translate/core/browser/translate_client.h +++ b/components/translate/core/browser/translate_client.h @@ -40,7 +40,7 @@ struct LanguageDetectionDetails; // TranslateManager is used (e.g. a single tab). class TranslateClient { public: - virtual ~TranslateClient() {}; + virtual ~TranslateClient() {} // Gets the TranslateDriver associated with the client. virtual TranslateDriver* GetTranslateDriver() = 0; diff --git a/content/browser/accessibility/one_shot_accessibility_tree_search.h b/content/browser/accessibility/one_shot_accessibility_tree_search.h index 3080a5d0ff0470..055cc5b19ca024 100644 --- a/content/browser/accessibility/one_shot_accessibility_tree_search.h +++ b/content/browser/accessibility/one_shot_accessibility_tree_search.h @@ -24,9 +24,9 @@ typedef bool (*AccessibilityMatchPredicate)( BrowserAccessibility* start_element, BrowserAccessibility* this_element); -#define DECLARE_ACCESSIBILITY_PREDICATE(PredicateName) \ - bool PredicateName(BrowserAccessibility* start_element, \ - BrowserAccessibility* this_element); +#define DECLARE_ACCESSIBILITY_PREDICATE(PredicateName) \ + bool PredicateName(BrowserAccessibility* start_element, \ + BrowserAccessibility* this_element) DECLARE_ACCESSIBILITY_PREDICATE(AccessibilityArticlePredicate); DECLARE_ACCESSIBILITY_PREDICATE(AccessibilityBlockquotePredicate); diff --git a/content/browser/appcache/appcache_url_request_job.h b/content/browser/appcache/appcache_url_request_job.h index ad8c9f7afb3242..67c324291fac40 100644 --- a/content/browser/appcache/appcache_url_request_job.h +++ b/content/browser/appcache/appcache_url_request_job.h @@ -18,7 +18,7 @@ namespace net { class GrowableIOBuffer; -}; +} namespace content { class AppCacheHost; diff --git a/device/udev_linux/udev_watcher.cc b/device/udev_linux/udev_watcher.cc index a2bf268e9f9c60..44c480fbe9d046 100644 --- a/device/udev_linux/udev_watcher.cc +++ b/device/udev_linux/udev_watcher.cc @@ -48,7 +48,7 @@ std::unique_ptr UdevWatcher::StartWatching(Observer* observer) { UdevWatcher::~UdevWatcher() { DCHECK(sequence_checker_.CalledOnValidSequence()); -}; +} void UdevWatcher::EnumerateExistingDevices() { DCHECK(sequence_checker_.CalledOnValidSequence()); diff --git a/gpu/command_buffer/client/gles2_interface.h b/gpu/command_buffer/client/gles2_interface.h index 85aa9546c2f982..e7194af8fefe8d 100644 --- a/gpu/command_buffer/client/gles2_interface.h +++ b/gpu/command_buffer/client/gles2_interface.h @@ -34,7 +34,7 @@ class GLES2Interface { GLES2Interface() = default; virtual ~GLES2Interface() = default; - virtual void FreeSharedMemory(void*) {}; + virtual void FreeSharedMemory(void*) {} // Include the auto-generated part of this class. We split this because // it means we can easily edit the non-auto generated parts right here in diff --git a/gpu/command_buffer/client/transfer_buffer.h b/gpu/command_buffer/client/transfer_buffer.h index 954621d8b6213d..39707e23409d5d 100644 --- a/gpu/command_buffer/client/transfer_buffer.h +++ b/gpu/command_buffer/client/transfer_buffer.h @@ -284,7 +284,7 @@ class ScopedResultPtr { other.result_ = nullptr; other.transfer_buffer_ = nullptr; return *this; - }; + } // Dereferencing behaviors T& operator*() const { return *result_; } diff --git a/media/base/ipc/media_param_traits_macros.h b/media/base/ipc/media_param_traits_macros.h index e1a6a2bd4cb5db..0b2d13a182e931 100644 --- a/media/base/ipc/media_param_traits_macros.h +++ b/media/base/ipc/media_param_traits_macros.h @@ -118,17 +118,16 @@ IPC_ENUM_TRAITS_MAX_VALUE(media::OutputDeviceStatus, media::OUTPUT_DEVICE_STATUS_MAX) IPC_ENUM_TRAITS_MAX_VALUE(media::PipelineStatus, - media::PipelineStatus::PIPELINE_STATUS_MAX); + media::PipelineStatus::PIPELINE_STATUS_MAX) IPC_ENUM_TRAITS_MAX_VALUE(media::SampleFormat, media::kSampleFormatMax) IPC_ENUM_TRAITS_MAX_VALUE(media::VideoCodec, media::kVideoCodecMax) -IPC_ENUM_TRAITS_MAX_VALUE(media::WaitingReason, - media::WaitingReason::kMaxValue); +IPC_ENUM_TRAITS_MAX_VALUE(media::WaitingReason, media::WaitingReason::kMaxValue) IPC_ENUM_TRAITS_MAX_VALUE(media::WatchTimeKey, - media::WatchTimeKey::kWatchTimeKeyMax); + media::WatchTimeKey::kWatchTimeKeyMax) IPC_ENUM_TRAITS_MIN_MAX_VALUE(media::VideoCodecProfile, media::VIDEO_CODEC_PROFILE_MIN, @@ -139,7 +138,7 @@ IPC_ENUM_TRAITS_MAX_VALUE(media::VideoPixelFormat, media::PIXEL_FORMAT_MAX) IPC_ENUM_TRAITS_MAX_VALUE(media::VideoRotation, media::VIDEO_ROTATION_MAX) IPC_ENUM_TRAITS_MAX_VALUE(media::container_names::MediaContainerName, - media::container_names::CONTAINER_MAX); + media::container_names::CONTAINER_MAX) #if defined(OS_ANDROID) IPC_ENUM_TRAITS_MIN_MAX_VALUE(media::MediaDrmKeyType, @@ -151,19 +150,19 @@ IPC_ENUM_TRAITS_VALIDATE( media::VideoColorSpace::PrimaryID, static_cast(value) == static_cast( - media::VideoColorSpace::GetPrimaryID(static_cast(value)))); + media::VideoColorSpace::GetPrimaryID(static_cast(value)))) IPC_ENUM_TRAITS_VALIDATE( media::VideoColorSpace::TransferID, static_cast(value) == static_cast( - media::VideoColorSpace::GetTransferID(static_cast(value)))); + media::VideoColorSpace::GetTransferID(static_cast(value)))) IPC_ENUM_TRAITS_VALIDATE( media::VideoColorSpace::MatrixID, static_cast(value) == static_cast( - media::VideoColorSpace::GetMatrixID(static_cast(value)))); + media::VideoColorSpace::GetMatrixID(static_cast(value)))) // Struct traits. diff --git a/media/base/key_systems.h b/media/base/key_systems.h index d9e0205e38495b..c8e54c78b7b8a0 100644 --- a/media/base/key_systems.h +++ b/media/base/key_systems.h @@ -76,7 +76,7 @@ class MEDIA_EXPORT KeySystems { const std::string& key_system) const = 0; protected: - virtual ~KeySystems() {}; + virtual ~KeySystems() {} }; // TODO(ddorwin): WebContentDecryptionModuleSessionImpl::initializeNewSession() diff --git a/media/filters/ffmpeg_bitstream_converter.h b/media/filters/ffmpeg_bitstream_converter.h index 3926bf80187093..dd133f4ea98f17 100644 --- a/media/filters/ffmpeg_bitstream_converter.h +++ b/media/filters/ffmpeg_bitstream_converter.h @@ -14,7 +14,7 @@ namespace media { // Interface for classes that allow reformating of FFmpeg bitstreams class MEDIA_EXPORT FFmpegBitstreamConverter { public: - virtual ~FFmpegBitstreamConverter() {}; + virtual ~FFmpegBitstreamConverter() {} // Reads the data in packet, and then overwrites this data with the // converted version of packet diff --git a/media/formats/mp4/box_definitions.h b/media/formats/mp4/box_definitions.h index 5c9d044277439c..4f78b652d3d628 100644 --- a/media/formats/mp4/box_definitions.h +++ b/media/formats/mp4/box_definitions.h @@ -39,7 +39,7 @@ enum SampleFlags { T(const T& other); \ ~T() override; \ bool Parse(BoxReader* reader) override; \ - FourCC BoxType() const override; + FourCC BoxType() const override struct MEDIA_EXPORT FileType : Box { DECLARE_BOX_METHODS(FileType); diff --git a/media/midi/midi_manager_alsa.h b/media/midi/midi_manager_alsa.h index e293f24eb8f5e4..c9ac0cfb713df0 100644 --- a/media/midi/midi_manager_alsa.h +++ b/media/midi/midi_manager_alsa.h @@ -212,7 +212,7 @@ class MIDI_EXPORT MidiManagerAlsa final : public MidiManager { public: iterator erase(iterator position) { return MidiPortStateBase::erase(position); - }; + } void push_back(std::unique_ptr port) { MidiPortStateBase::push_back(std::move(port)); } @@ -359,7 +359,7 @@ class MIDI_EXPORT MidiManagerAlsa final : public MidiManager { struct SndMidiEventDeleter { void operator()(snd_midi_event_t* coder) const { snd_midi_event_free(coder); - }; + } }; using SourceMap = std::unordered_map; diff --git a/media/midi/task_service.h b/media/midi/task_service.h index 0845b966bdc4dd..2e19b2ca2d7746 100644 --- a/media/midi/task_service.h +++ b/media/midi/task_service.h @@ -104,6 +104,6 @@ class MIDI_EXPORT TaskService final { DISALLOW_COPY_AND_ASSIGN(TaskService); }; -}; // namespace midi +} // namespace midi #endif // MEDIA_MIDI_TASK_SERVICE_H_ diff --git a/media/video/picture.h b/media/video/picture.h index 0dedb51ee554fc..95ce5fc91fe534 100644 --- a/media/video/picture.h +++ b/media/video/picture.h @@ -112,7 +112,7 @@ class MEDIA_EXPORT Picture { // without requesting new PictureBuffers. GpuVideoDecoder should read this // as a signal to update the size of the corresponding PicutreBuffer using // visible_rect() upon receiving this Picture from a VDA. - bool size_changed() const { return size_changed_; }; + bool size_changed() const { return size_changed_; } void set_size_changed(bool size_changed) { size_changed_ = size_changed; } diff --git a/net/base/registry_controlled_domains/registry_controlled_domain.h b/net/base/registry_controlled_domains/registry_controlled_domain.h index 9f3101a5c4ae14..9f86d6fb053c7d 100644 --- a/net/base/registry_controlled_domains/registry_controlled_domain.h +++ b/net/base/registry_controlled_domains/registry_controlled_domain.h @@ -125,7 +125,7 @@ class GURL; namespace url { class Origin; -}; +} struct DomainRule; diff --git a/ppapi/proxy/interface_list.h b/ppapi/proxy/interface_list.h index c71cef4d327a85..eed12630b85fea 100644 --- a/ppapi/proxy/interface_list.h +++ b/ppapi/proxy/interface_list.h @@ -63,7 +63,7 @@ class PPAPI_PROXY_EXPORT InterfaceList { // Permission required to return non-null for this interface. This will // be checked with the value set via SetProcessGlobalPermissionBits when // an interface is requested. - Permission required_permission() { return required_permission_; }; + Permission required_permission() { return required_permission_; } // Call this any time the interface is requested. It will log a UMA count // only the first time. This is safe to call from any thread, regardless of diff --git a/services/media_session/public/cpp/test/mock_media_session.h b/services/media_session/public/cpp/test/mock_media_session.h index 58931adb9d5576..dfbac532a31f2c 100644 --- a/services/media_session/public/cpp/test/mock_media_session.h +++ b/services/media_session/public/cpp/test/mock_media_session.h @@ -101,7 +101,7 @@ class COMPONENT_EXPORT(MEDIA_SESSION_TEST_SUPPORT_CPP) MockMediaSession void GetDebugInfo(GetDebugInfoCallback callback) override; void PreviousTrack() override; void NextTrack() override; - void SkipAd() override{}; + void SkipAd() override {} void Seek(base::TimeDelta seek_time) override; void Stop(SuspendType type) override; diff --git a/skia/config/sk_ref_cnt_ext_debug.h b/skia/config/sk_ref_cnt_ext_debug.h index 8a4e448a3d1301..93250e2a7e660e 100644 --- a/skia/config/sk_ref_cnt_ext_debug.h +++ b/skia/config/sk_ref_cnt_ext_debug.h @@ -47,17 +47,17 @@ inline SkRefCnt::~SkRefCnt() { } // Bootstrap for Blink's WTF::RefPtr namespace WTF { - inline void adopted(const SkRefCnt* object) { - if (!object) - return; - object->adopted(); - } - inline void requireAdoption(const SkRefCnt* object) { - if (!object) - return; - object->requireAdoption(); - } -}; +inline void adopted(const SkRefCnt* object) { + if (!object) + return; + object->adopted(); +} +inline void requireAdoption(const SkRefCnt* object) { + if (!object) + return; + object->requireAdoption(); +} +} // namespace WTF using WTF::adopted; using WTF::requireAdoption; diff --git a/third_party/blink/public/web/web_input_method_controller.h b/third_party/blink/public/web/web_input_method_controller.h index 33c4a739c020be..7d37775a3a5614 100644 --- a/third_party/blink/public/web/web_input_method_controller.h +++ b/third_party/blink/public/web/web_input_method_controller.h @@ -72,7 +72,7 @@ class WebInputMethodController { // Fetches the character range of the current composition, also called the // "marked range." - virtual WebRange CompositionRange() { return WebRange(); }; + virtual WebRange CompositionRange() { return WebRange(); } // Populate |bounds| with the composition character bounds for the ongoing // composition. Returns false if there is no focused input or any ongoing diff --git a/third_party/blink/renderer/core/css/css_value_pair.h b/third_party/blink/renderer/core/css/css_value_pair.h index 92312f72ffb0a6..1a160f1cb35c16 100644 --- a/third_party/blink/renderer/core/css/css_value_pair.h +++ b/third_party/blink/renderer/core/css/css_value_pair.h @@ -54,7 +54,7 @@ class CORE_EXPORT CSSValuePair : public CSSValue { bool KeepIdenticalValues() const { return identical_values_policy_ == kKeepIdenticalValues; - }; + } String CustomCSSText() const { String first = first_->CssText(); diff --git a/third_party/blink/renderer/core/css/css_variable_data.h b/third_party/blink/renderer/core/css/css_variable_data.h index 5d00486c379a5c..dc38000dd3caf5 100644 --- a/third_party/blink/renderer/core/css/css_variable_data.h +++ b/third_party/blink/renderer/core/css/css_variable_data.h @@ -88,7 +88,7 @@ class CORE_EXPORT CSSVariableData : public RefCounted { needs_url_resolution_(false), has_font_units_(false), has_root_font_units_(false), - absolutized_(false){}; + absolutized_(false) {} CSSVariableData(const CSSParserTokenRange&, bool is_animation_tainted, diff --git a/third_party/blink/renderer/core/css/resolver/font_builder.h b/third_party/blink/renderer/core/css/resolver/font_builder.h index 4dde02bc74302a..0f4b775bab13be 100644 --- a/third_party/blink/renderer/core/css/resolver/font_builder.h +++ b/third_party/blink/renderer/core/css/resolver/font_builder.h @@ -109,7 +109,7 @@ class CORE_EXPORT FontBuilder { } static FontVariantNumeric InitialVariantNumeric() { return FontVariantNumeric(); - }; + } static LayoutLocale* InitialLocale() { return nullptr; } static FontDescription::Kerning InitialKerning() { return FontDescription::kAutoKerning; diff --git a/third_party/blink/renderer/core/dom/document.h b/third_party/blink/renderer/core/dom/document.h index 2e0fbe1d2ac66d..2282c92d7339d9 100644 --- a/third_party/blink/renderer/core/dom/document.h +++ b/third_party/blink/renderer/core/dom/document.h @@ -748,7 +748,7 @@ class CORE_EXPORT Document : public ContainerNode, void SetReadyState(DocumentReadyState); bool IsLoadCompleted() const; - bool IsFreezingInProgress() const { return is_freezing_in_progress_; }; + bool IsFreezingInProgress() const { return is_freezing_in_progress_; } enum ParsingState { kParsing, kInDOMContentLoaded, kFinishedParsing }; void SetParsingState(ParsingState); @@ -1676,7 +1676,7 @@ class CORE_EXPORT Document : public ContainerNode, void SetFreezingInProgress(bool is_freezing_in_progress) { is_freezing_in_progress_ = is_freezing_in_progress; - }; + } DocumentLifecycle lifecycle_; diff --git a/third_party/blink/renderer/core/dom/element_traversal.h b/third_party/blink/renderer/core/dom/element_traversal.h index 190988383235c7..7eff38d23fb862 100644 --- a/third_party/blink/renderer/core/dom/element_traversal.h +++ b/third_party/blink/renderer/core/dom/element_traversal.h @@ -227,26 +227,26 @@ template inline TraversalDescendantRange> Traversal::DescendantsOf(const Node& root) { return TraversalDescendantRange>(&root); -}; +} template inline TraversalInclusiveDescendantRange> Traversal::InclusiveDescendantsOf(const ElementType& root) { return TraversalInclusiveDescendantRange>(&root); -}; +} template inline TraversalNextRange> Traversal::StartsAt(const ElementType& start) { return TraversalNextRange>(&start); -}; +} template inline TraversalNextRange> Traversal::StartsAfter(const Node& start) { return TraversalNextRange>( Traversal::Next(start)); -}; +} // Specialized for pure Element to exploit the fact that Elements parent is // always either another Element or the root. diff --git a/third_party/blink/renderer/core/layout/layout_object.h b/third_party/blink/renderer/core/layout/layout_object.h index 29f93927c681bb..1f3cf88c1bd5bf 100644 --- a/third_party/blink/renderer/core/layout/layout_object.h +++ b/third_party/blink/renderer/core/layout/layout_object.h @@ -245,7 +245,7 @@ class CORE_EXPORT LayoutObject : public ImageResourceObserver, bool HasNonZeroEffectiveOpacity() const; protected: - void EnsureIdForTesting() { fragment_.EnsureIdForTesting(); }; + void EnsureIdForTesting() { fragment_.EnsureIdForTesting(); } private: // DisplayItemClient methods. @@ -2504,13 +2504,13 @@ class CORE_EXPORT LayoutObject : public ImageResourceObserver, unsigned as_image_observer_count_ : 20; #endif -#define ADD_BOOLEAN_BITFIELD(field_name_, MethodNameBase) \ - private: \ - unsigned field_name_ : 1; \ - \ - public: \ - bool MethodNameBase() const { return field_name_; } \ - void Set##MethodNameBase(bool new_value) { field_name_ = new_value; } +#define ADD_BOOLEAN_BITFIELD(field_name_, MethodNameBase) \ + public: \ + bool MethodNameBase() const { return field_name_; } \ + void Set##MethodNameBase(bool new_value) { field_name_ = new_value; } \ + \ + private: \ + unsigned field_name_ : 1 class LayoutObjectBitfields { enum PositionedState { diff --git a/third_party/blink/renderer/core/layout/line/inline_box.h b/third_party/blink/renderer/core/layout/line/inline_box.h index 9c2f7868ec5e20..cf148a1ec2ab1e 100644 --- a/third_party/blink/renderer/core/layout/line/inline_box.h +++ b/third_party/blink/renderer/core/layout/line/inline_box.h @@ -379,13 +379,13 @@ class CORE_EXPORT InlineBox : public DisplayItemClient { // invalidation. void SetShouldDoFullPaintInvalidationRecursively(); -#define ADD_BOOLEAN_BITFIELD(field_name_, MethodNameBase) \ - private: \ - unsigned field_name_ : 1; \ - \ - public: \ - bool MethodNameBase() const { return field_name_; } \ - void Set##MethodNameBase(bool new_value) { field_name_ = new_value; } +#define ADD_BOOLEAN_BITFIELD(field_name_, MethodNameBase) \ + public: \ + bool MethodNameBase() const { return field_name_; } \ + void Set##MethodNameBase(bool new_value) { field_name_ = new_value; } \ + \ + private: \ + unsigned field_name_ : 1 class InlineBoxBitfields { DISALLOW_NEW(); diff --git a/third_party/blink/renderer/platform/fonts/opentype/font_settings.h b/third_party/blink/renderer/platform/fonts/opentype/font_settings.h index 835e92e328b9d7..316dc2bca73443 100644 --- a/third_party/blink/renderer/platform/fonts/opentype/font_settings.h +++ b/third_party/blink/renderer/platform/fonts/opentype/font_settings.h @@ -24,10 +24,10 @@ class FontTagValuePair { public: FontTagValuePair(const AtomicString& tag, T value) - : tag_(tag), value_(value){}; + : tag_(tag), value_(value) {} bool operator==(const FontTagValuePair& other) const { return tag_ == other.tag_ && value_ == other.value_; - }; + } const AtomicString& Tag() const { return tag_; } T Value() const { return value_; } @@ -46,7 +46,7 @@ class FontSettings { const T& at(wtf_size_t index) const { return list_.at(index); } bool operator==(const FontSettings& other) const { return list_ == other.list_; - }; + } String ToString() const { StringBuilder builder; wtf_size_t num_features = size(); diff --git a/third_party/crashpad/crashpad/client/crash_report_database.h b/third_party/crashpad/crashpad/client/crash_report_database.h index d115c74b9ee856..1d6a9ed03a8e21 100644 --- a/third_party/crashpad/crashpad/client/crash_report_database.h +++ b/third_party/crashpad/crashpad/client/crash_report_database.h @@ -163,7 +163,7 @@ class CrashReportDatabase { //! This is not implemented on macOS or Windows. std::map GetAttachments() const { return attachment_map_; - }; + } private: friend class CrashReportDatabase; diff --git a/ui/gfx/ipc/color/gfx_param_traits_macros.h b/ui/gfx/ipc/color/gfx_param_traits_macros.h index 7c1ba7c6da6b65..b500d58a46782c 100644 --- a/ui/gfx/ipc/color/gfx_param_traits_macros.h +++ b/ui/gfx/ipc/color/gfx_param_traits_macros.h @@ -16,13 +16,13 @@ #define IPC_MESSAGE_EXPORT GFX_IPC_COLOR_EXPORT IPC_ENUM_TRAITS_MAX_VALUE(gfx::ColorSpace::PrimaryID, - gfx::ColorSpace::PrimaryID::LAST); + gfx::ColorSpace::PrimaryID::LAST) IPC_ENUM_TRAITS_MAX_VALUE(gfx::ColorSpace::TransferID, - gfx::ColorSpace::TransferID::LAST); + gfx::ColorSpace::TransferID::LAST) IPC_ENUM_TRAITS_MAX_VALUE(gfx::ColorSpace::MatrixID, - gfx::ColorSpace::MatrixID::LAST); + gfx::ColorSpace::MatrixID::LAST) IPC_ENUM_TRAITS_MAX_VALUE(gfx::ColorSpace::RangeID, - gfx::ColorSpace::RangeID::LAST); + gfx::ColorSpace::RangeID::LAST) #undef IPC_MESSAGE_EXPORT #define IPC_MESSAGE_EXPORT