Loading: bulk style errors fix in core/loader

third_party/WebKit/Source/core/loader/NavigationScheduler.cpp

 /*
  * Copyright (C) 2006, 2007, 2008, 2009, 2010 Apple Inc. All rights reserved.
  * Copyright (C) 2008 Nokia Corporation and/or its subsidiary(-ies)
  * Copyright (C) 2008, 2009 Torch Mobile Inc. All rights reserved.
  * (http://www.torchmobile.com/)
  * Copyright (C) 2009 Adam Barth. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
@@ skipping 80 matching lines @@
   int value = gestureToken && gestureToken->hasGestures()
                   ? type + ScheduledLastEntry
                   : type;
   scheduledNavigationClobberHistogram.count(value);
 
   DEFINE_STATIC_LOCAL(
       CustomCountHistogram, scheduledClobberAbortTimeHistogram,
       ("Navigation.Scheduled.MaybeCausedAbort.Time", 1, 10000, 50));
   double navigationStart =
       frame->loader().provisionalDocumentLoader()->timing().navigationStart();
-  if (navigationStart)
+  if (navigationStart) {
     scheduledClobberAbortTimeHistogram.count(monotonicallyIncreasingTime() -
                                              navigationStart);
+  }
 }
 
 }  // namespace
 
 unsigned NavigationDisablerForUnload::s_navigationDisableCount = 0;
 
 class ScheduledNavigation
     : public GarbageCollectedFinalized<ScheduledNavigation> {
   WTF_MAKE_NONCOPYABLE(ScheduledNavigation);
 
@@ skipping 43 matching lines @@
                          const String& url,
                          bool replacesCurrentItem,
                          bool isLocationChange)
       : ScheduledNavigation(delay,
                             originDocument,
                             replacesCurrentItem,
                             isLocationChange),
         m_url(url),
         m_shouldCheckMainWorldContentSecurityPolicy(
             CheckContentSecurityPolicy) {
-    if (ContentSecurityPolicy::shouldBypassMainWorld(originDocument))
+    if (ContentSecurityPolicy::shouldBypassMainWorld(originDocument)) {
       m_shouldCheckMainWorldContentSecurityPolicy =
           DoNotCheckContentSecurityPolicy;
+    }
   }
 
   void fire(LocalFrame* frame) override {
     std::unique_ptr<UserGestureIndicator> gestureIndicator =
         createUserGestureIndicator();
     FrameLoadRequest request(originDocument(), m_url, "_self",
                              m_shouldCheckMainWorldContentSecurityPolicy);
     request.setReplacesCurrentItem(replacesCurrentItem());
     request.setClientRedirect(ClientRedirectPolicy::ClientRedirect);
 
@@ skipping 24 matching lines @@
   bool shouldStartTimer(LocalFrame* frame) override {
     return frame->document()->loadEventFinished();
   }
 
   void fire(LocalFrame* frame) override {
     std::unique_ptr<UserGestureIndicator> gestureIndicator =
         createUserGestureIndicator();
     FrameLoadRequest request(originDocument(), url(), "_self");
     request.setReplacesCurrentItem(replacesCurrentItem());
     if (equalIgnoringFragmentIdentifier(frame->document()->url(),
-                                        request.resourceRequest().url()))
+                                        request.resourceRequest().url())) {
       request.resourceRequest().setCachePolicy(
           WebCachePolicy::ValidatingCacheData);
+    }
     request.setClientRedirect(ClientRedirectPolicy::ClientRedirect);
     maybeLogScheduledNavigationClobber(
         ScheduledNavigationType::ScheduledRedirect, frame);
     frame->loader().load(request);
   }
 
  private:
   ScheduledRedirect(double delay,
                     Document* originDocument,
                     const String& url,
@@ skipping 114 matching lines @@
 NavigationScheduler::NavigationScheduler(LocalFrame* frame)
     : m_frame(frame),
       m_navigateTaskFactory(
           CancellableTaskFactory::create(this,
                                          &NavigationScheduler::navigateTask)),
       m_frameType(m_frame->isMainFrame()
                       ? WebScheduler::NavigatingFrameType::kMainFrame
                       : WebScheduler::NavigatingFrameType::kChildFrame) {}
 
 NavigationScheduler::~NavigationScheduler() {
-  if (m_navigateTaskFactory->isPending())
+  if (m_navigateTaskFactory->isPending()) {
     Platform::current()->currentThread()->scheduler()->removePendingNavigation(
         m_frameType);
+  }
 }
 
 bool NavigationScheduler::locationChangePending() {
   return m_redirect && m_redirect->isLocationChange();
 }
 
 bool NavigationScheduler::isNavigationScheduledWithin(double interval) const {
   return m_redirect && m_redirect->delay() <= interval;
 }
 
@@ skipping 31 matching lines @@
 
 void NavigationScheduler::scheduleRedirect(double delay, const String& url) {
   if (!shouldScheduleNavigation(url))
     return;
   if (delay < 0 || delay > INT_MAX / 1000)
     return;
   if (url.isEmpty())
     return;
 
   // We want a new back/forward list item if the refresh timeout is > 1 second.
-  if (!m_redirect || delay <= m_redirect->delay())
+  if (!m_redirect || delay <= m_redirect->delay()) {
     schedule(
         ScheduledRedirect::create(delay, m_frame->document(), url, delay <= 1));
+  }
 }
 
 bool NavigationScheduler::mustReplaceCurrentItem(LocalFrame* targetFrame) {
   // Non-user navigation before the page has finished firing onload should not
   // create a new back/forward item. See https://webkit.org/b/42861 for the
   // original motivation for this.
   if (!targetFrame->document()->loadEventFinished() &&
       !UserGestureIndicator::utilizeUserGesture())
     return true;
 
@@ skipping 126 matching lines @@
   m_navigateTaskFactory->cancel();
   m_redirect.clear();
 }
 
 DEFINE_TRACE(NavigationScheduler) {
   visitor->trace(m_frame);
   visitor->trace(m_redirect);
 }
 
 }  // namespace blink