Removed the frame parameter from didChangeBrandColor

public/web/WebFrameClient.h

Index: public/web/WebFrameClient.h
diff --git a/public/web/WebFrameClient.h b/public/web/WebFrameClient.h
index 3f76ede04069ffb058db2ce7d56801b2d4cf6ed3..c75af85905b11ef96cd2721abf456cf2209cc0b1 100644
--- a/public/web/WebFrameClient.h
+++ b/public/web/WebFrameClient.h
@@ -267,7 +267,7 @@ public:
     virtual void didChangeManifest(WebLocalFrame*) { }
 
     // The frame's brand color has changed.
-    virtual void didChangeBrandColor(WebLocalFrame*) { }
+    virtual void didChangeBrandColor() { }

[abarth-chromium, 2014/06/17 21:52:48]

This change isn't correct.  We always pass WebLocalFrame* to WebFrameClient
functions.  You're welcome to ignore the argument on the Chromium side.

[jam, 2014/06/17 22:45:05]

Adam: this was originally done since the WebFrameClient implementation on the
content side (RenderViewImpl) responded to many WebFrames. Now, there is one
RenderFrameImpl per WebFrameClient, so passing a pointer to the frame is
redundant. We are planning on removing all these, but were going to wait for the
repo merge since it'll be trivial to do it then as compared to now. but for new
APIs, I don't see much reason. you can see that for many of the methods that
we've moved here from WebViewClient, we have avoided that extra parameter.

 
 
     // Misc ----------------------------------------------------------------