add previously focused node to WebClient::focusedNodeChanged

public/web/WebViewClient.h

Index: public/web/WebViewClient.h
diff --git a/public/web/WebViewClient.h b/public/web/WebViewClient.h
index 908bbdb34545988de0589ab7f242653ccc0a4a1f..92b46569b62cb16909e63d47c283627d8c384106 100644
--- a/public/web/WebViewClient.h
+++ b/public/web/WebViewClient.h
@@ -186,8 +186,11 @@ public:
     virtual void focusNext() { }
     virtual void focusPrevious() { }
 
-    // Called when a new node gets focused.
+    // TODO(estade): remove this function.

[Mike West, 2015/01/19 08:52:16]

Totally tiny nit: We use FIXME in Blink.

[Evan Stade, 2015/01/20 22:23:54]

Done.

     virtual void focusedNodeChanged(const WebNode&) { }
+    // Called when a new node gets focused. First argument is the previously focused node, the
+    // second is the newly focused node. Either can be null.
+    virtual void focusedNodeChanged(const WebNode&, const WebNode&) { }

[dmazzoni, 2015/01/20 07:31:52]

Nit: Blink leaves out argument names in header files when they're obvious or
redundant, but not when it's helpful to distinguish between two arguments. I
think these should be named.

[Evan Stade, 2015/01/20 22:23:54]

Done.

 
     // Indicates two things:
     //   1) This view may have a new layout now.