From 11f1ab5eb495536bd72a83477b439e0a3cd00ffc Mon Sep 17 00:00:00 2001
From: dimas <myrna.zhou@gmail.com>
Date: Thu, 22 Jan 2015 15:34:06 +0700
Subject: [PATCH 1/3] Testing webbrowser inside listbox

---
 CefSharp.Wpf.Example/Views/BrowserTabView.xaml | 11 ++++++++---
 1 file changed, 8 insertions(+), 3 deletions(-)

diff --git a/CefSharp.Wpf.Example/Views/BrowserTabView.xaml b/CefSharp.Wpf.Example/Views/BrowserTabView.xaml
index 3f985a5717..e98473ed03 100644
--- a/CefSharp.Wpf.Example/Views/BrowserTabView.xaml
+++ b/CefSharp.Wpf.Example/Views/BrowserTabView.xaml
@@ -167,13 +167,16 @@
                         </VisualBrush>
                     </Rectangle.Fill>
                 </Rectangle>
-
+                <ListBox>
+                    <ListBoxItem Focusable="False">
                 <cefSharp:ChromiumWebBrowser x:Name="browser"
+                                  Height="500"
+                                  Width="700"
                                   Opacity="{Binding ElementName=opacitySlider, Path=Value}"
                                   Address="{Binding Address, Mode=TwoWay}"
                                   Title="{Binding Title, Mode=OneWayToSource}"
                                   WebBrowser="{Binding WebBrowser, Mode=OneWayToSource}">
-                    <!-- Just an example of how you may override the BrowserSettings. Disabled by default since it looks so
+                <!-- Just an example of how you may override the BrowserSettings. Disabled by default since it looks so
                          incredibly ugly... -->
                     <!--<cefSharp:ChromiumWebBrowser.BrowserSettings>
                         <CefSharp:BrowserSettings MinimumFontSize="36" />
@@ -207,7 +210,9 @@
                             <MenuItem Header="Paste" Command="{Binding WebBrowser.PasteCommand}"/>
                         </ContextMenu>
                     </FrameworkElement.ContextMenu>
-                </cefSharp:ChromiumWebBrowser>
+                        </cefSharp:ChromiumWebBrowser>
+                    </ListBoxItem>
+                </ListBox>
             </Grid>
         </DockPanel>
     </Grid>

From bd6f7937f92d77b73c825530e130f4f646225b20 Mon Sep 17 00:00:00 2001
From: dimas <myrna.zhou@gmail.com>
Date: Mon, 16 Mar 2015 18:50:56 +0800
Subject: [PATCH 2/3] Revert "Testing webbrowser inside listbox"

This reverts commit 11f1ab5eb495536bd72a83477b439e0a3cd00ffc.
---
 CefSharp.Wpf.Example/Views/BrowserTabView.xaml | 11 +++--------
 1 file changed, 3 insertions(+), 8 deletions(-)

diff --git a/CefSharp.Wpf.Example/Views/BrowserTabView.xaml b/CefSharp.Wpf.Example/Views/BrowserTabView.xaml
index 06735aee46..383daf0ea8 100644
--- a/CefSharp.Wpf.Example/Views/BrowserTabView.xaml
+++ b/CefSharp.Wpf.Example/Views/BrowserTabView.xaml
@@ -167,17 +167,14 @@
                         </VisualBrush>
                     </Rectangle.Fill>
                 </Rectangle>
-                <ListBox>
-                    <ListBoxItem Focusable="False">
+
                 <cefSharp:ChromiumWebBrowser x:Name="browser"
-                                  Height="500"
-                                  Width="700"
                                   Opacity="{Binding ElementName=opacitySlider, Path=Value}"
                                   Address="{Binding Address, Mode=TwoWay}"
                                   Title="{Binding Title, Mode=OneWayToSource}"
                                   AllowDrop="True"
                                   WebBrowser="{Binding WebBrowser, Mode=OneWayToSource}">
-                <!-- Just an example of how you may override the BrowserSettings. Disabled by default since it looks so
+                    <!-- Just an example of how you may override the BrowserSettings. Disabled by default since it looks so
                          incredibly ugly... -->
                     <!--<cefSharp:ChromiumWebBrowser.BrowserSettings>
                         <CefSharp:BrowserSettings MinimumFontSize="36" />
@@ -211,9 +208,7 @@
                             <MenuItem Header="Paste" Command="{Binding WebBrowser.PasteCommand}"/>
                         </ContextMenu>
                     </FrameworkElement.ContextMenu>
-                        </cefSharp:ChromiumWebBrowser>
-                    </ListBoxItem>
-                </ListBox>
+                </cefSharp:ChromiumWebBrowser>
             </Grid>
         </DockPanel>
     </Grid>

From 06ac7f76419a821db95918e1e3a81ed847eb6b3b Mon Sep 17 00:00:00 2001
From: dimas <myrna.zhou@gmail.com>
Date: Mon, 16 Mar 2015 18:58:21 +0800
Subject: [PATCH 3/3] Adding more condition to OnPreviewKey so Ctrl+A will work
 when web browser is put inside a listbox

---
 CefSharp.Wpf/ChromiumWebBrowser.cs | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/CefSharp.Wpf/ChromiumWebBrowser.cs b/CefSharp.Wpf/ChromiumWebBrowser.cs
index e639b56faa..2296e70efe 100644
--- a/CefSharp.Wpf/ChromiumWebBrowser.cs
+++ b/CefSharp.Wpf/ChromiumWebBrowser.cs
@@ -1134,8 +1134,10 @@ private void OnPreviewKey(KeyEventArgs e)
             // trigger the appropriate WM_ messages handled by our SourceHook, so we have to handle these extra keys here.
             // Hooking the Tab key like this makes the tab focusing in essence work like
             // KeyboardNavigation.TabNavigation="Cycle"; you will never be able to Tab out of the web browser control.
+            // We also add the condition to allow ctrl+a to work when the web browser control is put inside listbox.
             if (e.Key == Key.Tab || e.Key == Key.Home || e.Key == Key.End  || e.Key == Key.Up
-                                 || e.Key == Key.Down || e.Key == Key.Left || e.Key == Key.Right)
+                                 || e.Key == Key.Down || e.Key == Key.Left || e.Key == Key.Right
+                                 || (e.Key == Key.A && Keyboard.Modifiers == ModifierKeys.Control))
             {
                 var modifiers = GetModifiers(e);
                 var message = (int)(e.IsDown ? WM.KEYDOWN : WM.KEYUP);