General code cleanup of ChromeClient-related classes.

Source/core/page/ChromeClient.cpp

 /*
  * Copyright (C) 2006, 2007, 2009, 2011 Apple Inc. All rights reserved.
  * Copyright (C) 2008, 2010 Nokia Corporation and/or its subsidiary(-ies)
  * Copyright (C) 2012, Samsung Electronics. All rights reserved.
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
  * License as published by the Free Software Foundation; either
  * version 2 of the License, or (at your option) any later version.
  *
@@ skipping 25 matching lines @@
 #include <algorithm>
 
 namespace blink {
 
 void ChromeClient::setWindowRectWithAdjustment(const IntRect& pendingRect)
 {
     IntRect screen = screenInfo().availableRect;
     IntRect window = pendingRect;
 
     IntSize minimumSize = minimumWindowSize();
-    // Let size 0 pass through, since that indicates default size, not minimum size.
+    // Let size 0 pass through, since that indicates default size, not minimum
+    // size.
     if (window.width())
         window.setWidth(std::min(std::max(minimumSize.width(), window.width()), screen.width()));
     if (window.height())
         window.setHeight(std::min(std::max(minimumSize.height(), window.height()), screen.height()));
 
     // Constrain the window position within the valid screen area.
     window.setX(std::max(screen.x(), std::min(window.x(), screen.maxX() - window.width())));
     window.setY(std::max(screen.y(), std::min(window.y(), screen.maxY() - window.height())));
     setWindowRect(window);
 }
@@ skipping 76 matching lines @@
 bool ChromeClient::runJavaScriptPrompt(LocalFrame* frame, const String& prompt, const String& defaultValue, String& result)
 {
     ASSERT(frame);
     if (!canRunModalIfDuringPageDismissal(frame->tree().top(), ChromeClient::PromptDialog, prompt))
         return false;
     return openJavaScriptDialog(this, &ChromeClient::runJavaScriptPromptInternal, *frame, prompt, defaultValue, result);
 }
 
 void ChromeClient::mouseDidMoveOverElement(const HitTestResult& result)
 {
-    if (result.innerNode()) {
-        if (result.innerNode()->document().isDNSPrefetchEnabled())
-            prefetchDNS(result.absoluteLinkURL().host());
-    }
+    if (result.innerNode() && result.innerNode()->document().isDNSPrefetchEnabled())
+        prefetchDNS(result.absoluteLinkURL().host());
+
     showMouseOverURL(result);
 
     setToolTip(result);
 }
 
 void ChromeClient::setToolTip(const HitTestResult& result)
 {
-    // First priority is a potential toolTip representing a spelling or grammar error
+    // First priority is a potential toolTip representing a spelling or grammar
+    // error.
     TextDirection toolTipDirection;
     String toolTip = result.spellingToolTip(toolTipDirection);
 
-    // Next we'll consider a tooltip for element with "title" attribute
+    // Next we'll consider a tooltip for element with "title" attribute.
     if (toolTip.isEmpty())
         toolTip = result.title(toolTipDirection);
 
-    // Lastly, for <input type="file"> that allow multiple files, we'll consider a tooltip for the selected filenames
+    // Lastly, for <input type="file"> that allow multiple files, we'll consider
+    // a tooltip for the selected filenames.
     if (toolTip.isEmpty()) {
         if (Node* node = result.innerNode()) {
             if (isHTMLInputElement(*node)) {
                 HTMLInputElement* input = toHTMLInputElement(node);
                 toolTip = input->defaultToolTip();
 
                 // FIXME: We should obtain text direction of tooltip from
                 // ChromeClient or platform. As of October 2011, all client
                 // implementations don't use text direction information for
                 // ChromeClient::setToolTip. We'll work on tooltip text
                 // direction during bidi cleanup in form inputs.
                 toolTipDirection = LTR;
             }
         }
     }
 
     setToolTip(toolTip, toolTipDirection);
 }
 
 void ChromeClient::print(LocalFrame* frame)
 {
     // Defer loads in case the client method runs a new event loop that would
-    // otherwise cause the load to continue while we're in the middle of executing JavaScript.
+    // otherwise cause the load to continue while we're in the middle of
+    // executing JavaScript.
     ScopedPageLoadDeferrer deferrer;
 
     printInternal(frame);
 }
 
 void ChromeClient::setCursor(const Cursor& cursor)
 {
     m_lastSetMouseCursorForTesting = cursor;
     setCursorInternal(cursor);
 }
 
 Cursor ChromeClient::getLastSetCursorForTesting() const
 {
     return m_lastSetMouseCursorForTesting;
 }
 
 } // namespace blink