ClientAdapter::OnSetFocus - When DevTools is opened, OnSetFocus is called before OnAfterCreated

The causes GetBrowserWrapper to return null when DevTools is opened, for now we'll just create the wrapper
and pass it directly (wrapper is disposed when it goes out of scope).

Author: amaitland

CefSharp.Core/Internals/ClientAdapter.cpp

@@ -993,9 +993,12 @@ namespace CefSharp
                 return false;
             }
 
-            auto browserWrapper = GetBrowserWrapper(browser->GetIdentifier(), browser->IsPopup());
+			//For DevTools (which is hosted as a popup) OnSetFocus is called before OnAfterCreated so we don't
+			// have a reference to the standard popup IBrowser wrapper, so we just pass a
+			// short term reference.
+			CefSharpBrowserWrapper browserWrapper(browser);
 
-            return handler->OnSetFocus(_browserControl, browserWrapper, (CefFocusSource)source);
+            return handler->OnSetFocus(_browserControl, %browserWrapper, (CefFocusSource)source);
         }
 
         void ClientAdapter::OnTakeFocus(CefRefPtr<CefBrowser> browser, bool next)

CefSharp/Handler/IFocusHandler.cs

@@ -21,7 +21,7 @@ public interface IFocusHandler
         /// Called when the browser component is requesting focus.
         /// </summary>
         /// <param name="chromiumWebBrowser">the ChromiumWebBrowser control</param>
-        /// <param name="browser">the browser object</param>
+        /// <param name="browser">the browser object, do not keep a reference to this object outside of this method</param>
         /// <param name="source">Indicates where the focus request is originating from.</param>
         /// <returns>Return false to allow the focus to be set or true to cancel setting the focus.</returns>
         bool OnSetFocus(IWebBrowser chromiumWebBrowser, IBrowser browser, CefFocusSource source);