Defect 248426: Speak rendered text when no selection is made on Mac

content/renderer/render_view_impl.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/renderer/render_view_impl.h"
 
 #include <algorithm>
 #include <cmath>
 
 #include "base/auto_reset.h"
@@ skipping 1016 matching lines @@
   else
     focused_plugin_id_ = -1;
 }
 #endif
 
 #if defined(OS_MACOSX)
 void RenderViewImpl::PluginFocusChanged(bool focused, int plugin_id) {
   Send(new ViewHostMsg_PluginFocusChanged(routing_id(), focused, plugin_id));
 }
 
+void RenderViewImpl::OnGetSmartClipDataFromRect(const gfx::Rect& rect) {
+  if (webview()) {

[Alexei Svitkine (slow), 2014/06/19 14:55:09]

Nit: Reverse cond and early return instead.

+    std::string text = webview()->getSmartClipData(rect).utf8();

[Alexei Svitkine (slow), 2014/06/19 14:55:09]

Is it possible to iterate on the chars without converting to utf8 first to find
chars_to_skip?

+    // Skips to the first valid char. SmartClip returns a string
+    // of Text along with its position

[Alexei Svitkine (slow), 2014/06/19 14:55:09]

Nit: Missing punctuation at the end.

Also, the comment does not explain the reason we're doing this.

+    if (text.size()) {

[Alexei Svitkine (slow), 2014/06/19 14:55:09]

Nit: !text.empty(), same below.

+      size_t chars_to_skip = 0;
+      for (size_t i = 0; i < text.length(); ++i) {
+        if (isalpha(text[i])) {
+          chars_to_skip = i;
+          break;
+        }
+      }
+      if (text.size() && chars_to_skip > 0) {
+        std::string trimmed_text = text.substr(chars_to_skip, text.length());
+        Send(new ViewMsg_ExtractedDataFromRect(routing_id(), trimmed_text));
+      }
+    }
+  }
+}
+
 void RenderViewImpl::StartPluginIme() {
   IPC::Message* msg = new ViewHostMsg_StartPluginIme(routing_id());
   // This message can be sent during event-handling, and needs to be delivered
   // within that context.
   msg->set_unblock(true);
   Send(msg);
 }
 #endif  // defined(OS_MACOSX)
 
 #endif  // ENABLE_PLUGINS
@@ skipping 82 matching lines @@
     IPC_MESSAGE_HANDLER(ViewMsg_SetHistoryLengthAndPrune,
                         OnSetHistoryLengthAndPrune)
     IPC_MESSAGE_HANDLER(ViewMsg_EnableViewSourceMode, OnEnableViewSourceMode)
     IPC_MESSAGE_HANDLER(ViewMsg_SetAccessibilityMode, OnSetAccessibilityMode)
     IPC_MESSAGE_HANDLER(ViewMsg_DisownOpener, OnDisownOpener)
     IPC_MESSAGE_HANDLER(ViewMsg_ReleaseDisambiguationPopupBitmap,
                         OnReleaseDisambiguationPopupBitmap)
     IPC_MESSAGE_HANDLER(ViewMsg_WindowSnapshotCompleted,
                         OnWindowSnapshotCompleted)
     IPC_MESSAGE_HANDLER(ViewMsg_SelectWordAroundCaret, OnSelectWordAroundCaret)
+
 #if defined(OS_ANDROID)
+    IPC_MESSAGE_HANDLER(ViewMsg_ExtractSmartClipData, OnExtractSmartClipData)
     IPC_MESSAGE_HANDLER(InputMsg_ActivateNearestFindResult,
                         OnActivateNearestFindResult)
     IPC_MESSAGE_HANDLER(ViewMsg_FindMatchRects, OnFindMatchRects)
     IPC_MESSAGE_HANDLER(ViewMsg_SelectPopupMenuItems, OnSelectPopupMenuItems)
     IPC_MESSAGE_HANDLER(ViewMsg_UpdateTopControlsState,
                         OnUpdateTopControlsState)
-    IPC_MESSAGE_HANDLER(ViewMsg_ExtractSmartClipData, OnExtractSmartClipData)
 #elif defined(OS_MACOSX)
+    IPC_MESSAGE_HANDLER(ViewMsg_GetSmartClipDataFromRect,
+                        OnGetSmartClipDataFromRect)
     IPC_MESSAGE_HANDLER(ViewMsg_PluginImeCompositionCompleted,
                         OnPluginImeCompositionCompleted)
     IPC_MESSAGE_HANDLER(ViewMsg_SelectPopupMenuItem, OnSelectPopupMenuItem)
     IPC_MESSAGE_HANDLER(ViewMsg_SetInLiveResize, OnSetInLiveResize)
     IPC_MESSAGE_HANDLER(ViewMsg_SetWindowVisibility, OnSetWindowVisibility)
     IPC_MESSAGE_HANDLER(ViewMsg_WindowFrameChanged, OnWindowFrameChanged)
 #endif
     // Adding a new message? Add platform independent ones first, then put the
     // platform specific ones at the end.
 
@@ skipping 2865 matching lines @@
     std::vector<gfx::Size> sizes;
     ConvertToFaviconSizes(icon_urls[i].sizes(), &sizes);
     if (!url.isEmpty())
       urls.push_back(
           FaviconURL(url, ToFaviconType(icon_urls[i].iconType()), sizes));
   }
   SendUpdateFaviconURL(urls);
 }
 
 }  // namespace content