Let Android port access properly sized images

chrome/browser/android/resource_mapper.cc

+// Copyright (c) 2013 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 "chrome/browser/android/resource_mapper.h"
+
+#include <map>
+
+#include "base/logging.h"
+#include "grit/theme_resources.h"
+
+const int ResourceMapper::kMissingID = -1;
+
+int ResourceMapper::MapFromChromiumID(int resource_id) {

[Yaron, 2013/05/17 22:36:16]

So every time you map and id you reconstruct the map? That seems wasteful. Can
we lazily construct it once?

[gone, 2013/05/20 18:48:10]

Done.  Would it make more sense as a static member variable, though?  Not seeing
static maps defined in header files or anything...

+  // Create the mapping.  IDs start at 0 to correspond to the array that gets
+  // built in the corresponding ResourceID Java class.
+  std::map<int, int> id_map;
+  int next_id = 0;
+#define DEFINE_RESOURCE_ID(c_id,java_id) id_map[c_id] = next_id++;
+#include "chrome/browser/android/resource_id.h"
+#undef DEFINE_RESOURCE_ID
+
+  // Search the map for the ID.
+  std::map<int, int>::iterator iterator = id_map.find(resource_id);
+  if (iterator == id_map.end()) {
+    // The resource couldn't be found.
+    NOTREACHED();
+    return kMissingID;
+  } else {
+    return iterator->second;
+  }
+}