Removed the frame parameter from didChangeBrandColor

public/web/WebFrameClient.h

 /*
  * Copyright (C) 2011, 2012 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 249 matching lines @@
 
     // Called upon update to scroll position, document state, and other
     // non-navigational events related to the data held by WebHistoryItem.
     // WARNING: This method may be called very frequently.
     virtual void didUpdateCurrentHistoryItem(WebLocalFrame*) { }
 
     // The frame's manifest has changed.
     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 ----------------------------------------------------------------
 
     // Called to retrieve the provider of desktop notifications.
     virtual WebNotificationPresenter* notificationPresenter() { return 0; }
 
 
     // Editing -------------------------------------------------------------
 
@@ skipping 237 matching lines @@
     // Access the embedder API for (client-based) screen orientation client .
     virtual WebScreenOrientationClient* webScreenOrientationClient() { return 0; }
 
 protected:
     virtual ~WebFrameClient() { }
 };
 
 } // namespace blink
 
 #endif