Finish making shell/common/... compatible with .clang-tidy.

shell/common/base64.cc

@@ -131,10 +131,12 @@ size_t Base64::Encode(const void* srcv, size_t length, void* dstv) {
     c = (c >> 6 | b << 2) & 0x3F;
     b = (b >> 4 | a << 4) & 0x3F;
     a = a >> 2;
+    // NOLINTBEGIN(clang-analyzer-core.NullDereference)
     *dst++ = encode[a];
     *dst++ = encode[b];
     *dst++ = encode[c];
     *dst++ = encode[d];
+    // NOLINTEND(clang-analyzer-core.NullDereference)
   }
   if (remainder > 0) {
     int k1 = 0;
@@ -145,10 +147,12 @@ size_t Base64::Encode(const void* srcv, size_t length, void* dstv) {
       k1 = b >> 4;
       k2 = (b << 2) & 0x3F;
     }
+    // NOLINTBEGIN(clang-analyzer-core.NullDereference)
     *dst++ = encode[a >> 2];
     *dst++ = encode[(k1 | a << 4) & 0x3F];
     *dst++ = encode[k2];
     *dst++ = encode[EncodePad];
+    // NOLINTEND(clang-analyzer-core.NullDereference)
   }
   return EncodedSize(length);
 }

shell/common/shell_test_platform_view_vulkan.cc

@@ -76,8 +76,7 @@ ShellTestPlatformViewVulkan::OffScreenSurface::OffScreenSurface(
     fml::RefPtr<vulkan::VulkanProcTable> vk,
     std::shared_ptr<ShellTestExternalViewEmbedder>
         shell_test_external_view_embedder)
-    : valid_(false),
-      vk_(std::move(vk)),
+    : vk_(std::move(vk)),
       shell_test_external_view_embedder_(
           std::move(shell_test_external_view_embedder)) {
   if (!vk_ || !vk_->HasAcquiredMandatoryProcAddresses()) {

shell/common/shell_test_platform_view_vulkan.h

@@ -50,7 +50,7 @@ class ShellTestPlatformViewVulkan : public ShellTestPlatformView {
     GrDirectContext* GetContext() override;
 
    private:
-    bool valid_;
+    bool valid_ = false;
     fml::RefPtr<vulkan::VulkanProcTable> vk_;
     std::shared_ptr<ShellTestExternalViewEmbedder>
         shell_test_external_view_embedder_;