Reflow comments in the Push and Notification modules.

third_party/WebKit/Source/modules/notifications/Notification.cpp

 /*
  * Copyright (C) 2013 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 50 matching lines @@
 WebNotificationManager* notificationManager() {
   return Platform::current()->notificationManager();
 }
 
 }  // namespace
 
 Notification* Notification::create(ExecutionContext* context,
                                    const String& title,
                                    const NotificationOptions& options,
                                    ExceptionState& exceptionState) {
-  // The Web Notification constructor may be disabled through a runtime feature. The
-  // behavior of the constructor is changing, but not completely agreed upon yet.
+  // The Notification constructor may be disabled through a runtime feature when
+  // the platform does not support non-persistent notifications.
   if (!RuntimeEnabledFeatures::notificationConstructorEnabled()) {
     exceptionState.throwTypeError(
         "Illegal constructor. Use ServiceWorkerRegistration.showNotification() "
         "instead.");
     return nullptr;
   }
 
-  // The Web Notification constructor may not be used in Service Worker contexts.
+  // The Notification constructor may not be used in Service Worker contexts.
   if (context->isServiceWorkerGlobalScope()) {
     exceptionState.throwTypeError("Illegal constructor.");
     return nullptr;
   }
 
   if (!options.actions().isEmpty()) {
     exceptionState.throwTypeError(
         "Actions are only supported for persistent notifications shown using "
         "ServiceWorkerRegistration.showNotification().");
     return nullptr;
@@ skipping 120 matching lines @@
   UserGestureIndicator gestureIndicator(DefinitelyProcessingNewUserGesture);
   ScopedWindowFocusAllowedIndicator windowFocusAllowed(getExecutionContext());
   dispatchEvent(Event::create(EventTypeNames::click));
 }
 
 void Notification::dispatchErrorEvent() {
   dispatchEvent(Event::create(EventTypeNames::error));
 }
 
 void Notification::dispatchCloseEvent() {
-  // The notification will be showing when the user initiated the close, or it will be
-  // closing if the developer initiated the close.
+  // The notification should be Showing when the user initiated the close, or it

[harkness, 2016/10/04 14:31:16]

How about "if a" instead of "when the"?

[Peter Beverloo, 2016/10/04 14:38:04]

Done ("if the")

+  // should be Closing when the developer initiated the close.
   if (m_state != State::Showing && m_state != State::Closing)
     return;
 
   m_state = State::Closed;
   dispatchEvent(Event::create(EventTypeNames::close));
 }
 
 String Notification::title() const {
   return m_data.title;
 }
@@ skipping 85 matching lines @@
         break;
       default:
         NOTREACHED() << "Unknown action type: " << m_data.actions[i].type;
     }
 
     action.setAction(m_data.actions[i].action);
     action.setTitle(m_data.actions[i].title);
     action.setIcon(m_data.actions[i].icon.string());
     action.setPlaceholder(m_data.actions[i].placeholder);
 
-    // Not just the sequence of actions itself, but also the actions contained within the
-    // sequence should be frozen per the Web Notification specification.
+    // Both the Action dictionaries themselves and the sequence they'll be
+    // returned in are expected to the frozen. This cannot be done with WebIDL.
     actions[i] =
         freezeV8Object(toV8(action, scriptState), scriptState->isolate());
   }
 
   return actions;
 }
 
 String Notification::permissionString(
     mojom::blink::PermissionStatus permission) {
   switch (permission) {
@@ skipping 56 matching lines @@
 }
 
 DEFINE_TRACE(Notification) {
   visitor->trace(m_prepareShowMethodRunner);
   visitor->trace(m_loader);
   EventTargetWithInlineData::trace(visitor);
   ActiveDOMObject::trace(visitor);
 }
 
 }  // namespace blink