Fix implicit access to raw pointer of scoped_refptr.

chrome/browser/themes/theme_service.cc

Index: chrome/browser/themes/theme_service.cc
diff --git a/chrome/browser/themes/theme_service.cc b/chrome/browser/themes/theme_service.cc
index 11e8f1767a9b9aa2ed06f5019d260690fb9e900e..88081094f2f37c0b7fac1eb1531b892adcdf3132 100644
--- a/chrome/browser/themes/theme_service.cc
+++ b/chrome/browser/themes/theme_service.cc
@@ -568,7 +568,8 @@ void ThemeService::LoadThemePrefs() {
                            ? chrome::kThemePackMaterialDesignFilename
                            : chrome::kThemePackFilename);
     SwapThemeSupplier(BrowserThemePack::BuildFromDataPack(path, current_id));
-    loaded_pack = theme_supplier_ != nullptr;
+    if (theme_supplier_)

[danakj, 2016/05/10 21:37:02]

i'm meh about this change

[scheib, 2016/05/10 22:25:00]

Ok. I considered also:
- a static_cast here: too verbose and distracting.
- dropping the 'loaded_pack' variable entirely: but lost the semantic meaning
original author intended of why theme_supplier implies themes are loaded.

+      loaded_pack = true;
   }
 
   if (loaded_pack) {