Route 'Select All' OS method through RenderWidgetHostDelegate.

Route 'Select All' OS method through RenderWidgetHostDelegate.

This allows Cmd-A / "Select All" to be used on SSL interstitials on OSX.

BUG=507416
Committed: https://crrev.com/145aaa8a7762909442a684415333ebce4f65a229
Cr-Commit-Position: refs/heads/master@{#338248}

[lgarron, 2015-07-06 22:04:52]

lgarron@chromium.org changed reviewers:
+ dcheng@chromium.org, naskos.nasko83@gmail.com

[lgarron, 2015-07-06 22:04:53]

A followup to https://codereview.chromium.org/1162373002

Includes a new test. :-)

[lgarron, 2015-07-06 22:06:15]

Patchset #1 (id:1) has been deleted

[lgarron, 2015-07-06 22:11:05]

lgarron@chromium.org changed reviewers:
+ nasko@chromium.org
- naskos.nasko83@gmail.com

[lgarron, 2015-07-06 22:11:44]

Change reviewer email to Chromium Nasko. ;-)

[lgarron, 2015-07-06 22:44:28]

Patchset #3 (id:60001) has been deleted

[nasko, 2015-07-07 13:52:25]

LGTM

[mohsen, 2015-07-07 16:47:32]

https://codereview.chromium.org/1212373009/diff/80001/content/browser/frame_h...
File content/browser/frame_host/interstitial_page_impl_browsertest.cc (right):

https://codereview.chromium.org/1212373009/diff/80001/content/browser/frame_h...
content/browser/frame_host/interstitial_page_impl_browsertest.cc:400:
TearDownTestClipboard();
This line belongs to 'Paste' test.

[lgarron, 2015-07-07 17:46:34]

https://codereview.chromium.org/1212373009/diff/80001/content/browser/frame_h...
File content/browser/frame_host/interstitial_page_impl_browsertest.cc (right):

https://codereview.chromium.org/1212373009/diff/80001/content/browser/frame_h...
content/browser/frame_host/interstitial_page_impl_browsertest.cc:400:
TearDownTestClipboard();
On 2015/07/07 at 16:47:32, mohsen wrote:
> This line belongs to 'Paste' test.

Done.

I assumed this was placed in the last of the tests so that it can do a polite
teardown at the end if all the tests are run.
In any case, it's moot, since this CL now doesn't use the clipboard for this
test.

[lgarron, 2015-07-10 00:54:28]

The CQ bit was checked by lgarron@chromium.org to run a CQ dry run

[lgarron, 2015-07-10 00:54:28]

The patchset sent to the CQ was uploaded after l-g-t-m from nasko@chromium.org
Link to the patchset: https://codereview.chromium.org/1212373009/#ps100001
(title: "Move TearDownTestClipboard back to InterstitialPageImplTest.Paste")

[commit-bot: I haz the power, 2015-07-10 00:54:48]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1212373009/100001

[lgarron, 2015-07-10 00:55:21]

dcheng@: Nasko's LGTM is sufficient for landing, but you reviewed the CL that
initiated routing some OS functions through the RWHD. Would you mind sanity
checking this?

[commit-bot: I haz the power, 2015-07-10 02:28:48]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-07-10 02:28:49]

Dry run: This issue passed the CQ dry run.

[dcheng, 2015-07-10 06:03:52]

lgtm

[lgarron, 2015-07-10 06:09:02]

The CQ bit was checked by lgarron@chromium.org

[commit-bot: I haz the power, 2015-07-10 06:09:13]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1212373009/100001

[commit-bot: I haz the power, 2015-07-10 07:40:06]

Committed patchset #4 (id:100001)

[commit-bot: I haz the power, 2015-07-10 07:42:17]

Patchset 4 (id:??) landed as
https://crrev.com/145aaa8a7762909442a684415333ebce4f65a229
Cr-Commit-Position: refs/heads/master@{#338248}