Commit e17ef94d authored by nasko's avatar nasko Committed by Commit bot

Remove unnecessary crash keys.

Since bug 630103 was fixed, these crash keys are no longer necessary.

BUG=630103

Review-Url: https://codereview.chromium.org/2841203003
Cr-Commit-Position: refs/heads/master@{#468017}
parent 8ccdc38b
...@@ -120,13 +120,6 @@ size_t RegisterWebViewCrashKeys() { ...@@ -120,13 +120,6 @@ size_t RegisterWebViewCrashKeys() {
{"newframe_replicated_origin", kSmallSize}, {"newframe_replicated_origin", kSmallSize},
{"newframe_oopifs_possible", kSmallSize}, {"newframe_oopifs_possible", kSmallSize},
// Temporary for https://crbug.com/630103.
{"origin_mismatch_url", kLargeSize},
{"origin_mismatch_origin", kMediumSize},
{"origin_mismatch_transition", kSmallSize},
{"origin_mismatch_redirects", kSmallSize},
{"origin_mismatch_same_page", kSmallSize},
// Temporary for https://crbug.com/612711. // Temporary for https://crbug.com/612711.
{"aci_wrong_sp_extension_id", kSmallSize}, {"aci_wrong_sp_extension_id", kSmallSize},
......
...@@ -183,13 +183,6 @@ size_t RegisterCrashKeysHelper() { ...@@ -183,13 +183,6 @@ size_t RegisterCrashKeysHelper() {
{"newframe_replicated_origin", kSmallSize}, {"newframe_replicated_origin", kSmallSize},
{"newframe_oopifs_possible", kSmallSize}, {"newframe_oopifs_possible", kSmallSize},
// Temporary for https://crbug.com/630103.
{"origin_mismatch_url", crash_keys::kLargeSize},
{"origin_mismatch_origin", crash_keys::kMediumSize},
{"origin_mismatch_transition", crash_keys::kSmallSize},
{"origin_mismatch_redirects", crash_keys::kSmallSize},
{"origin_mismatch_same_page", crash_keys::kSmallSize},
// Temporary for https://crbug.com/612711. // Temporary for https://crbug.com/612711.
{"aci_wrong_sp_extension_id", kSmallSize}, {"aci_wrong_sp_extension_id", kSmallSize},
......
...@@ -230,13 +230,6 @@ size_t RegisterChromeCrashKeys() { ...@@ -230,13 +230,6 @@ size_t RegisterChromeCrashKeys() {
{"newframe_replicated_origin", kSmallSize}, {"newframe_replicated_origin", kSmallSize},
{"newframe_oopifs_possible", kSmallSize}, {"newframe_oopifs_possible", kSmallSize},
// Temporary for https://crbug.com/630103.
{"origin_mismatch_url", crash_keys::kLargeSize},
{"origin_mismatch_origin", crash_keys::kMediumSize},
{"origin_mismatch_transition", crash_keys::kSmallSize},
{"origin_mismatch_redirects", crash_keys::kSmallSize},
{"origin_mismatch_same_page", crash_keys::kSmallSize},
// Temporary for https://crbug.com/612711. // Temporary for https://crbug.com/612711.
{"aci_wrong_sp_extension_id", kSmallSize}, {"aci_wrong_sp_extension_id", kSmallSize},
......
...@@ -97,13 +97,6 @@ size_t RegisterCastCrashKeys() { ...@@ -97,13 +97,6 @@ size_t RegisterCastCrashKeys() {
{"newframe_replicated_origin", ::crash_keys::kSmallSize}, {"newframe_replicated_origin", ::crash_keys::kSmallSize},
{"newframe_oopifs_possible", ::crash_keys::kSmallSize}, {"newframe_oopifs_possible", ::crash_keys::kSmallSize},
// Temporary for https://crbug.com/630103.
{"origin_mismatch_url", ::crash_keys::kLargeSize},
{"origin_mismatch_origin", ::crash_keys::kMediumSize},
{"origin_mismatch_transition", ::crash_keys::kSmallSize},
{"origin_mismatch_redirects", ::crash_keys::kSmallSize},
{"origin_mismatch_same_page", ::crash_keys::kSmallSize},
// Temporary for https://crbug.com/612711. // Temporary for https://crbug.com/612711.
{"aci_wrong_sp_extension_id", ::crash_keys::kSmallSize}, {"aci_wrong_sp_extension_id", ::crash_keys::kSmallSize},
......
...@@ -5111,16 +5111,6 @@ void RenderFrameImpl::SendDidCommitProvisionalLoad( ...@@ -5111,16 +5111,6 @@ void RenderFrameImpl::SendDidCommitProvisionalLoad(
if (params.origin.scheme() != url::kFileScheme || if (params.origin.scheme() != url::kFileScheme ||
!render_view_->GetWebkitPreferences() !render_view_->GetWebkitPreferences()
.allow_universal_access_from_file_urls) { .allow_universal_access_from_file_urls) {
base::debug::SetCrashKeyValue("origin_mismatch_url", params.url.spec());
base::debug::SetCrashKeyValue("origin_mismatch_origin",
params.origin.Serialize());
base::debug::SetCrashKeyValue("origin_mismatch_transition",
base::IntToString(params.transition));
base::debug::SetCrashKeyValue("origin_mismatch_redirects",
base::IntToString(params.redirects.size()));
base::debug::SetCrashKeyValue(
"origin_mismatch_same_page",
base::IntToString(params.was_within_same_document));
CHECK(params.origin.IsSamePhysicalOriginWith(url::Origin(params.url))) CHECK(params.origin.IsSamePhysicalOriginWith(url::Origin(params.url)))
<< " url:" << params.url << " origin:" << params.origin; << " url:" << params.url << " origin:" << params.origin;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment