Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(86)

Side by Side Diff: third_party/WebKit/Source/web/WebFrameSerializerImpl.cpp

Issue 1878493002: ASSERT_NOT_REACHED() -> NOTREACHED() in web. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 /* 1 /*
2 * Copyright (C) 2009 Google Inc. All rights reserved. 2 * Copyright (C) 2009 Google Inc. All rights reserved.
3 * 3 *
4 * Redistribution and use in source and binary forms, with or without 4 * Redistribution and use in source and binary forms, with or without
5 * modification, are permitted provided that the following conditions are 5 * modification, are permitted provided that the following conditions are
6 * met: 6 * met:
7 * 7 *
8 * * Redistributions of source code must retain the above copyright 8 * * Redistributions of source code must retain the above copyright
9 * notice, this list of conditions and the following disclaimer. 9 * notice, this list of conditions and the following disclaimer.
10 * * Redistributions in binary form must reproduce the above 10 * * Redistributions in binary form must reproduce the above
(...skipping 409 matching lines...) Expand 10 before | Expand all | Expand 10 after
420 // Process end tag of element. 420 // Process end tag of element.
421 endTagToString(toElement(node), param); 421 endTagToString(toElement(node), param);
422 break; 422 break;
423 case Node::TEXT_NODE: 423 case Node::TEXT_NODE:
424 saveHTMLContentToBuffer(createMarkup(node), param); 424 saveHTMLContentToBuffer(createMarkup(node), param);
425 break; 425 break;
426 case Node::ATTRIBUTE_NODE: 426 case Node::ATTRIBUTE_NODE:
427 case Node::DOCUMENT_NODE: 427 case Node::DOCUMENT_NODE:
428 case Node::DOCUMENT_FRAGMENT_NODE: 428 case Node::DOCUMENT_FRAGMENT_NODE:
429 // Should not exist. 429 // Should not exist.
430 ASSERT_NOT_REACHED(); 430 NOTREACHED();
431 break; 431 break;
432 // Document type node can be in DOM? 432 // Document type node can be in DOM?
433 case Node::DOCUMENT_TYPE_NODE: 433 case Node::DOCUMENT_TYPE_NODE:
434 param->haveSeenDocType = true; 434 param->haveSeenDocType = true;
435 default: 435 default:
436 // For other type node, call default action. 436 // For other type node, call default action.
437 saveHTMLContentToBuffer(createMarkup(node), param); 437 saveHTMLContentToBuffer(createMarkup(node), param);
438 break; 438 break;
439 } 439 }
440 } 440 }
(...skipping 44 matching lines...) Expand 10 before | Expand all | Expand 10 after
485 // Report empty contents for invalid URLs. 485 // Report empty contents for invalid URLs.
486 m_client->didSerializeDataForFrame( 486 m_client->didSerializeDataForFrame(
487 WebCString(), WebFrameSerializerClient::CurrentFrameIsFinished); 487 WebCString(), WebFrameSerializerClient::CurrentFrameIsFinished);
488 } 488 }
489 489
490 DCHECK(m_dataBuffer.isEmpty()); 490 DCHECK(m_dataBuffer.isEmpty());
491 return didSerialization; 491 return didSerialization;
492 } 492 }
493 493
494 } // namespace blink 494 } // namespace blink
OLDNEW
« no previous file with comments | « third_party/WebKit/Source/web/WebDOMFileSystem.cpp ('k') | third_party/WebKit/Source/web/WebFrameWidgetImpl.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698