Implementation of brand-color

public/web/WebFrame.h

 /*
  * Copyright (C) 2009 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 138 matching lines @@
 
     // For a WebFrame with contents being rendered in another process, this
     // sets a layer for use by the in-process compositor. WebLayer should be
     // null if the content is being rendered in the current process.
     virtual void setRemoteWebLayer(blink::WebLayer*) = 0;
 
     // Initializes the various client interfaces.
     virtual void setPermissionClient(WebPermissionClient*) = 0;
     virtual void setSharedWorkerRepositoryClient(WebSharedWorkerRepositoryClient*) = 0;
 
+    // The brand color (if any) specified by the document loaded in this frame.
+    virtual WebColor brandColor() const = 0;

[abarth-chromium, 2014/06/13 05:40:41]

This is a property of the document (as you say in the comment).  It should be on
WebDocument.  (Also, please leave two blank lines between sections.)

[michaelbai, 2014/06/13 15:55:06]

I added this method here because
- I saw the similar property like iconURL() is in WebFrame.
- Also I added didChangeBrandColor() in WebFrameClient, it is suitable to have
brandColor() in WebDocument in this case? 


On 2014/06/13 05:40:41, abarth wrote:

 
     // Geometry -----------------------------------------------------------
 
     // NOTE: These routines do not force page layout so their results may
     // not be accurate if the page layout is out-of-date.
 
     // If set to false, do not draw scrollbars on this frame's view.
     virtual void setCanHaveScrollbars(bool) = 0;
 
     // The scroll offset from the top-left corner of the frame in pixels.
@@ skipping 523 matching lines @@
     WebPrivateOwnPtr<OpenedFrameTracker> m_openedFrameTracker;
 };
 
 #if BLINK_IMPLEMENTATION
 WebCore::Frame* toWebCoreFrame(WebFrame*);
 #endif
 
 } // namespace blink
 
 #endif