[content] Fix background color update handling in RWHVAura.

content/browser/renderer_host/render_widget_host_view_mac_unittest.mm

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/browser/renderer_host/render_widget_host_view_mac.h"
 
 #include <Cocoa/Cocoa.h>
 #include <stddef.h>
 #include <stdint.h>
 #include <tuple>
@@ skipping 1107 matching lines @@
   TestBrowserContext browser_context;
   MockRenderProcessHost* process_host =
       new MockRenderProcessHost(&browser_context);
   process_host->Init();
   MockRenderWidgetHostDelegate delegate;
   int32_t routing_id = process_host->GetNextRoutingID();
   MockRenderWidgetHostImpl* host =
       new MockRenderWidgetHostImpl(&delegate, process_host, routing_id);
   RenderWidgetHostViewMac* view = new RenderWidgetHostViewMac(host, false);
 
-  EXPECT_TRUE(view->GetBackgroundOpaque());
+  EXPECT_EQ(static_cast<unsigned>(SK_AlphaOPAQUE),
+            SkColorGetA(view->background_color()));
   EXPECT_TRUE([view->cocoa_view() isOpaque]);
 
   view->SetBackgroundColor(SK_ColorTRANSPARENT);
-  EXPECT_FALSE(view->GetBackgroundOpaque());
+  EXPECT_EQ(static_cast<unsigned>(SK_AlphaTRANSPARENT),

[danakj, 2017/02/28 17:52:31]

can EXPECT_EQ(TRANSPARENT, background_color()) here too

[Eric Seckler, 2017/03/01 10:22:09]

Done.

+            SkColorGetA(view->background_color()));
   EXPECT_FALSE([view->cocoa_view() isOpaque]);
 
   const IPC::Message* set_background;
   set_background = process_host->sink().GetUniqueMessageMatching(
       ViewMsg_SetBackgroundOpaque::ID);
   ASSERT_TRUE(set_background);
   std::tuple<bool> sent_background;
   ViewMsg_SetBackgroundOpaque::Read(set_background, &sent_background);
   EXPECT_FALSE(std::get<0>(sent_background));
 
   // Try setting it back.
   process_host->sink().ClearMessages();
   view->SetBackgroundColor(SK_ColorWHITE);
-  EXPECT_TRUE(view->GetBackgroundOpaque());
+  EXPECT_EQ(static_cast<unsigned>(SK_AlphaOPAQUE),

[danakj, 2017/02/28 17:52:31]

and EXPECT_EQ(WHITE, background_color()) if u like

[Eric Seckler, 2017/03/01 10:22:09]

Done.

+            SkColorGetA(view->background_color()));
   EXPECT_TRUE([view->cocoa_view() isOpaque]);
   set_background = process_host->sink().GetUniqueMessageMatching(
       ViewMsg_SetBackgroundOpaque::ID);
   ASSERT_TRUE(set_background);
   ViewMsg_SetBackgroundOpaque::Read(set_background, &sent_background);
   EXPECT_TRUE(std::get<0>(sent_background));
 
   host->ShutdownAndDestroyWidget(true);
 }
 
@@ skipping 483 matching lines @@
   // Verify that this IPC is asking for no monitoring or immediate updates.
   InputMsg_RequestCompositionUpdate::Read(composition_request_msg_for_child,
                                           &child_msg_params);
   is_child_msg_for_immediate_request = std::get<0>(child_msg_params);
   is_child_msg_for_monitor_request = std::get<1>(child_msg_params);
   EXPECT_FALSE(is_child_msg_for_immediate_request);
   EXPECT_FALSE(is_child_msg_for_monitor_request);
 }
 
 }  // namespace content