[Telemetry] OS X: Properly clean up power measurement

In the page cycler measurement, the call that verifies that a page has finished loading could throw a timeout exception causing the power metric to not be properly shut down.

This can cause many instances of powermetric to run in parallel.

This patch adds additional cleanup code to the powermetric to make sure the backing utility is well dead.

BUG=339162

Review URL: https://codereview.chromium.org/192243002

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@256078 0039d316-1c4b-4281-b951-d872f2087c98
cc: Don't try to reduce main thread latency in smoothness takes priority mode
    
If the compositor is in smoothness takes priority mode, do not try to
skip BeginMainFrames to reduce main thread latency because the two goals
are mutually exclusive. This fixes the problem of constantly going in
and out of low latency mode on pages where the DOM is constantly
modified in response to (impl-side) scrolling.
    
TEST=The green box on http://jsbin.com/sifisogo/11?quiet=1 should not
     move choppily (but may still move out of sync with the red box.)
BUG=309648

Review URL: https://codereview.chromium.org/188753002

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@256079 0039d316-1c4b-4281-b951-d872f2087c98
SVN changes up to revision 256078
Cast: reland 189423002 - Cap max receiver log size per rtcp packet ...

Fixed expectations on RtcpSenderTest. Longer term we may need to
refactor this to make it more robust. Ran cast_unittests locally to
verify the test now pass.


Cast: Cap max receiver log size per rtcp packet to 200 bytes.

This is a short term fix to hopefully reduce the effects of packets
queueing up on the receiver side. Previously we try fill up the RTCP
packet with as many receiver log as possible so we can have redundancy.
The redundancy that has been observed was more than 30 per message, and
we don't need this much redundancy. 200 is a good number where each
log message will be sent with redundancy of around 2-3, without losing
too many (< 1%) messages, assuming default rtcp interval settings.
That's fine because we don't need the logs to be completely reliable.


BUG=349265

Review URL: https://codereview.chromium.org/190823003

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@256080 0039d316-1c4b-4281-b951-d872f2087c98
Mac: Fix favicon positioning

setIconView has a side-effect of positioning the icon view depending on the
current state.
With the recent change introducing the CA throbber, we made it more efficient
and keep the same icon view all the time and just change its image.
This fix moves the positioning code to setIconImage to regain previous
side-effect.

BUG=349639

Review URL: https://codereview.chromium.org/191153003

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@256081 0039d316-1c4b-4281-b951-d872f2087c98
SVN changes up to revision 256080
SVN changes up to revision 256081