predictors: Version the resource_prefetch_predictor database.

chrome/browser/predictors/resource_prefetch_predictor.proto

 // Copyright 2016 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.
 
 // Protocol buffers used for storing in SQLite.
+// CAUTION: When any change is done here, bump kDatabaseVersion in
+//          resource_prefetch_predictor.cc.

[pasko, 2016/08/26 15:06:44]

I like it less because in case the file name changes or the constant moves
somewhere, this will become inconsistent.

I guess the advantage is less bloat in the protobuf? Or was it just hard to
access the proto enum value in resource_prefetch_predictor.cc?

[Benoit L, 2016/08/26 16:37:02]

Acknowledged.

 
 syntax = "proto2";
 
 package chrome_browser_predictors;
 
 // Required in Chrome.
 option optimize_for = LITE_RUNTIME;
 
 message Metadata {
   optional string main_page_url = 1;
@@ skipping 37 matching lines @@
   optional string resource_url = 2;
   optional ResourceType resource_type = 3;
   optional uint32 number_of_hits = 4;
   optional uint32 number_of_misses = 5;
   optional uint32 consecutive_misses = 6;
   optional double average_position = 7;
   optional Priority priority = 8;
   optional bool has_validators = 9;
   optional bool always_revalidate = 10;
 }