Make H264BitReader ignore trailing zero bytes and stop bit.

content/common/gpu/media/h264_parser.cc

 // 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/common/gpu/media/h264_parser.h"
 
 #include "base/logging.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/stl_util.h"
 
@@ skipping 706 matching lines @@
   IN_RANGE_OR_RETURN(pps->pic_init_qs_minus26, -26, 25);
 
   READ_SE_OR_RETURN(&pps->chroma_qp_index_offset);
   IN_RANGE_OR_RETURN(pps->chroma_qp_index_offset, -12, 12);
   pps->second_chroma_qp_index_offset = pps->chroma_qp_index_offset;
 
   READ_BOOL_OR_RETURN(&pps->deblocking_filter_control_present_flag);
   READ_BOOL_OR_RETURN(&pps->constrained_intra_pred_flag);
   READ_BOOL_OR_RETURN(&pps->redundant_pic_cnt_present_flag);
 
-  if (br_.HasMoreRBSPData()) {
-    READ_BOOL_OR_RETURN(&pps->transform_8x8_mode_flag);
+  int temp;
+  if (br_.ReadBits(1, &temp)) {
+    pps->transform_8x8_mode_flag = temp != 0;

[Ami GONE FROM CHROMIUM, 2012/08/02 19:49:48]

!=0 unnecessary

[xiaomings, 2012/08/03 00:37:58]

Done.

     READ_BOOL_OR_RETURN(&pps->pic_scaling_matrix_present_flag);
 
     if (pps->pic_scaling_matrix_present_flag) {
       DVLOG(4) << "Picture scaling matrix present";
       res = ParsePPSScalingLists(*sps, pps.get());
       if (res != kOk)
         return res;
     }
 
     READ_SE_OR_RETURN(&pps->second_chroma_qp_index_offset);
@@ skipping 376 matching lines @@
 
     default:
       DVLOG(4) << "Unsupported SEI message";
       break;
   }
 
   return kOk;
 }
 
 }  // namespace content