Commit 30d05d5f authored by Mohammad Refaat's avatar Mohammad Refaat Committed by Commit Bot

Formatting follow-up for cr/1791629

Change-Id: I6678cfc2882800c5b2e0f9e95625872ab8ed7885
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1819622
Auto-Submit: Mohammad Refaat <mrefaat@chromium.org>
Reviewed-by: default avatarEugene But <eugenebut@chromium.org>
Commit-Queue: Eugene But <eugenebut@chromium.org>
Cr-Commit-Position: refs/heads/master@{#699090}
parent 045bd9da
......@@ -2162,9 +2162,9 @@ enum class EnterTabSwitcherSnapshotResult {
BOOL showActivityIndicator = NO;
if (@available(iOS 13, *)) {
// TODO(crbug.com/632772): Visited links don't clearing doesn't require
// disabling web usage with iOS 13. Stop disabling web usage once iOS 12
// is not supported.
// TODO(crbug.com/632772): Visited links clearing doesn't require disabling
// web usage with iOS 13. Stop disabling web usage once iOS 12 is not
// supported.
showActivityIndicator = disableWebUsageDuringRemoval;
disableWebUsageDuringRemoval = NO;
}
......
......@@ -4389,8 +4389,8 @@ NSString* const kBrowserViewControllerSnackbarCategory =
targetDeviceName:[command targetDeviceName]];
}
- (void)showActivityOverlay:(BOOL)shown {
if (!shown) {
- (void)showActivityOverlay:(BOOL)show {
if (!show) {
[self.activityOverlayCoordinator stop];
self.activityOverlayCoordinator = nil;
} else if (!self.activityOverlayCoordinator) {
......
......@@ -112,9 +112,9 @@ class GURL;
// Sends the tab to another of the user's devices using the data in |command|.
- (void)sendTabToSelf:(SendTabToSelfCommand*)command;
// Show/Hide the activity indicator overlay that appears over the view to
// Shows/Hides the activity indicator overlay that appears over the view to
// prevent interaction with the web page.
- (void)showActivityOverlay:(BOOL)shown;
- (void)showActivityOverlay:(BOOL)show;
@end
......
......@@ -53,7 +53,7 @@ void WebFramesManagerImpl::OnWebViewUpdated(
RemoveAllWebFrames();
// TODO(crbug.com/956516): ScriptMessageHandlers should all be removed
// manually using |removeScriptMessageHandlerForName|, however this is not
// possible because of the cases where the webviewconfiguration is purged,
// possible because of the cases where the WKWebViewConfiguration is purged,
// in these cases the message router is deleted and it will not have
// message handlers for the web view.
[message_router removeAllScriptMessageHandlersForWebView:old_web_view];
......
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