Skip to content

Commit

Permalink
Revert 215210 "Remove webkit_chromium_resources.pak entries." because…
Browse files Browse the repository at this point in the history
… it was causing the chromium_builder_qa to fail.

I do not think the fix is complex, but I am not set up to build Chrome on Windows quickly

Builder output:
804>Time Elapsed 00:05:33.76
809>------ Build started: Project: chromium_builder_qa, Configuration: Release Win32 ------
809>Build started 8/2/2013 8:09:33 AM.
768>RC : fatal error RC1110: could not open ..\build\Release\obj\global_intermediate\webkit\webkit_chromium_resources.rc
768>  
768>
768>Build FAILED.

> Remove webkit_chromium_resources.pak entries.
> 
> webkit_chromium_resources.pak was kept along side with blink_resources.pak,
> while DumpRenderTree was being updated on the Blink side, since that already
> happened and Blink has rolled into Chromium, we should be able to remove this
> now.
> 
> BUG=265748
> TBR=abarth@chromium.org, thestig@chromium.org
> 
> Review URL: https://codereview.chromium.org/20628004

TBR=tfarina@chromium.org

Review URL: https://codereview.chromium.org/21937002

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@215311 0039d316-1c4b-4281-b951-d872f2087c98
  • Loading branch information
pkotwicz@chromium.org committed Aug 2, 2013
1 parent 45cf615 commit 27e1136
Show file tree
Hide file tree
Showing 14 changed files with 25 additions and 3 deletions.
1 change: 1 addition & 0 deletions android_webview/android_webview.gyp
Original file line number Diff line number Diff line change
Expand Up @@ -54,6 +54,7 @@
'<(SHARED_INTERMEDIATE_DIR)/net/net_resources.pak',
'<(SHARED_INTERMEDIATE_DIR)/ui/ui_resources/ui_resources_100_percent.pak',
'<(SHARED_INTERMEDIATE_DIR)/webkit/blink_resources.pak',
'<(SHARED_INTERMEDIATE_DIR)/webkit/webkit_chromium_resources.pak',
'<(SHARED_INTERMEDIATE_DIR)/webkit/webkit_resources_100_percent.pak',
],
},
Expand Down
1 change: 1 addition & 0 deletions chrome/chrome_dll.gypi
Original file line number Diff line number Diff line change
Expand Up @@ -148,6 +148,7 @@
'<(SHARED_INTERMEDIATE_DIR)/net/net_resources.rc',
'<(SHARED_INTERMEDIATE_DIR)/ui/ui_resources/ui_unscaled_resources.rc',
'<(SHARED_INTERMEDIATE_DIR)/webkit/blink_resources.rc',
'<(SHARED_INTERMEDIATE_DIR)/webkit/webkit_chromium_resources.rc',
],
'include_dirs': [
'<(DEPTH)/third_party/wtl/include',
Expand Down
1 change: 1 addition & 0 deletions chrome/chrome_repack_chrome.gypi
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,7 @@
'<(SHARED_INTERMEDIATE_DIR)/content/content_resources.pak',
'<(SHARED_INTERMEDIATE_DIR)/content/browser/tracing/tracing_resources.pak',
'<(SHARED_INTERMEDIATE_DIR)/webkit/blink_resources.pak',
'<(SHARED_INTERMEDIATE_DIR)/webkit/webkit_chromium_resources.pak',
],
}],
['enable_extensions==1', {
Expand Down
4 changes: 4 additions & 0 deletions chrome/chrome_tests.gypi
Original file line number Diff line number Diff line change
Expand Up @@ -455,6 +455,7 @@
'<(SHARED_INTERMEDIATE_DIR)/net/net_resources.rc',
'<(SHARED_INTERMEDIATE_DIR)/ui/ui_resources/ui_unscaled_resources.rc',
'<(SHARED_INTERMEDIATE_DIR)/webkit/blink_resources.rc',
'<(SHARED_INTERMEDIATE_DIR)/webkit/webkit_chromium_resources.rc',

'browser/ui/views/accessibility/browser_views_accessibility_browsertest.cc',
],
Expand Down Expand Up @@ -1923,6 +1924,7 @@
'<(SHARED_INTERMEDIATE_DIR)/net/net_resources.rc',
'<(SHARED_INTERMEDIATE_DIR)/ui/ui_resources/ui_unscaled_resources.rc',
'<(SHARED_INTERMEDIATE_DIR)/webkit/blink_resources.rc',
'<(SHARED_INTERMEDIATE_DIR)/webkit/webkit_chromium_resources.rc',
],
'include_dirs': [
'<(DEPTH)/third_party/wtl/include',
Expand Down Expand Up @@ -2161,6 +2163,7 @@
'<(SHARED_INTERMEDIATE_DIR)/net/net_resources.rc',
'<(SHARED_INTERMEDIATE_DIR)/ui/ui_resources/ui_unscaled_resources.rc',
'<(SHARED_INTERMEDIATE_DIR)/webkit/blink_resources.rc',
'<(SHARED_INTERMEDIATE_DIR)/webkit/webkit_chromium_resources.rc',
],
'include_dirs': [
'<(DEPTH)/third_party/wtl/include',
Expand Down Expand Up @@ -2544,6 +2547,7 @@
'<(SHARED_INTERMEDIATE_DIR)/net/net_resources.rc',
'<(SHARED_INTERMEDIATE_DIR)/ui/ui_resources/ui_unscaled_resources.rc',
'<(SHARED_INTERMEDIATE_DIR)/webkit/blink_resources.rc',
'<(SHARED_INTERMEDIATE_DIR)/webkit/webkit_chromium_resources.rc',
],
'include_dirs': [
'<(DEPTH)/third_party/wtl/include',
Expand Down
1 change: 1 addition & 0 deletions chrome/test/gpu/test_support_gpu.gypi
Original file line number Diff line number Diff line change
Expand Up @@ -60,6 +60,7 @@
'<(SHARED_INTERMEDIATE_DIR)/content/content_resources.rc',
'<(SHARED_INTERMEDIATE_DIR)/net/net_resources.rc',
'<(SHARED_INTERMEDIATE_DIR)/webkit/blink_resources.rc',
'<(SHARED_INTERMEDIATE_DIR)/webkit/webkit_chromium_resources.rc',
],
'conditions': [
['win_use_allocator_shim==1', {
Expand Down
1 change: 1 addition & 0 deletions content/DEPS
Original file line number Diff line number Diff line change
Expand Up @@ -31,6 +31,7 @@ include_rules = [
"+grit/content_resources.h",
"+grit/ui_resources.h",
"+grit/ui_strings.h",
"+grit/webkit_chromium_resources.h",
"+grit/webkit_resources.h",
"+grit/webkit_strings.h",
"+grit/webui_resources_map.h",
Expand Down
1 change: 1 addition & 0 deletions content/browser/gpu/test_support_gpu.gypi
Original file line number Diff line number Diff line change
Expand Up @@ -29,6 +29,7 @@
'<(SHARED_INTERMEDIATE_DIR)/content/content_resources.rc',
'<(SHARED_INTERMEDIATE_DIR)/net/net_resources.rc',
'<(SHARED_INTERMEDIATE_DIR)/webkit/blink_resources.rc',
'<(SHARED_INTERMEDIATE_DIR)/webkit/webkit_chromium_resources.rc',
],
'conditions': [
['win_use_allocator_shim==1', {
Expand Down
1 change: 1 addition & 0 deletions content/content_shell.gypi
Original file line number Diff line number Diff line change
Expand Up @@ -320,6 +320,7 @@
'<(SHARED_INTERMEDIATE_DIR)/ui/ui_strings/ui_strings_en-US.pak',
'<(SHARED_INTERMEDIATE_DIR)/webkit/devtools_resources.pak',
'<(SHARED_INTERMEDIATE_DIR)/webkit/blink_resources.pak',
'<(SHARED_INTERMEDIATE_DIR)/webkit/webkit_chromium_resources.pak',
'<(SHARED_INTERMEDIATE_DIR)/webkit/webkit_resources_100_percent.pak',
'<(SHARED_INTERMEDIATE_DIR)/webkit/webkit_strings_en-US.pak',
],
Expand Down
1 change: 1 addition & 0 deletions content/content_tests.gypi
Original file line number Diff line number Diff line change
Expand Up @@ -896,6 +896,7 @@
# resulting .res files get referenced multiple times.
'<(SHARED_INTERMEDIATE_DIR)/net/net_resources.rc',
'<(SHARED_INTERMEDIATE_DIR)/webkit/blink_resources.rc',
'<(SHARED_INTERMEDIATE_DIR)/webkit/webkit_chromium_resources.rc',
'<(SHARED_INTERMEDIATE_DIR)/webkit/webkit_strings_en-US.rc',
],
'dependencies': [
Expand Down
3 changes: 3 additions & 0 deletions tools/gritsettings/resource_ids
Original file line number Diff line number Diff line change
Expand Up @@ -66,6 +66,9 @@
"chrome_frame/resources/chrome_frame_resources.grd": {
"includes": [8000],
},
"third_party/WebKit/Source/WebKit/chromium/WebKit.grd": {
"includes": [8500],
},
"ui/base/strings/app_locale_settings.grd": {
"messages": [9000],
},
Expand Down
2 changes: 1 addition & 1 deletion webkit/child/webkitplatformsupport_impl.cc
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@
#include "base/synchronization/lock.h"
#include "base/time/time.h"
#include "content/public/common/webplugininfo.h"
#include "grit/blink_resources.h"
#include "grit/webkit_chromium_resources.h"
#include "grit/webkit_resources.h"
#include "grit/webkit_strings.h"
#include "net/base/data_url.h"
Expand Down
2 changes: 1 addition & 1 deletion webkit/common/cursors/webcursor_mac.mm
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@
#include "base/logging.h"
#include "base/mac/mac_util.h"
#include "base/mac/scoped_cftyperef.h"
#include "grit/blink_resources.h"
#include "grit/webkit_chromium_resources.h"
#include "skia/ext/skia_utils_mac.h"
#include "third_party/WebKit/public/platform/WebSize.h"
#include "third_party/WebKit/public/web/WebCursorInfo.h"
Expand Down
2 changes: 1 addition & 1 deletion webkit/support/platform_support_win.cc
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@
#include "base/strings/string16.h"
#include "base/strings/string_piece.h"
#include "base/win/resource_util.h"
#include "grit/blink_resources.h"
#include "grit/webkit_chromium_resources.h"
#include "grit/webkit_resources.h"
#include "ui/base/resource/resource_bundle.h"
#include "webkit/support/test_webkit_platform_support.h"
Expand Down
7 changes: 7 additions & 0 deletions webkit/webkit_resources.gyp
Original file line number Diff line number Diff line change
Expand Up @@ -18,6 +18,13 @@
},
'includes': [ '../build/grit_action.gypi' ],
},
{
'action_name': 'webkit_chromium_resources',
'variables': {
'grit_grd_file': '../third_party/WebKit/Source/WebKit/chromium/WebKit.grd',
},
'includes': [ '../build/grit_action.gypi' ],
},
{
'action_name': 'blink_chromium_resources',
'variables': {
Expand Down

0 comments on commit 27e1136

Please sign in to comment.