OOP PDF: Forward keyboard events to a constrained web dialog's initiator's delegate.

OOP PDF: Forward keyboard events to a constrained web dialog's initiator's delegate.

WebDialogWebContentsDelegateViews forwards keyboard events that it
receives to the Browser for its initiating WebContents. The
out-of-process PDF plugin is implemented as a nested WebContents
containing the extension implementation within the WebContents of the
tab.

When a print-preview dialog is launched for a PDF from the
out-of-process plugin, the dialog's initiator is the nested WebContents,
which does not have an associated Browser. This causes keyboard events
to be dropped, breaking keyboard shortcuts. This CL fixes this by
changing WebDialogWebContentsDelegateViews to forward keyboard events to
its initiator's delegate.

BUG=423453
TEST=Run with --out-of-process-pdf. Open a PDF and use the print toolbar
button to launch the print preview dialog. From this dialog, keyboard
shortcuts should work, e.g. escape should cancel the dialog.

Committed: https://crrev.com/7ed352b8552d5549410d53b0eef14b68aa41bf1a
Cr-Commit-Position: refs/heads/master@{#305125}

[Sam McNally, 2014-11-20 03:43:40]

Patchset #1 (id:1) has been deleted

[Sam McNally, 2014-11-20 03:52:00]

sammc@chromium.org changed reviewers:
+ sky@chromium.org

[sky, 2014-11-20 15:06:08]

LGTM

[Sam McNally, 2014-11-20 23:00:40]

The CQ bit was checked by sammc@chromium.org

[commit-bot: I haz the power, 2014-11-20 23:02:12]

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

[commit-bot: I haz the power, 2014-11-21 00:43:49]

Committed patchset #1 (id:20001)

[commit-bot: I haz the power, 2014-11-21 00:44:21]

Patchset 1 (id:??) landed as
https://crrev.com/7ed352b8552d5549410d53b0eef14b68aa41bf1a
Cr-Commit-Position: refs/heads/master@{#305125}

[dgarrett, 2014-11-21 01:09:18]

On 2014/11/21 00:44:21, I haz the power (commit-bot) wrote:
> Patchset 1 (id:??) landed as
> https://crrev.com/7ed352b8552d5549410d53b0eef14b68aa41bf1a
> Cr-Commit-Position: refs/heads/master@{#305125}

Just btw, I think you had the wrong bug number in the description.