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

content/common/view_messages.h

 // 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.
 
 // IPC messages for page rendering.
 // Multiply-included message file, hence no include guard.
 
 #include "base/memory/shared_memory.h"
 #include "base/process/process.h"
 #include "base/strings/string16.h"
@@ skipping 866 matching lines @@
 // process to release the magnified image.
 IPC_MESSAGE_ROUTED1(ViewMsg_ReleaseDisambiguationPopupBitmap,
                     cc::SharedBitmapId /* id */)
 
 // Notifies the renderer that a snapshot has been retrieved.
 IPC_MESSAGE_ROUTED3(ViewMsg_WindowSnapshotCompleted,
                     int /* snapshot_id */,
                     gfx::Size /* size */,
                     std::vector<unsigned char> /* png */)
 
+// Sends an Async IPC request to webview for complete content of a

[jamesr, 2014/07/18 06:05:04]

nit: the fact that this is an async IPC and that it is being sent to a view are
all reflected in the IPC declaration:

IPC_MESSAGE_ROUTE0 means it's an async IPC (sync ones are _SYNC) and ViewMsg_
means it is targetted at a view.

can you shorten the comment up to just have the stuff that's particular to this?
 Maybe "Request the complete rendered text of the web page" or something along
those lines?

[sarka, 2014/07/19 06:28:36]

Done.

+// web page as plain text.
+IPC_MESSAGE_ROUTED0(ViewMsg_GetRenderedText)
+
 #if defined(OS_MACOSX)
 IPC_ENUM_TRAITS_MAX_VALUE(blink::ScrollerStyle, blink::ScrollerStyleOverlay)
 
 // Notification of a change in scrollbar appearance and/or behavior.
 IPC_MESSAGE_CONTROL5(ViewMsg_UpdateScrollbarTheme,
                      float /* initial_button_delay */,
                      float /* autoscroll_button_delay */,
                      bool /* jump_on_track_click */,
                      blink::ScrollerStyle /* preferred_scroller_style */,
                      bool /* redraw */)
@@ skipping 767 matching lines @@
                            uint32 /* font id */)
 
 // Informs the browser that a plugin has gained or lost focus.
 IPC_MESSAGE_ROUTED2(ViewHostMsg_PluginFocusChanged,
                     bool, /* focused */
                     int /* plugin_id */)
 
 // Instructs the browser to start plugin IME.
 IPC_MESSAGE_ROUTED0(ViewHostMsg_StartPluginIme)
 
+// Receives content of a web page as plain text.
+IPC_MESSAGE_ROUTED1(ViewMsg_GetRenderedTextCompleted, std::string);
+
 #elif defined(OS_WIN)
 // Request that the given font characters be loaded by the browser so it's
 // cached by the OS. Please see RenderMessageFilter::OnPreCacheFontCharacters
 // for details.
 IPC_SYNC_MESSAGE_CONTROL2_0(ViewHostMsg_PreCacheFontCharacters,
                             LOGFONT /* font_data */,
                             base::string16 /* characters */)
 #endif
 
 #if defined(OS_POSIX)
@@ skipping 13 matching lines @@
 // Since the browser keeps handles to the allocated transport DIBs, this
 // message is sent to tell the browser that it may release them when the
 // renderer is finished with them.
 IPC_MESSAGE_CONTROL1(ViewHostMsg_FreeTransportDIB,
                      TransportDIB::Id /* DIB id */)
 #endif
 
 // Adding a new message? Stick to the sort order above: first platform
 // independent ViewMsg, then ifdefs for platform specific ViewMsg, then platform
 // independent ViewHostMsg, then ifdefs for platform specific ViewHostMsg.