Loading: bulk style errors fix in core/loader

third_party/WebKit/Source/core/loader/appcache/ApplicationCacheHost.cpp

 /*
  * Copyright (C) 2009 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 80 matching lines @@
 
   WrappedResourceRequest wrapped(request);
 
   const WebApplicationCacheHost* spawningHost = nullptr;
   Frame* spawningFrame = frame.tree().parent();
   if (!spawningFrame || !spawningFrame->isLocalFrame())
     spawningFrame = frame.loader().opener();
   if (!spawningFrame || !spawningFrame->isLocalFrame())
     spawningFrame = &frame;
   if (DocumentLoader* spawningDocLoader =
-          toLocalFrame(spawningFrame)->loader().documentLoader())
+          toLocalFrame(spawningFrame)->loader().documentLoader()) {
     spawningHost = spawningDocLoader->applicationCacheHost()
                        ? spawningDocLoader->applicationCacheHost()->m_host.get()
                        : nullptr;
+  }
 
   m_host->willStartMainResourceRequest(wrapped, spawningHost);
 
   // NOTE: The semantics of this method, and others in this interface, are
   // subtly different than the method names would suggest. For example, in this
   // method never returns an appcached response in the SubstituteData out
   // argument, instead we return the appcached response thru the usual resource
   // loading pipeline.
 }
 
@@ skipping 76 matching lines @@
 }
 
 void ApplicationCacheHost::notifyApplicationCache(
     EventID id,
     int progressTotal,
     int progressDone,
     WebApplicationCacheHost::ErrorReason errorReason,
     const String& errorURL,
     int errorStatus,
     const String& errorMessage) {
-  if (id != kProgressEvent)
+  if (id != kProgressEvent) {
     InspectorInstrumentation::updateApplicationCacheStatus(
         m_documentLoader->frame());
+  }
 
   if (m_defersEvents) {
     // Event dispatching is deferred until document.onload has fired.
     m_deferredEvents.append(DeferredEvent(id, progressTotal, progressDone,
                                           errorReason, errorURL, errorStatus,
                                           errorMessage));
     return;
   }
   dispatchDOMEvent(id, progressTotal, progressDone, errorReason, errorURL,
                    errorStatus, errorMessage);
@@ skipping 44 matching lines @@
     const String& errorURL,
     int errorStatus,
     const String& errorMessage) {
   if (!m_domApplicationCache)
     return;
 
   const AtomicString& eventType = ApplicationCache::toEventType(id);
   if (eventType.isEmpty() || !m_domApplicationCache->getExecutionContext())
     return;
   Event* event = nullptr;
-  if (id == kProgressEvent)
+  if (id == kProgressEvent) {
     event = ProgressEvent::create(eventType, true, progressDone, progressTotal);
-  else if (id == kErrorEvent)
+  } else if (id == kErrorEvent) {
     event = ApplicationCacheErrorEvent::create(errorReason, errorURL,
                                                errorStatus, errorMessage);
-  else
+  } else {
     event = Event::create(eventType);
+  }
   m_domApplicationCache->dispatchEvent(event);
 }
 
 ApplicationCacheHost::Status ApplicationCacheHost::getStatus() const {
   return m_host ? static_cast<Status>(m_host->getStatus()) : kUncached;
 }
 
 bool ApplicationCacheHost::update() {
   return m_host ? m_host->startUpdate() : false;
 }
 
 bool ApplicationCacheHost::swapCache() {
   bool success = m_host ? m_host->swapCache() : false;
-  if (success)
+  if (success) {
     InspectorInstrumentation::updateApplicationCacheStatus(
         m_documentLoader->frame());
+  }
   return success;
 }
 
 void ApplicationCacheHost::abort() {
   if (m_host)
     m_host->abort();
 }
 
 bool ApplicationCacheHost::isApplicationCacheEnabled() {
   DCHECK(m_documentLoader->frame());
@@ skipping 31 matching lines @@
   notifyApplicationCache(kErrorEvent, 0, 0, reason, url.string(), status,
                          message);
 }
 
 DEFINE_TRACE(ApplicationCacheHost) {
   visitor->trace(m_domApplicationCache);
   visitor->trace(m_documentLoader);
 }
 
 }  // namespace blink